[Bug 211626] Review Request: xtide - Calculate tide all over the world
bugzilla at redhat.com
bugzilla at redhat.com
Sun Oct 22 02:51:17 UTC 2006
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: xtide - Calculate tide all over the world
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=211626
------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp 2006-10-21 22:51 EST -------
Thank you for checking.
Well..
(In reply to comment #20)
> * I also propose to standardize the libtcd building such that
> it can be done with
>
> export CFLAGS="$RPM_OPT_FLAGS -DCOMPAT114"
> %configure
> %{__make} %{?_smp_mflags} \
> %if %{build_libtcd}
> libtcd.so \
> SOMAJOR=`echo %{libtcd_soname} | sed -e 's|^\(.*\)\.\(.*\)\.\(.*\)$|\1|'` \
> SOMINOR=`echo %{libtcd_soname} | sed -e 's|^\(.*\)\.\(.*\)\.\(.*\)$|\2|'` \
> SOVER=` echo %{libtcd_soname} | sed -e 's|^\(.*\)\.\(.*\)\.\(.*\)$|\3|'`
> %endif
>
> I attach the updated libtcd-2.3.1-shared.patch.
Okay, I will recheck CFLAGS issue again.
>
>
> * Now there is the issue of missing dependency of tideEditor and xttpd which
> requires /etc/xtide.conf and /usr/share/xtide/harmonics-2004-06-14.tcd.
> To workaround this issue I propose to add a package which will
> contain those common datas, called, for example tcd-utils-data and
> which will contain .......
I think that "xtide-data" is more preferable and I will introduce
this.
> * There is a strange $f on the following line
> %{__install} -c -p -m755 libtcd.so.*.*.* $f $RPM_BUILD_ROOT%{_libdir}
!? Oops.... I will correct this.
> * In my opinion /etc/xttpd.conf should be instead /etc/sysconfig/xttpd,
> with the corresponding adjustment of xttpd.init
Okay.
(In reply to comment #21)
> The patch is for tideEditor-wrapper.sh
Oh, good catch. I will apply.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the Fedora-package-review
mailing list