Bug 242228 - Review Request: ps2eps - PS-to-EPS converter
Review Request: ps2eps - PS-to-EPS converter
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Patrice Dumas
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-06-02 07:43 EDT by Terje Røsten
Modified: 2011-05-05 11:28 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-06-06 11:14:24 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
pertusus: fedora‑review+
tibbs: fedora‑cvs+


Attachments (Terms of Use)
fix insecure tmp use + add shebang (1.44 KB, patch)
2007-06-02 12:54 EDT, Patrice Dumas
no flags Details | Diff

  None (edit)
Description Terje Røsten 2007-06-02 07:43:24 EDT
Spec URL: http://web.phys.ntnu.no/~terjeros/ps2eps/ps2eps.spec
SRPM URL: http://web.phys.ntnu.no/~terjeros/ps2eps/ps2eps-1.64-1.fc7.src.rpm
Description:
ps2eps is a tool to produce Encapsulated PostScript Files (EPS/EPSF)
from usual one-paged Postscript documents. It calculates correct
Bounding Boxes for those EPS files and filters some special postscript
command sequences that can produce errorneous results on printers.
Comment 1 Patrice Dumas 2007-06-02 12:51:38 EDT
I suggest adding -p to the man pages install command to keep
timestamps.

Also I suggest using %defattr (-, root, root, -) instead of
%defattr (-, root, root).

INSTALL.txt shouldn't be shipped.

bbox is a bit too generic for a program name that is mainly
used from another application. I think it could be discussed with
upstream.
Comment 2 Patrice Dumas 2007-06-02 12:54:49 EDT
Created attachment 155988 [details]
fix insecure tmp use + add shebang

This patch hopefully fixes an insecure use of tmp bug. I
also patched the shebang at the same time. I think that the
issue should really be brought up upstream.
Comment 3 Terje Røsten 2007-06-02 13:05:04 EDT
> This patch hopefully fixes an insecure use of tmp bug. I
> also patched the shebang at the same time. I think that the
> issue should really be brought up upstream.

Thanks, will send upstream a mail, here is new spec and srpms:

Spec URL: http://web.phys.ntnu.no/~terjeros/ps2eps/ps2eps.spec
SRPM URL: http://web.phys.ntnu.no/~terjeros/ps2eps/ps2eps-1.64-2.fc7.src.rpm
Comment 4 Patrice Dumas 2007-06-02 13:16:25 EDT
* rpmlint is silent
* free software, license included
* follow packaging guidelines
* spec is legible
* sane provides
* source match upstream
89aec3f075cee662f2367b12337b5c3e  ps2eps-1.64.tar.gz
* %files section right

APPROVED

- please bring the issue of the bbox name to upstream
- I personally prefer adding explicitly names in %files to
  know when things change. This is only a suggestion/personal
  preference.
Comment 5 Terje Røsten 2007-06-05 16:39:24 EDT
> APPROVED

Thanks!
 
> - please bring the issue of the bbox name to upstream

Ok.

> - I personally prefer adding explicitly names in %files to
>   know when things change. 

Yeah, a nice advise, however not critical is this case as files is installed by
hand and
not by e.g. configure && make && make install


New Package CVS Request
=======================
Package Name: ps2eps
Short Description: PS-to-EPS converter
Owners: terjeros@phys.ntnu.no
Branches: FC-5 FC-6 F-7
InitialCC:


Comment 6 Patrice Dumas 2007-06-05 16:52:53 EDT
(In reply to comment #5)

> Yeah, a nice advise, however not critical is this case as files is installed by
> hand and
> not by e.g. configure && make && make install

Indeed! You are so right.
Comment 7 Tom "spot" Callaway 2007-06-05 17:09:21 EDT
cvs done
Comment 8 Patrice Dumas 2008-11-20 15:45:10 EST
Package Change Request
======================
Package Name: ps2eps
New Branches: EL-5
Owners: pertusus


Terje agreed by mail that I could maintain the EPEL branch. Terje, I think that you should confirm here. Also tell if you want to be comaintaner of EPEL branches.
Comment 9 Terje Røsten 2008-11-20 16:42:32 EST
I can confirm that, thanks Pat!
Comment 10 Dennis Gilmore 2008-11-25 11:56:14 EST
CVS Done
Comment 11 Orion Poplawski 2011-05-03 13:13:22 EDT
Package Change Request
======================
Package Name: ps2eps
New Branches: el6
Owners: orion
Comment 12 Terje Røsten 2011-05-03 13:28:13 EDT
In private communication Orion and I agreed that Orion will maintain EPEL 6 branches of ps2eps, thanks!
Comment 13 Jason Tibbitts 2011-05-05 11:28:43 EDT
Git done (by process-git-requests).

Note You need to log in before you can comment on or make changes to this bug.