[Bug 234581] Review Request: Mapnik cartography library - first package, need sponsor.

bugzilla at redhat.com bugzilla at redhat.com
Tue Jul 31 16:12:23 UTC 2007


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: Mapnik cartography library - first package, need sponsor.


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234581





------- Additional Comments From kms at passback.co.uk  2007-07-31 12:12 EST -------
> Some files have Windows-style end-of-file encodings. Please fix them
> to Unix-style encodings

Files removed in upstream SVN.

> Summary should not end with dot.

Fixed.

> QUESTION How is libmapnik.so used?
>   - Is this library used only by this package?
>   - Or can this library be linked from other software?

Still waiting on upstream.

> - In %changelog, please use %%_prefix. Otherwise %_prefix is expanded
>   in %changelog.

Fixed.

> * compilation flags
>   - Fedora specific compilation flags are not honored.
>   - -O3 flag is generally forbidden (this make debugging difficult)

Fixed in SPEC file / patches.  Upstream did not want to change.

> * Fonts
>   - Fedora has dejavu-fonts, dejavu-fonts-experimental and
>     shipping duplicate dejavu fonts in this package is not
>     allowed.

Fixed in SPEC file / patches.  Still needed in upstream to build on other
platforms (eg Windows).

> * sitelib vs sitearch
>   - This package installs 
>     /usr/lib/python2.5/site-packages/mapnik/_mapnik.so ,
>     which is arch dependent.
>     python related files must be installed under %python_sitearch.
>     not under %python_sitelib.

Fixed.

I am still waiting on upstream to fix the soversion and then to make a release.
 I have not published an updated version of my SPEC file as I have switched to
building from SVN prior to the upstream release.

-- 
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 package-review mailing list