[Bug 352761] Review Request: ds9 - Astronomical Data Visualization Application
bugzilla at redhat.com
bugzilla at redhat.com
Tue Dec 4 01:30:15 UTC 2007
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: ds9 - Astronomical Data Visualization Application
https://bugzilla.redhat.com/show_bug.cgi?id=352761
------- Additional Comments From wart at kobold.org 2007-12-03 20:30 EST -------
(In reply to comment #17)
> * Version specific dependency
>
> I have patched some files and now the version number of tcllib has to written in
> the specfile. It's something like this
>
> %define tcllibver 1.10
> Requires: tcllib = %{tcllibver}
>
> The ideal would be:
> 1) To have a versionless tcllib directory
> OR
> 2) To have a way of obtaining the tcllib version, similar to this macro to
> obtain the tcl version:
> %{!?tcl_version: %define tcl_version %(echo 'puts $tcl_version' | tclsh)}
> I have searched for some version information inside tcllib and I have found none.
Or option 3 (best solution):
Don't source the files directly, but instead use the 'package require' command
in Tcl to load these files. For example, instead of:
source /usr/share/tcl8.4/msgcat1.3/msgcat.tcl
Use:
package require msgcat
...or if you require a specific version of the msgcat package because you know
an earlier one won't work:
package require msgcat 1.3.4
This will perform the equivalent of the source command, but succeed regardless
of the installation directory or version of tcllib. In fact, the 'package
require' command was introduced in a much earlier version of Tcl to make things
like this possible, so that you don't have to hard code directory paths to
packages inside the application.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
More information about the Fedora-package-review
mailing list