[Bug 182463] Review Request: cairomm (C++ bindings for cairo)

bugzilla at redhat.com bugzilla at redhat.com
Sun Apr 9 18:06:03 UTC 2006

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: cairomm (C++ bindings for cairo)


------- Additional Comments From rvinyard at cs.nmsu.edu  2006-04-09 14:06 EST -------
Spec Name or Url: http://miskatonic.cs.nmsu.edu/pub/cairomm.spec

SRPM Name or Url: http://miskatonic.cs.nmsu.edu/pub/fedora/5/srpms/cairomm-0.6.

Summary of changes:
- New upstream release
- Added docs back in by moving them in install and including in devel files

Other changes from feedback:
* missing "BuildRequires: pkgconfig"

* cairomm-devel should really "Requires: pkgconfig", because (1) it
contains a pkg-config file and (2) the library headers are stored in
a path that is unlikely to be found if pkg-config is not used
 - ADDED to cairomm BuildRequires and cairomm-devel Requires

* prefer "make DESTDIR=%{buildroot} install" over %makeinstall, since
while the former is de facto standard, the latter is just a hack

* AUTHORS %doc file is about libxmlplusplus and points to its home page
 - UPSTREAM RELEASE fixes this

* README %doc file has libxml++ at the top, very confusing, both files
should really be corrected
 - UPSTREAM RELEASE fixes this

* Also note that the cairomm-devel package contains more than "headers
and static library". ;)  The very important *.so symlink is included, too.

* As a hint at the bottom:
 - CHANGED to the latter suggestion... directory style

Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

More information about the fedora-extras-list mailing list