Bug 430848 - libspectre - A library for rendering PostScript(TM) documents
libspectre - A library for rendering PostScript(TM) documents
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
low Severity low
: ---
: ---
Assigned To: Parag AN(पराग)
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-01-30 00:08 EST by Matthias Clasen
Modified: 2008-01-30 13:46 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-01-30 13:46:23 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
panemade: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Matthias Clasen 2008-01-30 00:08:25 EST
%{name} is a small library for rendering PostScript(TM) documents.
It provides a convenient easy to use API for handling and rendering
PostScript documents.

Spec: http://people.redhat.com/mclasen/libspectre.spec
SRPM: http://people.redhat.com/mclasen/libspectre-0.2.0-1.fc9.src.rpm
Comment 1 Parag AN(पराग) 2008-01-30 00:22:35 EST
error: Installed (but unpackaged) file(s) found:
/usr/lib/pkgconfig/libspectre.pc
Comment 2 Bill Nottingham 2008-01-30 00:27:53 EST
Assuming this used to be in evince?
Comment 3 Matthias Clasen 2008-01-30 00:44:04 EST
Ah, sorry. Forgot to rebuild the srpm with the latest spec file. 

Bill: no, I don't think this is code that was in evince per se, but rather code
that was written by the evince guys, and optionally can be used by evince. Using
it improves the ps support in evince, from what I read - e.g. if you use
libspectre, the dvi backend will support ps specials.
Comment 4 Parag AN(पराग) 2008-01-30 00:45:43 EST
Can't see any docs installed.
Good to provide tests directory as %doc for -devel package.
Comment 5 Matthias Clasen 2008-01-30 00:48:06 EST
Better srpm & spec uploaded, same location.
Comment 6 Parag AN(पराग) 2008-01-30 00:59:50 EST
Review:
+ package builds in mock (rawhide i386).
koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=383374
+ rpmlint output on SRPM and for RPM.
libspectre.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 24)
==> Not a problem as its cosmetic message and can be fixed using sed.

+ source files match upstream url
6dc4382d8c9cd355a76c4516f6e6f6e3  libspectre-0.2.0.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ BuildRequires are proper.
+ Compiler flags used correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ libspectre.pc file present.
+ -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ ldconfig scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Package libspectre-0.2.0-1.fc9 =>
  Provides: libspectre.so.1
  Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1)
libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.3) libc.so.6(GLIBC_2.3.4)
libc.so.6(GLIBC_2.4) libc.so.6(GLIBC_2.7) libgs.so.8 libspectre.so.1 rtld(GNU_HASH)
+ Package libspectre-devel-0.2.0-1.fc9 =>
  Requires: libspectre = 0.2.0-1.fc8 libspectre.so.1 pkgconfig

+ Not a GUI App

APPROVED.
Comment 7 Matthias Clasen 2008-01-30 01:18:03 EST
New Package CVS Request
=======================
Package Name: libspectre
Short Description: A library for rendering PostScript(TM) documents
Owners: mclasen@redhat.com
Branches: 
InitialCC: 
Cvsextras Commits:
Comment 8 Kevin Fenzi 2008-01-30 12:27:45 EST
cvs done.
Comment 9 Matthias Clasen 2008-01-30 13:46:23 EST
package built

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