On Sun, May 20, 2018 at 10:40:18AM +0200, Rafael Sadowski wrote:
> espie@ and some others have let me know that the bulk is not stable
> because the bulk needs both qwt flavors.

Bah, i plead guilty for that one.

> With many ideas and help from espie@ you will find below a first attempt.
> All comsumers need a little tweak to find the correct qwt:
> 
> - geo/qgis
> - security/ophcrack
> - x11/kde4/plasma-addons
> - x11/kde4/smokeqt
> 
> If you like it could you test that in an bulk?

I have to admit i dont like patching qgis, which already has several
ways of finding qwt, and i'd have a hard time upstreaming another one.
By default, it's looking first for a lib named qwt-qt5.. cf
https://github.com/qgis/QGIS/blob/release-3_0/cmake/FindQwt.cmake#L15

So how about going the debian way, which puts the headers in the common
'subpackage' (cf https://packages.debian.org/buster/libqwt-headers) as
they are the same for both variants, and just renames the library:
- the qt4 version is libqwt
- the qt5 version is libqwt-qt5

this way, the ecosystem is already 'ready' for this layout, at least for
qgis... which is the only user so far or the qt5 version. And that
triggers less churn in the other qt4 consumers, since the libname/path
doesnt change.

Opinions ?

> -QTVER =      qt5
> +QTVER =              qt5


> -QTVER =      qt4
> +QTVER =              qt4

Unnecessary churn :)

Thx for working on this !

Landry

Reply via email to