[Bug 352761] Review Request: ds9 - Astronomical Data Visualization Application

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx  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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]