Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 588994 - sci-geosciences/qgis-2.12.2 and dev-python/sip-4.18: error: request for member ‘insert’ in ‘qm’, which is of pointer type ‘QMap<long long int, QMap<int, QVariant> >*’
Summary: sci-geosciences/qgis-2.12.2 and dev-python/sip-4.18: error: request for membe...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Sci-geo Project
URL: https://github.com/qgis/QGIS/commit/f...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-17 14:49 UTC by Martin von Gagern
Modified: 2016-08-17 11:00 UTC (History)
2 users (show)

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


Attachments
commit fc6559aa (fc6559aa053317cda8ced657c3013a0d4c6549e9.patch,2.24 KB, patch)
2016-07-17 14:49 UTC, Martin von Gagern
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin von Gagern 2016-07-17 14:49:47 UTC
Created attachment 440940 [details, diff]
commit fc6559aa

After upgrading grass, I failed to re-merge my qgis-2.12.2. The most likely difference to the previous successful run is that now I have sip-4.18 installed, where I had 4.17-r1 before.

/var/tmp/portage/sci-geosciences/qgis-2.12.2/work/qgis-2.12.2/python/core/conversions.sip: In function ‘int convertTo_QMap_3800_0600QMap_1800_0100QVariant(PyObject*, void**, int*, PyObject*)’:
/var/tmp/portage/sci-geosciences/qgis-2.12.2/work/qgis-2.12.2/python/core/conver
sions.sip:735:8: error: request for member ‘insert’ in ‘qm’, which is of pointer
 type ‘QMap<long long int, QMap<int, QVariant> >*’ (maybe you meant to use ‘->’ 
?)
     qm.insert(k, *t);
        ^

https://github.com/qgis/QGIS/commit/fc6559aa053317cda8ced657c3013a0d4c6549e9 deals with this issue upstream. According to its commit message, the code in question never actually worked. The preprocessor switches were meant for sip 4.12 but due to a hex conversion error targeted 4.18 by accident. So until this is fixed in the ebuild, I'd suggest the following solution:

mkdir -p /etc/portage/patches/sci-geosciences/qgis-2.12.2
cd /etc/portage/patches/sci-geosciences/qgis-2.12.2
wget https://github.com/qgis/QGIS/commit/fc6559aa053317cda8ced657c3013a0d.patch
emerge -1 =sci-geosciences/qgis-2.12.2
Comment 1 Andreas Plesch 2016-07-18 14:12:13 UTC
Thanks for the detective work. I can confirm that suggested solution, eg. using the patch works for my setup as well:

https://bugs.gentoo.org/show_bug.cgi?id=583784#c3
Comment 2 Ettore Di Giacinto (RETIRED) gentoo-dev 2016-08-15 09:58:36 UTC
I can confirm. Patch works also for Sabayon: https://github.com/Sabayon/for-gentoo/tree/master/sci-geosciences/qgis
Comment 3 Thomas Deutschmann (RETIRED) gentoo-dev 2016-08-17 11:00:37 UTC
Thanks for the report. A version which contains the fix is now in tree:

> commit e245985388006bcd402c21b41985b34f835f63c7
> Author: Thomas Deutschmann
> Date:   Wed Aug 17 12:53:39 2016 +0200
> 
>     sci-geosciences/qgis: Bump to v2.14.5
> 
>      - Version bump to new upstream release v2.14.5 (Fixes Gentoo-Bug #581146,
>        #583784, #588994).
> 
>      - Bumped to EAPI=6; This makes multilib eclass unnecessary because
>        get_libdir is now part of EAPI=6.
> 
>      - "oracle" USE flag added (Fixes Gentoo-Bug #585156).
> 
>      - Disabling test suite because upstream doesn't allow running from
>        install path [Link 1] (Closes Gentoo-Bug #499894).
> 
>      - Package license updated after upstream clearified its own license
>        [Link 2].
> 
>     Link 1: https://github.com/qgis/QGIS/blob/final-2_14_5/src/core/qgsapplication.cpp#L374
>     Link 2: https://github.com/qgis/QGIS/commit/d5bc39cb8ccc047543d37359ae7fc2d34c89a538
> 
>     Gentoo-Bug: https://bugs.gentoo.org/581146
>     Gentoo-Bug: https://bugs.gentoo.org/585156
>     Gentoo-Bug: https://bugs.gentoo.org/588994
>     Gentoo-Bug: https://bugs.gentoo.org/583784
>     Gentoo-Bug: https://bugs.gentoo.org/499894
> 
>     Acked-by: Amy Winston
> 
>     Package-Manager: portage-2.3.0
>