Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 634228 - dev-build/cmake: can virtx be removed from tests?
Summary: dev-build/cmake: can virtx be removed from tests?
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-14 10:09 UTC by Rolf Eike Beer
Modified: 2024-07-03 01:47 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rolf Eike Beer archtester 2017-10-14 10:09:50 UTC
I don't see any reason for the virtx in CMake tests. I have machines that run the nightly test suite every day, which don't have any X server (a T5120 doesn't even have a monitor plug). When I remove the virtx I can run the testsuite through ebuild also without problems. I don't think there really is any test that requires X, and if it does the virtx usage should probably depend on the "qt5" use flag.

Reproducible: Always
Comment 1 Michael Palimaka (kensington) gentoo-dev 2017-10-19 10:36:47 UTC
Thanks!
Comment 2 Larry the Git Cow gentoo-dev 2017-10-19 10:36:58 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7b5d2bedf7dee3fd132ea3346bca0cc519d5c368

commit 7b5d2bedf7dee3fd132ea3346bca0cc519d5c368
Author:     Michael Palimaka <kensington@gentoo.org>
AuthorDate: 2017-10-19 10:36:33 +0000
Commit:     Michael Palimaka <kensington@gentoo.org>
CommitDate: 2017-10-19 10:36:47 +0000

    dev-util/cmake: virtualx is not required for tests
    
    Closes: https://bugs.gentoo.org/634228
    Package-Manager: Portage-2.3.8, Repoman-2.3.3

 dev-util/cmake/cmake-3.9.4.ebuild | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)
Comment 3 Andreas Sturmlechner gentoo-dev 2017-10-19 15:46:32 UTC
commit 4c5a71f593ee14dee9af88eac235615a7c3d2ad3
Author: Lars Wendler <polynomial-c@gentoo.org>
Date:   Thu Oct 19 15:36:17 2017 +0200

    Revert "dev-util/cmake: virtualx is not required for tests"
    
    This reverts commit 7b5d2bedf7dee3fd132ea3346bca0cc519d5c368.
    virtualx is still required for at least 11 tests.
Comment 4 Larry the Git Cow gentoo-dev 2024-06-24 20:15:43 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=400074954458cc15116212bf2d208dd62f22cb7b

commit 400074954458cc15116212bf2d208dd62f22cb7b
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2024-06-24 20:14:20 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2024-06-24 20:15:31 +0000

    dev-build/cmake: Drop virtualx, use QT_QPA_PLATFORM=offscreen instead
    
    Bug: https://bugs.gentoo.org/634228
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 dev-build/cmake/cmake-9999.ebuild | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
Comment 5 Larry the Git Cow gentoo-dev 2024-07-03 01:47:16 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=91ec0f63156ec7a8fd98fc95eed8ddbcecf0f484

commit 91ec0f63156ec7a8fd98fc95eed8ddbcecf0f484
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2024-07-03 01:27:46 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2024-07-03 01:28:41 +0000

    dev-build/cmake: add 3.29.6
    
    Also, filter a bunch of variables which confuse the testsuite when
    it tries to compare output against a regex.
    
    Note that already did 9b0fbdcf296c0bf013b58637f2a87a5bdc125e52 in preparation
    for FetchContent changes coming down the line... :(
    
    Closes: https://bugs.gentoo.org/634228
    Signed-off-by: Sam James <sam@gentoo.org>

 dev-build/cmake/Manifest            |   3 +
 dev-build/cmake/cmake-3.29.6.ebuild | 282 ++++++++++++++++++++++++++++++++++++
 dev-build/cmake/cmake-9999.ebuild   |   2 +
 3 files changed, 287 insertions(+)