[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#50299: [PATCH v2 01/27] lint: check-tests-true: Allow #:tests? #t fo
From: |
Maxim Cournoyer |
Subject: |
bug#50299: [PATCH v2 01/27] lint: check-tests-true: Allow #:tests? #t for some build systems. |
Date: |
Mon, 27 Sep 2021 21:14:57 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Hello!
Maxime Devos <maximedevos@telenet.be> writes:
> emacs-build-system sets #:tests? #f by default, so the linter
> shouldn't warn if #:tests? #t is set for packages using
> emacs-build-system. Likewise for texlive-build-system
>
> * guix/lint.scm (check-tests-true): Do not warn if the build system
> is emacs-build-system or texlive-build-system.
> * tests/lint.scm
> ("tests-true: #:tests? #t acceptable for emacs packages")
> ("tests-true: #:tests? #t acceptable for texlive packages"): New tests.
[...]
> diff --git a/tests/lint.scm b/tests/lint.scm
> index 0f51b9ef79..7b04dc98d4 100644
> --- a/tests/lint.scm
> +++ b/tests/lint.scm
> @@ -35,6 +35,7 @@
> #:use-module (guix tests http)
> #:use-module (guix download)
> #:use-module (guix git-download)
> + #:use-module (guix build-system emacs)
> #:use-module (guix build-system gnu)
> #:use-module (guix packages)
> #:use-module (guix lint)
> @@ -324,6 +325,25 @@
> `(#:tests? ,(not (%current-target-system)))))))
> (check-tests-true pkg)))
>
> +;; The emacs-build-system sets #:tests? #f by default.
> +(test-equal "tests-true: #:tests? #t acceptable for emacs packages"
> + '()
> + (let ((pkg (dummy-package "x"
> + (build-system emacs-build-system)
> + (arguments
> + `(#:tests? #t)))))
> + (check-tests-true pkg)))
> +
> +;; Likewise, though the 'check' phase is deleted by default,
> +;; so #:tests? #t won't be useful by itself.
> +(test-equal "tests-true: #:tests? #t acceptable for texlive packages"
> + '()
> + (let ((pkg (dummy-package "x"
> + (build-system emacs-build-system)
^ texlive-build-system ? :-)
Thanks!
Maxim
- bug#50299: [PATCH v2 21/27] gnu: lablgtk: Don't run tests when cross-compiling., (continued)
- bug#50299: [PATCH v2 21/27] gnu: lablgtk: Don't run tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 15/27] gnu: ecl: Don't pretend to enable tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 07/27] gnu: swi-prolog: Don't use 'cc'., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 20/27] gnu: ocaml4.07-fftw3: Don't run tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 19/27] gnu: python2-promise: Don't run tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 23/27] gnu: pjproject: Don't run tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 24/27] gnu: tdlib: Don't run tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 25/27] gnu: extra-cmake-modules: Don't run tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 08/27] gnu: swi-prolog: Use cross-compiled bash in shebangs., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 16/27] gnu: perl-unicode-utf8: Don't run tests when cross-compiling., Maxime Devos, 2021/09/27
- bug#50299: [PATCH v2 01/27] lint: check-tests-true: Allow #:tests? #t for some build systems.,
Maxim Cournoyer <=