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@xxxxxxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxx" ./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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review