On Tue, Jul 25, 2023 at 10:58 PM Boris Kolpackov
<boris@xxxxxxxxxxxxxxxxx> wrote:
>
> Randy Dunlap <rdunlap@xxxxxxxxxxxxx> writes:
>
> > On 7/24/23 00:46, Boris Kolpackov wrote:
> >
> > > Tested with Qt5 5.15 and Qt6 6.4. Note that earlier versions of Qt5
> > > are no longer guaranteed to work.
This patch did not change scripts/kconfig/qconf-cfg.sh at all.
Hence, Qt5 is always linked.
Please change scripts/kconfig/qconf-cfg.sh so that:
1. If Qt6 is installed, please link Qt6.
2. If Qt6 is not installed but Qt5 is installed,
please link Qt5
> >
> > Was there such a guarantee somewhere?
>
> I don't believe there was anything explicit, but seeing that the
> qconf source code hardly changed in the past couple of years, it
> could have worked with earlier versions of Qt5 and some people
> could have relied on that.
>
>
> > Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> > Tested-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
>
> Thanks!
A request to reviewers/testers:
Please make sure you tested this
with both Qt5 and Qt6.
To check the Qt version you are running:
1. 'make xconfig'
2. See Help -> About
--
Best Regards
Masahiro Yamada
