GNU bug report logs - #72835
[PATCH qt-team 00/38] update qt6 to 6.7.2

Previous Next

Full log

Message #377 received at 72835 <at> debbugs.gnu.org (full text, mbox):

From: Z572 <zhengjunjie <at> iscas.ac.cn>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 72835 <at> debbugs.gnu.org, 宋文武 <iyzsong <at> envs.net>
Subject: Re: [bug#72835] [PATCH qt-team 32/38] gnu: Add qtconnectivity.
Date: Wed, 18 Sep 2024 10:10:27 +0800

[Message part 1 (text/plain, inline)]

Maxim Cournoyer <maxim.cournoyer <at> gmail.com> writes:

> Hi Zheng,
>
> Zheng Junjie <zhengjunjie <at> iscas.ac.cn> writes:
>
>> * gnu/packages/qt.scm (qtconnectivity): New variable.
>>
>> Change-Id: Ia2c1d9d260b47cd3a18c9ab5ac7e8f5ea83b7e11
>> ---
>>  gnu/packages/qt.scm | 23 +++++++++++++++++++++++
>>  1 file changed, 23 insertions(+)
>>
>> diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
>> index 1956fec57a..df8759b832 100644
>> --- a/gnu/packages/qt.scm
>> +++ b/gnu/packages/qt.scm
>> @@ -1688,6 +1688,29 @@ (define-public qtconnectivity-5
>>      (description "The Qt Connectivity modules provides modules for interacting
>>  with Bluetooth and NFC.")))
>>  
>> +(define-public qtconnectivity
>> +  (package
>> +    (inherit qtsvg)
>> +    (name "qtconnectivity")
>> +    (version "6.6.3")
>> +    (source (origin
>> +              (method url-fetch)
>> +              (uri (qt-url name version))
>> +              (sha256
>> +               (base32
>> +                "066mf4d6a81ywviwr8bvm1mpm2ykjzysvcc0v2x82h5bl28vl6h9"))))
>> +    (native-inputs
>> +     (list perl pkg-config qtdeclarative))
>> +    (inputs
>> +     (list bluez qtbase))
>> +    (arguments (substitute-keyword-arguments (package-arguments qtsvg)
>> +                 ((#:phases phases)
>> +                  #~(modify-phases #$phases
>> +                      (delete 'delete-installed-tests)))))
>> +    (synopsis "Qt Connectivity module")
>> +    (description "The Qt Connectivity modules provides modules for interacting
>> +with Bluetooth and NFC.")))
>
> I'm not sure about using inheritance this way; the packages are
> unrelated. You'd at least want to ensure propagated-inputs is forced to
> '() in case qtsvg had some added in the future.
>
> I think I had refrained from using this style for Qt 6 thus far, as I
> found it obfuscated the package definitions for Qt 5.
>
> Could you rewrite it to use the style as used for example by
> qtshadertools?  Only the home page and license fields need to be
> specified.  So it's one field difference, and better clarity.
>
> Could you please send a v2?

done.

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.