[Buildroot] [PATCH v4 6/6] qt5: Remove *.la/*.prl fixup script

[Arnout Vandecappelle]

On 27/03/2019 23:13, Andreas Naumann wrote:
> Hi,
> 
> I'd really be interested if somebody remembers why this was needed. *.la
> files are fixed in another generic step anyway, so why do it twice?

 That's easy: at the time this was added (commit
1737b5648c9f778149eb7da5897b66a9577d22c4 from 2013) the fixup was only done for
autotools packages, not for all packages. That changed only with commit
67f8138a46388f28d00dc8711b64d04ac7ebb464 in 2015.

> Also, I have not found absolute /usr/lib pathes in any of the prl files
> without this.

 This I can't explain. I've added Thomas in Cc but I doubt that he remembers.

 But isn't this exactly what gets fixed by the previous patch?

 BTW, this patch should also remove the definition of the macro.

 Regards,
 Arnout


> 
> regards,
> Andreas
> 
> 
> Am 22.03.19 um 19:39 schrieb Andreas Naumann:
>> After fixing Prefix in our custom qt.conf, Qt computes and embeds the
>> correct sysroot prefixed pathes in all *.la and *.prl files.
>>
>> Signed-off-by: Andreas Naumann <anaumann at ultratronik.de> ---
>> package/pkg-qmake.mk | 1 - 1 file changed, 1 deletion(-)
>>
>> diff --git a/package/pkg-qmake.mk b/package/pkg-qmake.mk index
>> 11cfb809fc..9293253fae 100644 --- a/package/pkg-qmake.mk +++
>> b/package/pkg-qmake.mk @@ -77,7 +77,6 @@ define $(2)_INSTALL_STAGING_CMDS
>> $$(TARGET_MAKE_ENV) \ $$($(2)_MAKE_ENV) \ $$(MAKE) -C $$($$(PKG)_SRCDIR)
>> $$($(2)_INSTALL_STAGING_OPTS) - $$(QT5_LA_PRL_FILES_FIXUP) endef endif
>>
>>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot