[Bug 211626] Review Request: xtide - Calculate tide all over the world

bugzilla at redhat.com bugzilla at redhat.com
Thu Nov 23 16:22:33 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-11-23 11:22 EST -------
Now packages are updated. David, thank you for
repackaing and improving xtide related packages.

All the related srpms/rpms are under
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/

The following is a copy from the mail sent from David.

---------------------------------------------------------
I made the following changes.

xtide tarball:

- To change the user and/or group under which xttpd tries to run (the
defaults are nobody/nobody), set the variables xttpd_user and/or
xttpd_group for configure.  xttpd now exits if it cannot change to the
specified user/group.

    bash-3.1$ xttpd_user=xttpd xttpd_group=xttpd ./configure

You can also set the webmaster address for xttpd this way if you like.

    bash-3.1$ webmasteraddr="somebody at somewehere.else" ./configure

- Instead of always barfing when HOME is unset, barf only when the
need arises to write something to the home directory, and do without
any config files that would normally be there.  Setting HOME for xttpd
thus becomes optional, eliminating a common source of confusion.

tideEditor tarball:

- Get name of TCD file to edit from command line instead of HFILE_PATH.
- Check /etc/xtide.conf for WVS_PATH if environment variable is not set.
- Provided icon-64x64.png for use with desktop environments.

tcd-utils tarball:

- Automaked and cleaned up without tideEditor.

I was going to review the RPMs to see if I missed anything (it is not
trivial since under Slackware we do not use RPMs) but I was unable to
access http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/
when I tried this afternoon.  So I downloaded the xttpd rc script from
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=211626 (attached
on 2006-10-26) and put this in the XTide tarball as
scripts/Fedora/rc.xttpd.
-------------------------------------------------------------------------------
Then:

* For xttpd
  xttpd binary is still wrapped so that we change the enviroment needed
  by xttpd to arguments (which is needed for using 'daemon' function
  in /etc/rc.d/init.d/functions)

* For tideEditor
  Now tideEditor take a tcd file as a argument, not environment. For launching
  tideEditor from GNOME Menu, we have to give one argument (tcd file) to
  tideEditor. So for GNOME/KDE Menu usage only I give a wrapper script for
  tideEditor (i.e. normally there is no need for tideEditor wrapper script
  so usually I don't call the script, only for GNOME/KDE menu usage).

* Packaging:
  Now libtcd, xtide, tcd-utils, tideEditor are seperated.
  xtide, tcd-utils, tideEditor all require libtcd.

  Still I create small package 'xtide-common' which is required by
  xtide and tideEditor.
  xtide/xttpd package are unified.

Perhaps at last I have to submit a new review request for libtcd, tcd-utils,
tideEditor, however, before doing so I want to have a discussion on
this bug report, especially for packaging issue.

FILES:
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SPECS/libtcd.spec
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SPECS/tcd-utils.spec
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SPECS/tideEditor.spec
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SPECS/xtide.spec

http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SRPMS/libtcd-2.2-1.fc7.src.rpm
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SRPMS/tcd-utils-20061120-1.fc7.src.rpm
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SRPMS/tideEditor-1.3.12-1.fc7.src.rpm
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/packages/xtide/SRPMS/xtide-2.9-0.1.date20061122.fc7.src.rpm


-- 
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 package-review mailing list