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/bugzilla/show_bug.cgi?id=194612
Summary: Review Request: pstoedit Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: bugzilla-sink@leemhuis.info ReportedBy: denis@poolshark.org QAContact: fedora-package-review@redhat.com
Spec URL: http://www.poolshark.org/src/pstoedit.spec SRPM URL: http://www.poolshark.org/src/pstoedit-3.44-1.src.rpm Description: Translates PostScript and PDF graphics into other vector formats
Pstoedit converts PostScript and PDF files to various vector graphic formats. The resulting files can be edited or imported into various drawing packages. Pstoedit comes with a large set of integrated format drivers.
Misc notes :
- The main goal of having pstoedit in Extras is for the benefit of inkscape. See bug 175257.
- You'll notice the call to make doesn't use the _smp_flags macros. That's intentional, pstoedit's autoconf setup doesn't like parallel builds. Bug filed upstream.
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
denis@poolshark.org changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |175257 nThis| |
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
pertusus@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|bugzilla-sink@leemhuis.info |pertusus@free.fr OtherBugsDependingO|163776 |163778 nThis| |
------- Additional Comments From pertusus@free.fr 2006-06-14 11:14 EST ------- It would be nice for xfig too.
Not a a blocker but did you consider packaging plotutils, since it is especially advertized in the readme that it gives much more output formats.
I am seriously considering packaging ming, and then you'll be able to enable swf support but it may be approved before that.
From my reading of configure.ac, it seems that you miss
Buildrequires: gd-devel gd-devel requires libpng-devel and zlib-devel, but I think that it makes sense to keep the BR on libpng-devel.
There is a BR (and a Requires) on ghostscript missing (for gs).
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
------- Additional Comments From pertusus@free.fr 2006-06-14 11:24 EST ------- pstoedit overwrite CXXFLAGS. This should be reported upstream and in the meantime, the best solution seems to me to be a patch for configure that removes the line 22593
(line 398 in configure.ac for upstream)
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
------- Additional Comments From denis@poolshark.org 2006-06-14 19:28 EST ------- Spec URL: http://www.poolshark.org/src/pstoedit.spec SRPM URL: http://www.poolshark.org/src/pstoedit-3.44-2.src.rpm
Thanks for catching the missing Req and BRs. I filed the configure.ac bug upstream.
Yes, I was aware of the optional dependency on plotutils, but my main goal was to close bug 175257 since the ability to import postscript files into inkscape (and xfig) was requested by multiple people. I would recommend that we go on with the review of pstoedit without plotutils support for now, and build a new release later when plotutils is available (I'll work on it). If you submit a package for ming, I'll review 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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
------- Additional Comments From pertusus@free.fr 2006-06-15 03:23 EST ------- There are .so files in /usr/lib/pstoedit/. Are these dlopened 'internal' libraries?
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
------- Additional Comments From denis@poolshark.org 2006-06-15 06:47 EST ------- Right. I had initially put them in the devel package, but that doesn't work. They're not automatically linked into the pstoedit executable, but rather loaded internally by pstoedit init code (see src/dynload.cpp).
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
------- Additional Comments From pertusus@free.fr 2006-06-15 10:47 EST ------- Indeed they seem to qualify as internal dlopened libraries. It is somehow strange that upstream don't simply link them, it would be cleaner, but it isn't really problematic. That was the last issue I found, so now for the formal review:
* rpmlint is silent * follow naming guidelines * licence is GPL and included * source match upstream 13f24cb070da3f6af82ed84f4e53f049 * build on FC5 * buildrequires seem right, although I haven't built in mock * ldconfig is run * creates the directory it owns, except /usr/share/aclocal/. Not a big deal, in my opinion, as there are other packages that do that. I am not sure but it seems that there were some discussions about that, but I can't recall the result. I don't consider that a blocker but you may want to raise the issue on the extras list * things in -devel are right. There are .so in the main package, but these are dlopened libraries
APPROVED
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
------- Additional Comments From pertusus@free.fr 2006-06-15 11:21 EST ------- (In reply to comment #3)
release later when plotutils is available (I'll work on it). If you submit a package for ming, I'll review it.
I'm not in a hurry to package ming since it evolves rapidly and is a bit beta. As long as nobody ask explicitly for it I'll refrain...
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
pertusus@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| |
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
------- Additional Comments From denis@poolshark.org 2006-06-15 11:50 EST ------- Patrice, many thanks for your review.
buildrequires seem right, although I haven't built in mock
I did. Builds cleanly for FC-4, FC-5 and devel.
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: pstoedit
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194612
denis@poolshark.org 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: pstoedit
https://bugzilla.redhat.com/show_bug.cgi?id=194612
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora Version|devel |rawhide
package-review@lists.fedoraproject.org