Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 690584 - sci-geosciences/mapnik-3.0.23 version bump
Summary: sci-geosciences/mapnik-3.0.23 version bump
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Deadline: 2020-10-26
Assignee: Sci-geo Project
URL:
Whiteboard:
Keywords: PMASKED
Depends on:
Blocks:
 
Reported: 2019-07-24 14:55 UTC by Thomas Beutin
Modified: 2020-12-12 04:23 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
ebuild (mapnik-3.0.22.ebuild,2.63 KB, text/plain)
2019-07-24 14:55 UTC, Thomas Beutin
Details
patch (mapnik-3.0.22-scons.patch,1.47 KB, text/plain)
2019-07-24 14:56 UTC, Thomas Beutin
Details
ebuild patch converting 3.0.18 to 3.0.23 (mapnik-3.0.18_to_3.0.23.ebuild.patch,2.88 KB, patch)
2020-10-02 18:33 UTC, Chris Mayo
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Beutin 2019-07-24 14:55:44 UTC
Created attachment 584296 [details]
ebuild

According to
https://mapnik.org/news/release-3.0.22 there is an updated version of mapnik-v3 available. The working ebuild is attached, the patch referenced within follows. This also fixes bug 689892.
Comment 1 Thomas Beutin 2019-07-24 14:56:12 UTC
Created attachment 584298 [details]
patch
Comment 2 Jonas Stein gentoo-dev 2019-08-02 12:16:57 UTC
Thank you for your contribution. I had a short look on the ebuild. 
Here a few ideas:

* Please test the ebuild with repoman full -d -x https://wiki.gentoo.org/wiki/Repoman
* We always try to use the latest EAPI, please bump to EAPI=7
* Please remove obsolete lines
* why is inherit eutils required here?
* A bot can look for additional mistakes, if you upload a PR via github
Comment 3 Thomas Beutin 2019-08-05 12:32:38 UTC
(In reply to Jonas Stein from comment #2)
> Thank you for your contribution. I had a short look on the ebuild. 
> Here a few ideas:
> 
> * Please test the ebuild with repoman full -d -x
> https://wiki.gentoo.org/wiki/Repoman
> * We always try to use the latest EAPI, please bump to EAPI=7
> * Please remove obsolete lines
> * why is inherit eutils required here?
> * A bot can look for additional mistakes, if you upload a PR via github

Hi Jonas, thanks for your comments. Actually, i'm not in the ebuild writing business (yet), i'm just a "user" w/ some programming skills. So what i basically did was copying the old ebuild from the tree and adapting it to the new mapnik version. Additional effort from my side was put into the check of the scons build tool configuration to get rid of this not-using-libxml2 issue.

If i can find some time to dig into the ebuild EAPI version bumping, i'll do, but any help how to do is more than very welcome.
Comment 4 Thomas Beutin 2019-08-05 13:27:48 UTC
btw: i opened an upstream bug for the SConstruct file addressed in my patch: https://github.com/mapnik/mapnik/issues/4075
Comment 5 Vincent de Phily 2020-10-02 13:23:00 UTC
Note that 3.0.23 release includes the fix for #689892 and builds with python 3, so the bump is needed to avoid removal from the tree in 24 days.
Comment 6 Chris Mayo 2020-10-02 18:33:58 UTC
Created attachment 663559 [details, diff]
ebuild patch converting 3.0.18 to 3.0.23

ebuild upgrade that removes inherit scons-utils by using mapnik's own scons.py.
Comment 7 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-10-26 07:41:34 UTC
Package removed.
Comment 8 redneb 2020-12-12 04:23:36 UTC
In case anyone's interested, I created an overlay [1] for mapnik 3.0.23. It is based off of Chris Mayo's ebuild (with some minor tweaks) and it also contains a patch (backported from mapnik's master branch) to make mapnik 3.0.23 work with boost 1.74.

[1] https://github.com/redneb/gentoo-mapnik-overlay