<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Probably want to put most of this file into<br>./source-builder/config/renode-common-1.cfg<br>and set parameters for it (version) from a new file<br>./bare/config/devel/renode-1.13.3-1.cfg</blockquote><div><br>okay, I will revise this, thanks for the suggestion<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <span style="color:rgb(80,0,80)">> > +  ./build.sh<br></span>This looks brittle. Does their script handle host detection, etc?</blockquote><div><br>The script does handle host detection, etc: <a href="https://github.com/renode/renode/blob/master/build.sh#L212">https://github.com/renode/renode/blob/master/build.sh#L212</a><br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <span style="color:rgb(80,0,80)">> > +++ b/bare/config/devel/renode.bse</span><span style="color:rgb(80,0,80)">t<br></span>Add the version to the build set name and pass that through to the<br>build instructions.</blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <span style="color:rgb(80,0,80)">> > +devel/renode-git-1<br></span>the "-git" name is usually used to build the git head or a specific commit.<br>SInce you build a release version, you should prefer to use the<br>version numbering to identify what gets built.</blockquote><div><br>I will revise this, thanks </div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 12, 2023 at 4:17 AM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Jul 11, 2023 at 12:01 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
><br>
> This probably requires some documentation, as it requires several<br>
> packages that are not going to be typically installed already by RTEMS<br>
> users/developers. Chris, is there any place to document required<br>
> dependencies/packages for a build set?<br>
><br>
> The patch has a whitespace problem<br>
><br>
> +# Renode source<br>
> +#<br>
>  It might be a non-ASCII character, as it gets eaten by email but is<br>
> in the patch file.<br>
><br>
> Gedare<br>
><br>
> On Mon, Jul 10, 2023 at 5:13 AM Muhammad Sulthan Mazaya<br>
> <<a href="mailto:msulthanmazaya@gmail.com" target="_blank">msulthanmazaya@gmail.com</a>> wrote:<br>
> ><br>
> > An implementation of renode rsb installation config. It uses renode's<br>
> > github release as source and build the package using their build script.<br>
> ><br>
> > ---<br>
> >  bare/config/devel/renode-git-1.cfg | 47 ++++++++++++++++++++++++++++++<br>
> >  bare/config/devel/renode.bset      |  9 ++++++<br>
> >  2 files changed, 56 insertions(+)<br>
> >  create mode 100644 bare/config/devel/renode-git-1.cfg<br>
> >  create mode 100644 bare/config/devel/renode.bset<br>
> ><br>
> > diff --git a/bare/config/devel/renode-git-1.cfg b/bare/config/devel/renode-git-1.cfg<br>
> > new file mode 100644<br>
> > index 0000000..df424a6<br>
> > --- /dev/null<br>
> > +++ b/bare/config/devel/renode-git-1.cfg<br>
Probably want to put most of this file into<br>
./source-builder/config/renode-common-1.cfg<br>
and set parameters for it (version) from a new file<br>
./bare/config/devel/renode-1.13.3-1.cfg<br>
<br>
I would recommend that you follow the design for how qemu gets built.<br>
<br>
> > @@ -0,0 +1,47 @@<br>
> > +#<br>
> > +# Renode from github<br>
> > +#<br>
> > +<br>
> > +%if %{release} == %{nil}<br>
> > + %define release 1<br>
> > +%endif<br>
> > +<br>
> > +%define renode_version 1.13.3<br>
> > +<br>
> > +Name:      renode-%{renode_version}-%{_host}-%{release}<br>
> > +Summary:   Renode v%{renode_version}<br>
> > +Version:   %{renode_version}<br>
> > +Release:   %{release}<br>
> > +URL:      <a href="http://www.renode.io" rel="noreferrer" target="_blank">http://www.renode.io</a><br>
> > +<br>
> > +#<br>
> > +# Renode source<br>
> > +#<br>
> > +%source set renode <a href="https://github.com/renode/renode/releases/download/v%%7Brenode_version%7D/renode_%%7Brenode_version%7D_source.tar.xz" rel="noreferrer" target="_blank">https://github.com/renode/renode/releases/download/v%{renode_version}/renode_%{renode_version}_source.tar.xz</a><br>
> > +<br>
> > +#<br>
> > +# Prepare the source code.<br>
> > +#<br>
> > +%prep<br>
> > +  build_top=$(pwd)<br>
> > +<br>
> > +  source_dir_renode="renode_%{renode_version}_source"<br>
> > +  %source setup renode -q -n renode_%{renode_version}_source<br>
> > +<br>
> > +  cd ${build_top}<br>
> > +<br>
> > +%build<br>
> > +  build_top=$(pwd)<br>
> > +<br>
> > +  cd ${source_dir_renode}<br>
> > +  ./build.sh<br>
This looks brittle. Does their script handle host detection, etc?<br>
<br>
> > +<br>
> > +  cd ${build_top}<br>
> > +<br>
> > +%install<br>
> > +  build_top=$(pwd)<br>
> > +<br>
> > +  mkdir -p %{_bindir}<br>
> > +  cp -r ./${source_dir_renode}/* %{_bindir}<br>
> > +<br>
> > +  cd ${build_top}<br>
> > diff --git a/bare/config/devel/renode.bset b/bare/config/devel/renode.bset<br>
> > new file mode 100644<br>
> > index 0000000..d64a2a2<br>
> > --- /dev/null<br>
> > +++ b/bare/config/devel/renode.bset<br>
Add the version to the build set name and pass that through to the<br>
build instructions.<br>
<br>
> > @@ -0,0 +1,9 @@<br>
> > +#<br>
> > +# Build set for Renode<br>
> > +#<br>
> > +<br>
> > +%if %{release} == %{nil}<br>
> > +    %define release 1<br>
> > +%endif<br>
> > +<br>
> > +devel/renode-git-1<br>
the "-git" name is usually used to build the git head or a specific commit.<br>
SInce you build a release version, you should prefer to use the<br>
version numbering to identify what gets built.<br>
<br>
> > --<br>
> > 2.34.1<br>
> ><br>
> > _______________________________________________<br>
> > devel mailing list<br>
> > <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>