qt-build-system: Add QT_QPA_PLATFORM=offscreen?

qt-build-system: Add QT_QPA_PLATFORM=offscreen?

[Hartmut Goebel]

[-- Attachment #1.1: Type: text/plain, Size: 1141 bytes --]

Hi Danny,

(forwarding to guix-devel to eventually get more opinions on this)

Am 21.01.20 um 23:43 schrieb Danny Milosavljevic:
>> +         (add-before 'check 'check-setup
>> +           (lambda _
>> +             (setenv "QT_QPA_PLATFORM" "offscreen")
>> +             #t)))))
> Hmm, would it make sense to do that automatically in qt-build-system ?

I would be fine with this, since I find myself adding this snippet quite
often when building Qt/KDE applications. But there *might* be cases,
where tests fail due to this setting. (While I'm not aware of one, I
could imaging some test checks whether it is actually painting.)

Thus I propose implementing this in some way to easily disable it if
required: Either

- by adding this a new phase (as in the snippet shown above), so it can
easily be disabled

- by adding another keyword-argument (proposal: `#:qpa-platform`,
default: "offscreen", if +f, will not be set.

WDYT?

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [bug#39229] qt-build-system: Add QT_QPA_PLATFORM=offscreen?

[Mike Rosset]

Hartmut Goebel writes:

> Hi Danny,
>
> (forwarding to guix-devel to eventually get more opinions on this)
>
> Am 21.01.20 um 23:43 schrieb Danny Milosavljevic:
>>> +         (add-before 'check 'check-setup
>>> +           (lambda _
>>> +             (setenv "QT_QPA_PLATFORM" "offscreen")
>>> +             #t)))))
>> Hmm, would it make sense to do that automatically in qt-build-system ?
>
> I would be fine with this, since I find myself adding this snippet quite
> often when building Qt/KDE applications. But there *might* be cases,
> where tests fail due to this setting. (While I'm not aware of one, I
> could imaging some test checks whether it is actually painting.)
>
> Thus I propose implementing this in some way to easily disable it if
> required: Either
>
> - by adding this a new phase (as in the snippet shown above), so it can
> easily be disabled
>
> - by adding another keyword-argument (proposal: `#:qpa-platform`,
> default: "offscreen", if +f, will not be set.
>
> WDYT?


I think adding this to qt-build-system with the keywords is a good
suggestion.

++

Mike

Re: qt-build-system: Add QT_QPA_PLATFORM=offscreen?

[Efraim Flashner]

[-- Attachment #1: Type: text/plain, Size: 1790 bytes --]

On Wed, Jan 22, 2020 at 09:44:02AM +0100, Hartmut Goebel wrote:
> Hi Danny,
> 
> (forwarding to guix-devel to eventually get more opinions on this)
> 
> Am 21.01.20 um 23:43 schrieb Danny Milosavljevic:
> >> +         (add-before 'check 'check-setup
> >> +           (lambda _
> >> +             (setenv "QT_QPA_PLATFORM" "offscreen")
> >> +             #t)))))
> > Hmm, would it make sense to do that automatically in qt-build-system ?
> 

I was about to suggest this.
(ins)efraim@E5400 ~/workspace/guix$ grep QT_QPA_PLATFORM\" gnu/packages/kde*scm | wc -l
41
(ins)efraim@E5400 ~/workspace/guix$ grep define-public gnu/packages/kde*scm | wc -l
124
(ins)efraim@E5400 ~/workspace/guix$ grep qt-build-system gnu/packages/kde*scm | wc -l
27


> I would be fine with this, since I find myself adding this snippet quite
> often when building Qt/KDE applications. But there *might* be cases,
> where tests fail due to this setting. (While I'm not aware of one, I
> could imaging some test checks whether it is actually painting.)
> 
> Thus I propose implementing this in some way to easily disable it if
> required: Either
> 
> - by adding this a new phase (as in the snippet shown above), so it can
> easily be disabled
> 
> - by adding another keyword-argument (proposal: `#:qpa-platform`,
> default: "offscreen", if +f, will not be set.
> 
> WDYT?
> 

I like a new phase before the 'check phase. Are there other things we'd
want to add? Looking through kde-framworks.scm the only one I'd feel
comfortable adding would be the TZ ones.



-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]