<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 27, 2020 at 3:04 AM Mritunjay Sharma <<a href="mailto:mritunjaysharma394@gmail.com">mritunjaysharma394@gmail.com</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"><div dir="ltr"><div dir="ltr">Thank you so much for taking time to review the PATCH. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 26, 2020 at 8:56 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">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, Aug 25, 2020 at 4:36 PM Mritunjay Sharma<br>
<<a href="mailto:mritunjaysharma394@gmail.com" target="_blank">mritunjaysharma394@gmail.com</a>> wrote:<br>
><br>
> This patch add supports for EPICS7 Package to RTEMS RSB.<br>
> Signed-off-by: Mritunjay Sharma <<a href="mailto:mritunjaysharma394@gmail.com" target="_blank">mritunjaysharma394@gmail.com</a>><br>
> ---<br>
>  rtems/config/epics/epics-7-1.cfg    | 22 +++++++++++<br>
>  rtems/config/epics/epics-base.bset  | 26 +++++++++++++<br>
>  source-builder/config/epics-7-1.cfg | 57 +++++++++++++++++++++++++++++<br>
>  3 files changed, 105 insertions(+)<br>
>  create mode 100644 rtems/config/epics/epics-7-1.cfg<br>
>  create mode 100644 rtems/config/epics/epics-base.bset<br>
>  create mode 100644 source-builder/config/epics-7-1.cfg<br>
><br>
> diff --git a/rtems/config/epics/epics-7-1.cfg b/rtems/config/epics/epics-7-1.cfg<br>
> new file mode 100644<br>
> index 0000000..30ed9cb<br>
> --- /dev/null<br>
> +++ b/rtems/config/epics/epics-7-1.cfg<br>
<br>
should this file be epics-7.0-1.cfg?<br>
<br></blockquote><div>Yes, thanks for highlighting it. Fixed this. </div><div>  </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> @@ -0,0 +1,22 @@<br>
> +#<br>
> +# EPICS 7<br>
EPICS 7.0?<br>
<br>
> +#<br>
> +<br>
> +%if %{release} == %{nil}<br>
> + %define release 1<br>
> +%endif<br>
> +<br>
> +%include %{_configdir}/rtems-bsp.cfg<br>
> +<br>
> +#<br>
> +# EPICS Version<br>
> +#<br>
> +%define epics_version 7.0<br>
> +<br>
> +%hash sha512 epics-base-%{epics_version}.tar.gz 63edef9d3019e8253b81610286578a1e917cc9df75db3e8f5aaf99210b4fa842be296cf7cbd384726b81b648c4e0d041c4ef266ae81a580940d7613ce8ce48e0<br>
> +<br>
> +#<br>
> +#<br>
Only one blank comment line.<br>
<br></blockquote><div>Fixed. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +# EPICS Build configuration<br>
> +#<br>
> +%include %{_configdir}/epics-7-1.cfg<br>
> diff --git a/rtems/config/epics/epics-base.bset b/rtems/config/epics/epics-base.bset<br>
> new file mode 100644<br>
> index 0000000..e56fc6a<br>
> --- /dev/null<br>
> +++ b/rtems/config/epics/epics-base.bset<br>
> @@ -0,0 +1,26 @@<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>
is this correct?<br>
<br>
Do you want to apply this to the 5 branch of rsb?<br>
<br></blockquote><div>I have removed this line. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +<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>
1 blank line only<br></blockquote><div><br></div><div>Fixed. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +#<br>
> +# Build EPICS.<br>
> +#<br>
> +epics/epics-7-1 .<br>
Stray . ?<br></blockquote><div><br></div><div>If I was not using '.' and I was simply using, the build failed, in the beginning, itself with the following error:</div><div>`Build Set: epics/epics-base</div>error: epics/epics-base:15: cannot find file: epics/<a href="http://epics-7.0-1.cf" target="_blank">epics-7.0-1.cf</a> `</div></div></blockquote><div><br></div><div>[UPDATE]: Removed the '.' The above error was due to missing space after line 15. On adding space, it was resolved.  </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><br></div><div class="gmail_quote">The command I am entering is: `/source-builder/sb-set-builder --with-rtems-bsp="xilinx_zynq_a9_qemu" --log=log_epics epics/epics-base  --trace --prefix=$HOME/development/rtems/5-arm --host=arm-rtems5`<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> \ No newline at end of file<br>
> diff --git a/source-builder/config/epics-7-1.cfg b/source-builder/config/epics-7-1.cfg<br>
> new file mode 100644<br>
> index 0000000..2a52ea5<br>
> --- /dev/null<br>
> +++ b/source-builder/config/epics-7-1.cfg<br>
> @@ -0,0 +1,57 @@<br>
> +#<br>
> +# EPICS 7.x.x Version 1.<br>
> +#<br>
> +# This configuration file configure's, make's and install's EPICS.<br>
> +#<br>
> +<br>
> +%if %{release} == %{nil}<br>
> +%define release 1<br>
> +%endif<br>
> +<br>
> +%define epics_version 7.0<br>
> +<br>
> +Name:      epics-base-%{epics_version}-%{_host}-%{release}<br>
> +Summary:   EPICS v%{epics_version} for target %{_target} on host %{_host}<br>
> +Version:   %{epics_version}<br>
> +Release:   %{release}<br>
> +URL:     <a href="https://epics.mpg.de/" rel="noreferrer" target="_blank">https://epics.mpg.de/</a><br>
This doesn't look like the right url.<br></blockquote><div><br></div><div>I cross-checked and it seems to be working. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +<br>
> +#<br>
> +# Source<br>
> +#<br>
> +%source set epics-base <a href="https://gitlab.fhi.mpg.de/junkes/epics-base/-/archive/%%7Bepics_version%7D/epics-base-%%7Bepics_version%7D.tar.gz" rel="noreferrer" target="_blank">https://gitlab.fhi.mpg.de/junkes/epics-base/-/archive/%{epics_version}/epics-base-%{epics_version}.tar.gz</a><br>
ditto<br>
<br>
where do official EPICS releases go?<br></blockquote><div><br></div><div>I think we need Heinz intervention in this and I am ready to help him get it done.</div><div>We have to use this repo for the time being.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +<br>
> +#<br>
> +# Prepare the source code.<br>
> +#<br>
> +%prep<br>
> +  build_top=$(pwd)<br>
> +<br>
> +  source_dir_epics="epics-base-%{epics_version}"<br>
> +<br>
> +  %source setup epics-base -q -n epics-base-%{epics_version}<br>
I'm curious, can you use the source_dir_epics variable here? </blockquote><div><br></div><div>I didn't see any affect not using  `source_dir_epics variable `. This time I used `cd epics-base-%{epics_version}`<br></div><div>and saw no changes as such. What do you suggest to use? </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +<br>
> +  cd ${build_top}<br>
> +<br>
> +%build<br>
> +  build_top=$(pwd)<br>
> +<br>
> +  cd ${source_dir_epics}<br>
> +<br>
> +  %{build_build_flags}<br>
> +<br>
> +  #<br>
> +  # Using macros to dynamically path to RTEMS_BASE and RTEMS_VERSION using --with-rtems-bsp<br>
> +  #<br>
> +  %{__make} RTEMS_BASE=%{_prefix} RTEMS_VERSION=%{rtems_version} CROSS_COMPILER_TARGET_ARCHS=RTEMS-%{with_rtems_bsp}<br>
<br>
Can you line break this to under 80 characters? might need to use backslashes \<br></blockquote><div><br></div><div>Fixed. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I guess EPICS is normally built from within its source tree? Just want<br>
to be sure. I haven't built it in awhile.<br></blockquote><div><br></div><div>Yes, in order to build EPICS we have to inside epics-base-7.0 directory and enter `make` there. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +<br>
> +  cd ${build_top}<br>
> +<br>
> +%install<br>
> +  build_top=$(pwd)<br>
> +<br>
> +  rm -rf $SB_BUILD_ROOT<br>
I think there is %{__rmdir} command available<br></blockquote><div><br></div><div>Thanks. Changed it.  </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<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>
<br>
also line breaks<br></blockquote><div><br></div><div>Fixed this. I will send the patch soon but I am still getting the `error: copying tree (4):` error. Will start a new thread for it.</div><div>Thanks</div><div>Mritunjay</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +  cd ${build_top}<br>
> --<br>
> 2.25.1<br>
><br>
</blockquote></div></div>
</blockquote></div></div>