Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=352761
Summary: Review Request: ds9 - Astronomical Data Visualization Application Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: sergio.pasra@gmail.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://sergiopr.fedorapeople.org/ds9.spec SRPM URL: http://sergiopr.fedorapeople.org/ds9-4.13-8.fc7.src.rpm Description: SAOImage DS9 is an astronomical imaging and data visualization application. DS9 supports FITS images and binary tables, multiple frame buffers, region manipulation, and many scale algorithms and colormaps. It provides for easy communication with external analysis tasks and is highly configurable and extensible.
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
sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |249812
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
sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |333091
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
sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |333081
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
sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |352741
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 tibbs@math.uh.edu 2007-10-25 14:19 EST ------- A couple of quick comments:
You should use %{__ln_s} in %build since you use macroized versions of the other commands.
If you don't need to update the icon cache, it's probably best to remove the comments from your %post and %postun scriptlets and use the %post -p /sbin/ldconfig form instead.
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 sergio.pasra@gmail.com 2007-10-28 18:38 EST ------- I have modified the specfile, now the most of the tcl libraries are loaded using tcl_pkgrequire
Spec URL: http://sergiopr.fedorapeople.org/ds9.spec SRPM URL: http://sergiopr.fedorapeople.org/ds9-4.13-9.fc7.src.rpm
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
Bug 352761 depends on bug 333091, which changed state.
Bug 333091 Summary: Programs linked with tktable cannot find the libraries https://bugzilla.redhat.com/show_bug.cgi?id=333091
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NOTABUG
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
Bug 352761 depends on bug 352741, which changed state.
Bug 352741 Summary: Review Request: tkimg - More Image Formats for Tk https://bugzilla.redhat.com/show_bug.cgi?id=352741
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution| |NEXTRELEASE
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
Bug 352761 depends on bug 249812, which changed state.
Bug 249812 Summary: missing headers https://bugzilla.redhat.com/show_bug.cgi?id=249812
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEEDINFO |CLOSED Resolution| |RAWHIDE
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 sergio.pasra@gmail.com 2007-11-09 05:23 EST ------- This is a new version of the package that builds with the last version of blt
Spec URL: http://sergiopr.fedorapeople.org/ds9.spec SRPM URL: http://sergiopr.fedorapeople.org/ds9-4.13-10.fc7.src.rpm
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 mtasaka@ioa.s.u-tokyo.ac.jp 2007-11-09 11:32 EST ------- Well, I just (only just) tried to rebuild -10.fc7 on dist-f9 but it failed.
Just a note: You can try to rebuild a arbitrary srpm on koji by $ koji build --scratch <target> <srpm_you_want_to_try>
Currently <target> can be dist-f9, dist-f8-updates-candidate, or dist-fc7-updates-candidate. If rebuild is successful, the result rpms are placed under http://koji.fedoraproject.org/scratch/<your_FAS_name>/task_<id>/ .
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 mtasaka@ioa.s.u-tokyo.ac.jp 2007-11-09 11:33 EST ------- The result is http://koji.fedoraproject.org/koji/taskinfo?taskID=231751
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 sergio.pasra@gmail.com 2007-11-09 22:25 EST ------- It was a problem with one of the dependencies and I have fixed it. Now it builds
http://koji.fedoraproject.org/koji/taskinfo?taskID=232886
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
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |182235 nThis| |
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-11-12 02:41 EST ------- I fear that the license of the file ---------------------------------------------------- saotk/util/FlexLexer.h ---------------------------------------------------- is very similar with "the advertising clause" of old (4-clause) BSD, which is GPL-imcompatible.
From my eye, the only file in ds9 source tarball
of which the license is in question is this file.
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
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |mtasaka@ioa.s.u-tokyo.ac.jp Status|NEW |ASSIGNED
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 sergio.pasra@gmail.com 2007-11-12 06:24 EST ------- This file is very similar, if not identical, to the file /usr/include/FlexLexer.h from the flex package. I will look into this and I will try to use the header/code/library from the flex package instead of the source code in the tarball
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
matt@truch.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |matt@truch.net
------- Additional Comments From matt@truch.net 2007-11-14 17:06 EST ------- I'm interested in testing this package. Is there a srpm that will build for F8 or F7? I'm getting failures on both my machine and in koji.
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 sergio.pasra@gmail.com 2007-11-14 18:23 EST ------- This the new upstream version:
Spec URL: http://sergiopr.fedorapeople.org/ds9.spec SRPM URL: http://sergiopr.fedorapeople.org/ds9-5.0-1.fc7.src.rpm
I have build it in my home computer and in koji http://koji.fedoraproject.org/koji/taskinfo?taskID=241997
It still has the problem of FlexLexer.h from comment 7. I have communicated upstream the problem and they are willing to fix it.
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 sergio.pasra@gmail.com 2007-11-15 02:44 EST ------- Matthew, you probably need to install the testing version of funtools-devel and funtools-libs
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 matt@truch.net 2007-11-15 10:01 EST ------- (In reply to comment #11)
Matthew, you probably need to install the testing version of funtools-devel and funtools-libs
Great, and I have now build (scratch builds) in koji for F7 and F8, which I and a couple colleagues are already using to look at real data, so we're psyched. Thanks for the effort so far; can't wait for it to be in Fedora proper.
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 sergio.pasra@gmail.com 2007-11-15 10:29 EST ------- (In reply to comment #12) Thanks to you, I couldn't get anyone here at my institution to test the program :-/
(In reply to comment #11)
Matthew, you probably need to install the testing version of funtools-devel and funtools-libs
Great, and I have now build (scratch builds) in koji for F7 and F8, which I and a couple colleagues are already using to look at real data, so we're psyched. Thanks for the effort so far; can't wait for it to be in Fedora proper.
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
Bug 352761 depends on bug 333081, which changed state.
Bug 333081 Summary: Programs linked with blt cannot find the libraries https://bugzilla.redhat.com/show_bug.cgi?id=333081
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |ERRATA
------- Additional Comments From sergio.pasra@gmail.com 2007-11-30 10:54 EST ------- Well, although upstream is willing to fix the problem with the file FlexLexer.h, it will probably pass another month until they find the time to do it. So I have created a new package with a temporal workaround:
* I have downloaded from the flex CVS a version of FlexLexer.h with an updated license (it's release 1.22)
* I have created a tarball without the offending FlexLexer.h
In both cases I have tried to follow the guidelines in http://fedoraproject.org/wiki/Packaging/SourceURL
So these are the updated specfile and SRPM: SPEC: http://sergiopr.fedorapeople.org/ds9.spec SRPM: http://sergiopr.fedorapeople.org/ds9-5.0-2.fc7.src.rpm
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
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|devel |rawhide
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|182235 | nThis| |
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-12-01 03:31 EST ------- License check passed.
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 mtasaka@ioa.s.u-tokyo.ac.jp 2007-12-01 04:15 EST ------- Well, for 5.0-2:
* Version specific dependency ---------------------------------------------------------- [tasaka1@dhcp158 ~]$ ds9 Error in startup script: couldn't read file "/usr/share/tcl8.4/tcllib-1.9/base64/base64.tcl": no such file or directory while executing "source /usr/share/tcl8.4/tcllib-1.9/base64/base64.tcl" invoked from within "if {![catch {package present checkdns}]} { set ds9(root) "/usr/share/ds9"
source /usr/share/tcl8.4/msgcat1.3/msgcat.tcl source /usr/share..." (file "/usr/share/ds9/src/ds9.tcl" line 65) ---------------------------------------------------------- - Now rawhide tcllib is tcllib-1.10-1.fc9. * At least a patch against src/ds9.tcl seems needed * It is better that the dependency against tcllib is version specific, i.e. "Requires: tcllib = 1.9" so that like this time we can know that ds9 has to be rebuilt when tcllib is upgraded. * Or you have some better solution?
* Scriptlets http://fedoraproject.org/wiki/Packaging/ScriptletSnippets - The installed desktop files contains MimeType key and desktop database must be updated.
* The location of icon - "sun.gif" seems rather generic name and IMO it is better that this icon is moved under %_datadir/pixmaps/%name (desktop file modification is needed).
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 sergio.pasra@gmail.com 2007-12-03 07:43 EST ------- * 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.
So, I have submitted a bug to see if 1) is possible https://bugzilla.redhat.com/show_bug.cgi?id=408571 but for the moment the specfile should do the work
* Scriptlets Fixed
* The location of icon I have moved the icon to /usr/share/pixmaps/ds9 fixed
New SRPM and SPEC
SPEC: http://sergiopr.fedorapeople.org/ds9.spec SRPM: http://sergiopr.fedorapeople.org/ds9-5.0-4.fc7.src.rpm
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
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-12-03 10:11 EST ------- One note:
(In reply to comment #16)
- Version specific dependency
- Now rawhide tcllib is tcllib-1.10-1.fc9.
- On F-8 and F-7, tcllib is still 1.9. You can handle this by ------------------------------------------------------- if 0%{?fedora} >= 9 %define tcllibver 1.10 %else %define tcllibver 1.9 %endif -------------------------------------------------------- , for example.
--------------------------------------------------------- This package (ds9) is APPROVED by me ---------------------------------------------------------
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
sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From sergio.pasra@gmail.com 2007-12-03 10:26 EST ------- New Package CVS Request ======================= Package Name: ds9 Short Description: Astronomical Data Visualization Application Owners: sergiopr Branches: F-8 F-7 InitialCC: Cvsextras Commits: yes
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
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2007-12-03 14:13 EST ------- cvs done.
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@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:
- 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.
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 sergio.pasra@gmail.com 2007-12-04 04:57 EST ------- I see. Originally ds9 is shipped as a statically compiled binary, and the tcl files are included in the binary by means of a virtual zip-compressed mount point. As I'm not familiar with tcl, I thought that source is the standard way of importing.
I have followed you suggestion, thank you very match!
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
sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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
sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
------- Additional Comments From sergio.pasra@gmail.com 2008-07-16 15:42 EST ------- Package Change Request ====================== Package Name: ds9 New Branches: EL-5
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
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-07-16 20:14 EST ------- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=352761
Orion Poplawski orion@cora.nwra.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |orion@cora.nwra.com
--- Comment #25 from Orion Poplawski orion@cora.nwra.com 2011-05-04 17:10:13 EDT --- I'm not sure how this ever passed review. This package bundles a large number of libraries, including an mpeg1 encoder which is forbidden.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=352761
--- Comment #26 from Sergio Pascual sergio.pasra@gmail.com 2011-05-04 17:53:30 EDT --- This package has been always a nightmare. If it contains software that is forbidden in Fedora I think we should remove ds9. Or give it to someone else who wants to patch and unbundled libraries new libraries each new upstream release.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=352761
--- Comment #27 from Orion Poplawski orion@cora.nwra.com 2011-05-04 18:07:24 EDT --- I may be willing to take it on. I've been working today towards a patched 6.1 release. I think I relearned a lot of stuff, but hopefully it will be all set.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=352761
--- Comment #28 from Sergio Pascual sergio.pasra@gmail.com 2011-05-04 18:35:25 EDT --- In that case I'm going to orphan it. I will orphan also its dependencies
* funtools * wcstools * xpa
These are astronomical libraries, wcstools is particularly buggy
* tkimg
A Tcl/Tk module to handle images. It has problems of it's own, like bundled libraries
package-review@lists.fedoraproject.org