<div dir="ltr">Thank you for the feedback, Gedare. <div><br></div><div>Sending in the new patch soon.</div><div><br></div><div>Thanks</div><div>Mritunjay</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 27, 2020 at 9:19 PM 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">Mritunjay,<br>
<br>
On Wed, Aug 26, 2020 at 3:58 PM Mritunjay Sharma<br>
<<a href="mailto:mritunjaysharma394@gmail.com" target="_blank">mritunjaysharma394@gmail.com</a>> wrote:<br>
><br>
> Modified as per the suggestions of Dr. Gedare. Please do review<br>
> Signed-off-by: Mritunjay Sharma <<a href="mailto:mritunjaysharma394@gmail.com" target="_blank">mritunjaysharma394@gmail.com</a>><br>
> ---<br>
>  .../epics/{epics-7-1.cfg => epics-7.0-1.cfg}  |  5 ++--<br>
>  rtems/config/epics/epics-base.bset            | 14 +----------<br>
>  .../config/{epics-7-1.cfg => epics-7.0-1.cfg} | 24 ++++++++++++-------<br>
>  3 files changed, 19 insertions(+), 24 deletions(-)<br>
>  rename rtems/config/epics/{epics-7-1.cfg => epics-7.0-1.cfg} (87%)<br>
>  rename source-builder/config/{epics-7-1.cfg => epics-7.0-1.cfg} (64%)<br>
><br>
> diff --git a/rtems/config/epics/epics-7-1.cfg b/rtems/config/epics/epics-7.0-1.cfg<br>
> similarity index 87%<br>
> rename from rtems/config/epics/epics-7-1.cfg<br>
> rename to rtems/config/epics/epics-7.0-1.cfg<br>
> index 30ed9cb..34270ee 100644<br>
> --- a/rtems/config/epics/epics-7-1.cfg<br>
> +++ b/rtems/config/epics/epics-7.0-1.cfg<br>
> @@ -1,5 +1,5 @@<br>
>  #<br>
> -# EPICS 7<br>
> +# EPICS 7.0<br>
>  #<br>
><br>
>  %if %{release} == %{nil}<br>
> @@ -15,8 +15,7 @@<br>
><br>
>  %hash sha512 epics-base-%{epics_version}.tar.gz 63edef9d3019e8253b81610286578a1e917cc9df75db3e8f5aaf99210b4fa842be296cf7cbd384726b81b648c4e0d041c4ef266ae81a580940d7613ce8ce48e0<br>
><br>
> -#<br>
>  #<br>
>  # EPICS Build configuration<br>
>  #<br>
> -%include %{_configdir}/epics-7-1.cfg<br>
> +%include %{_configdir}/epics-7.0-1.cfg<br>
> diff --git a/rtems/config/epics/epics-base.bset b/rtems/config/epics/epics-base.bset<br>
> index e56fc6a..7ee3815 100644<br>
> --- a/rtems/config/epics/epics-base.bset<br>
> +++ b/rtems/config/epics/epics-base.bset<br>
> @@ -1,26 +1,14 @@<br>
>  #<br>
>  # Build set for EPICS<br>
>  #<br>
> -<br>
>  %define release 1<br>
><br>
> -#<br>
> -# RTEMS Version<br>
> -#<br>
> -%define rtems_version 5<br>
> -<br>
> -#<br>
> -# The RTEMS URL paths.<br>
> -#<br>
> -%include rtems-urls.bset<br>
> -<br>
>  #<br>
>  # The RTEMS Package defines<br>
>  #<br>
>  %include rtems-package.bset<br>
><br>
> -<br>
>  #<br>
>  # Build EPICS.<br>
>  #<br>
> -epics/epics-7-1 .<br>
> \ No newline at end of file<br>
> +epics/epics-7.0-1<br>
> diff --git a/source-builder/config/epics-7-1.cfg b/source-builder/config/epics-7.0-1.cfg<br>
> similarity index 64%<br>
> rename from source-builder/config/epics-7-1.cfg<br>
> rename to source-builder/config/epics-7.0-1.cfg<br>
> index 2a52ea5..2a06008 100644<br>
> --- a/source-builder/config/epics-7-1.cfg<br>
> +++ b/source-builder/config/epics-7.0-1.cfg<br>
> @@ -27,7 +27,7 @@ URL:    <a href="https://epics.mpg.de/" rel="noreferrer" target="_blank">https://epics.mpg.de/</a><br>
>  %prep<br>
>    build_top=$(pwd)<br>
><br>
> -  source_dir_epics="epics-base-%{epics_version}"<br>
> + # source_dir_epics="epics-base-%{epics_version}"<br>
you can keep this variable, that wasn't my point.<br>
<br>
><br>
>    %source setup epics-base -q -n epics-base-%{epics_version}<br>
I was wondering if you should/could use the variable here?<br>
<br>
><br>
> @@ -36,22 +36,30 @@ URL:          <a href="https://epics.mpg.de/" rel="noreferrer" target="_blank">https://epics.mpg.de/</a><br>
>  %build<br>
>    build_top=$(pwd)<br>
><br>
> -  cd ${source_dir_epics}<br>
> +  cd epics-base-%{epics_version}<br>
<br>
and here. not that the name is likely to change, but it is good<br>
programming to use a variable to store something that gets used<br>
repeatedly.<br>
<br>
><br>
>    %{build_build_flags}<br>
><br>
>    #<br>
> -  # Using macros to dynamically path to RTEMS_BASE and RTEMS_VERSION using --with-rtems-bsp<br>
> +  # Using macros to dynamically path to RTEMS_BASE and RTEMS_VERSION using<br>
> +  # --with-rtems-bsp<br>
>    #<br>
> -  %{__make} RTEMS_BASE=%{_prefix} RTEMS_VERSION=%{rtems_version} CROSS_COMPILER_TARGET_ARCHS=RTEMS-%{with_rtems_bsp}<br>
> +  %{__make} RTEMS_BASE=%{_prefix} \<br>
> +   RTEMS_VERSION=%{rtems_version} \<br>
> +   CROSS_COMPILER_TARGET_ARCHS=RTEMS-%{with_rtems_bsp}<br>
indent these broken lines, probably two "nesting levels" (maybe 4 spaces?)<br>
<br>
><br>
>    cd ${build_top}<br>
><br>
> -%install<br>
> +#%install<br>
>    build_top=$(pwd)<br>
><br>
> -  rm -rf $SB_BUILD_ROOT<br>
> +  %{__rmdir} $SB_BUILD_ROOT<br>
> +<br>
> +  cd epics-base-%{epics_version}<br>
keep source_dir-epics<br>
<br>
> +<br>
> +  %{__make} DESTDIR=$SB_BUILD_ROOT \<br>
> +  RTEMS_BASE=%{_prefix} \<br>
> +  RTEMS_VERSION=%{rtems_version} \<br>
> +  CROSS_COMPILER_TARGET_ARCHS=RTEMS-%{with_rtems_bsp} install<br>
ditto<br>
<br>
><br>
> -  cd ${source_dir_epics}<br>
> -  %{__make} DESTDIR=$SB_BUILD_ROOT RTEMS_BASE=%{_prefix} RTEMS_VERSION=%{rtems_version} CROSS_COMPILER_TARGET_ARCHS=RTEMS-%{with_rtems_bsp}<br>
>    cd ${build_top}<br>
> --<br>
<br>
after you make these changes and fix the URL, send a new single patch<br>
with all changes squashed together.<br>
<br>
> 2.25.1<br>
><br>
</blockquote></div>