[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#920878: provide a package name for lrelease-pro



Hi Dmitry,

On Sat, Feb 29, 2020 at 11:48:47PM +0300, Dmitry Shachnev wrote:
> As we discussed on IRC, a separate package won't help much, as it cannot
> declare a dependency on qt5-qmake:native anyway. The :native qualifier only
> makes sense in Build-Depends.

Details. It's technically possible, but we don't want it here:

> Also, I figured out that lrelease (lrelease-pro in new Qt) does not
> actually need *native* qmake, the cross qmake works as well. We just need
> to specify QMAKE environment variable that it honors:
> 
> https://code.qt.io/cgit/qt/qttools.git/tree/src/linguist/lprodump/main.cpp?h=v5.14.1#n439
> 
> As I just demonstrated in https://bugs.debian.org/889752#10, this approach
> works. So I propose to close this bug and export QMAKE to qmake cross
> wrapper in the affected packages instead.

The approach is much better. Thank you.

I'm left wondering whether we should ask Guillem to include QMAKE in
/usr/share/dpkg/buildtools.mk. Once doing so, exporting QMAKE can be as
simple as:

DPKG_EXPORT_BUILDTOOLS=1
include /usr/share/dpkg/buildtools.mk

or

include /usr/share/dpkg/buildtools.mk
export QMAKE

If you agree, I propose repurposing this bug report for asking Guillem
to extend buildtools.mk.

Helmut


Reply to: