Re: [PATCH] Add googletest
Efraim Flashner writes: > On Sun, Jun 05, 2016 at 04:09:28PM -0500, Lukas Gradl wrote: >> Efraim Flashner writes: >> >> > On Thu, Jun 02, 2016 at 10:44:58AM -0500, Lukas Gradl wrote: >> >> >> >> Hi Leo, >> >> >> > ... >> >> >> >> The tool "pump.py" is distributed with the Googletest source. It >> >> appears to be a part of the source rather than a bundeled dependency and >> >> the only reference to it that I found was in Googletest, so I would >> >> assume it is a part of Googletest. Therefore I did not create a >> >> seperate package for that, but used the one provided in the Googletest >> >> source tree. With that, the pre-generated headers can be reproduced. >> >> The attached patch does that. >> >> >> >> Thank you! >> >> >> > >> > A couple last things I found, otherwise it looks great. >> > >> >> From 7d4aaba39821130ba824c4b2f8bd236e0d1f1639 Mon Sep 17 00:00:00 2001 >> >> From: Lukas Gradl >> >> Date: Thu, 2 Jun 2016 10:36:17 -0500 >> >> Subject: [PATCH] gnu: Add googletest. >> >> >> >> * gnu/packages/check.scm (googletest): New variable. >> >> --- >> >> gnu/packages/check.scm | 64 >> >> ++ >> >> 1 file changed, 64 insertions(+) >> >> >> >> diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm >> >> index 9eef7a9..4971c71 100644 >> >> --- a/gnu/packages/check.scm >> >> +++ b/gnu/packages/check.scm >> >> @@ -5,6 +5,7 @@ >> >> ;;; Copyright © 2015 Andreas Enge >> >> ;;; Copyright © 2016 Efraim Flashner >> >> ;;; Copyright © 2016 Roel Janssen >> >> +;;; Copyright © 2016 Lukas Gradl >> >> ;;; >> >> ;;; This file is part of GNU Guix. >> >> ;;; >> >> @@ -24,6 +25,7 @@ >> >> (define-module (gnu packages check) >> >>#:use-module (gnu packages) >> >>#:use-module (gnu packages autotools) >> >> + #:use-module (gnu packages python) >> >>#:use-module (guix licenses) >> >>#:use-module (guix packages) >> >>#:use-module (guix download) >> >> @@ -193,3 +195,65 @@ in the code. Cppcheck primarily detects the types >> >> of bugs that the compilers >> >> normally do not detect. The goal is to detect only real errors in the >> >> code >> >> (i.e. have zero false positives).") >> >> (license gpl3+))) >> >> + >> >> +(define-public googletest >> >> + (package >> >> +(name "googletest") >> >> +(version "1.7.0") >> >> +(source >> >> + (origin >> >> + (method url-fetch) >> >> + (uri >> >> +(string-append >> >> + "https://github.com/google/googletest/archive/release-"; >> >> + version ".tar.gz")) >> > >> > add:(file-name (string-append name "-" version ".tar.gz")) >> >> I added it. >> >> > >> >> + (sha256 >> >> +(base32 >> >> + "1k0nf1l9cb3prdmsvaajl5i31bx86c1mw0d5jgzykz7rzm36afpp" >> >> +(build-system gnu-build-system) >> >> +(native-inputs >> >> + `(("python-2" ,python-2) >> >> + ("autoconf" ,autoconf) >> >> + ("automake" ,automake) >> >> + ("libtool" ,libtool))) >> >> +(arguments >> >> + `( #:phases >> >> + (modify-phases %standard-phases >> >> + (add-before 'configure 'autoconf >> >> + (lambda _ >> >> +(zero? (system* "autoreconf" "-vfi" >> >> + (add-before 'autoconf 'generate-headers >> >> + (lambda _ >> >> + (begin >> >> + (delete-file "include/gtest/gtest-param-test.h") >> >> + (system* "python2" "scripts/pump.py" >> >> +"include/gtest/gtest-param-test.h.pump") >> >> + (delete-file "include/gtest/internal/gtest-tuple.h") >> >> + (system* "python2" "scripts/pump.py" >> >> +"include/gtest//internal/gtest-tuple.h.pump") >> >> + (delete-file >> >> +"include/gtest/internal/gtest-param-util-generated.h") >> >> + (system* >> >> +"python2" "scripts/pump.py" >> >> + >> >> "include/gtest/internal/gtest-param-util-generated.h.pump") >> >> + (delete-file "include/gtest/internal/gtest-type-util.h") >> >> + (system* "python2" "scripts/pump.py" >> >> + >> >> "include/gtest/internal/gtest-type-util.h.pump" >> >> + (replace 'install >> >> + (lambda _ >> >> + (let ((out (assoc-ref %outputs "out")) >> >> + (version version)) >> > ^ >> > you don't actually use this variable -- >> > >> >> Oh, Sorry about that. This is a leftover from a previous attemt that I >> forgot to remove. >> >> >> + (begin >> >> + (install-file "lib/.libs/libgtest_main.a" >> >> + (string-append out "/lib")) >> >> + (install-file "lib/.libs/libgtest.a" >> >> + (string-append out "/lib")) >> >> + (copy-recursi
Re: [PATCH] Add googletest
On Sun, Jun 05, 2016 at 04:09:28PM -0500, Lukas Gradl wrote: > Efraim Flashner writes: > > > On Thu, Jun 02, 2016 at 10:44:58AM -0500, Lukas Gradl wrote: > >> > >> Hi Leo, > >> > > ... > >> > >> The tool "pump.py" is distributed with the Googletest source. It > >> appears to be a part of the source rather than a bundeled dependency and > >> the only reference to it that I found was in Googletest, so I would > >> assume it is a part of Googletest. Therefore I did not create a > >> seperate package for that, but used the one provided in the Googletest > >> source tree. With that, the pre-generated headers can be reproduced. > >> The attached patch does that. > >> > >> Thank you! > >> > > > > A couple last things I found, otherwise it looks great. > > > >> From 7d4aaba39821130ba824c4b2f8bd236e0d1f1639 Mon Sep 17 00:00:00 2001 > >> From: Lukas Gradl > >> Date: Thu, 2 Jun 2016 10:36:17 -0500 > >> Subject: [PATCH] gnu: Add googletest. > >> > >> * gnu/packages/check.scm (googletest): New variable. > >> --- > >> gnu/packages/check.scm | 64 > >> ++ > >> 1 file changed, 64 insertions(+) > >> > >> diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm > >> index 9eef7a9..4971c71 100644 > >> --- a/gnu/packages/check.scm > >> +++ b/gnu/packages/check.scm > >> @@ -5,6 +5,7 @@ > >> ;;; Copyright © 2015 Andreas Enge > >> ;;; Copyright © 2016 Efraim Flashner > >> ;;; Copyright © 2016 Roel Janssen > >> +;;; Copyright © 2016 Lukas Gradl > >> ;;; > >> ;;; This file is part of GNU Guix. > >> ;;; > >> @@ -24,6 +25,7 @@ > >> (define-module (gnu packages check) > >>#:use-module (gnu packages) > >>#:use-module (gnu packages autotools) > >> + #:use-module (gnu packages python) > >>#:use-module (guix licenses) > >>#:use-module (guix packages) > >>#:use-module (guix download) > >> @@ -193,3 +195,65 @@ in the code. Cppcheck primarily detects the types of > >> bugs that the compilers > >> normally do not detect. The goal is to detect only real errors in the > >> code > >> (i.e. have zero false positives).") > >> (license gpl3+))) > >> + > >> +(define-public googletest > >> + (package > >> +(name "googletest") > >> +(version "1.7.0") > >> +(source > >> + (origin > >> + (method url-fetch) > >> + (uri > >> +(string-append > >> + "https://github.com/google/googletest/archive/release-"; > >> + version ".tar.gz")) > > > > add:(file-name (string-append name "-" version ".tar.gz")) > > I added it. > > > > >> + (sha256 > >> +(base32 > >> + "1k0nf1l9cb3prdmsvaajl5i31bx86c1mw0d5jgzykz7rzm36afpp" > >> +(build-system gnu-build-system) > >> +(native-inputs > >> + `(("python-2" ,python-2) > >> + ("autoconf" ,autoconf) > >> + ("automake" ,automake) > >> + ("libtool" ,libtool))) > >> +(arguments > >> + `( #:phases > >> + (modify-phases %standard-phases > >> + (add-before 'configure 'autoconf > >> + (lambda _ > >> +(zero? (system* "autoreconf" "-vfi" > >> + (add-before 'autoconf 'generate-headers > >> + (lambda _ > >> + (begin > >> + (delete-file "include/gtest/gtest-param-test.h") > >> + (system* "python2" "scripts/pump.py" > >> +"include/gtest/gtest-param-test.h.pump") > >> + (delete-file "include/gtest/internal/gtest-tuple.h") > >> + (system* "python2" "scripts/pump.py" > >> +"include/gtest//internal/gtest-tuple.h.pump") > >> + (delete-file > >> +"include/gtest/internal/gtest-param-util-generated.h") > >> + (system* > >> +"python2" "scripts/pump.py" > >> + > >> "include/gtest/internal/gtest-param-util-generated.h.pump") > >> + (delete-file "include/gtest/internal/gtest-type-util.h") > >> + (system* "python2" "scripts/pump.py" > >> + > >> "include/gtest/internal/gtest-type-util.h.pump" > >> + (replace 'install > >> + (lambda _ > >> + (let ((out (assoc-ref %outputs "out")) > >> + (version version)) > > ^ > > you don't actually use this variable -- > > > > Oh, Sorry about that. This is a leftover from a previous attemt that I > forgot to remove. > > >> + (begin > >> + (install-file "lib/.libs/libgtest_main.a" > >> + (string-append out "/lib")) > >> + (install-file "lib/.libs/libgtest.a" > >> + (string-append out "/lib")) > >> + (copy-recursively > >> + (string-append "include") > > > > don't need the string-append above > > > > Same here. > > >> + (string-ap
Re: [PATCH] Add googletest
Efraim Flashner writes: > On Thu, Jun 02, 2016 at 10:44:58AM -0500, Lukas Gradl wrote: >> >> Hi Leo, >> > ... >> >> The tool "pump.py" is distributed with the Googletest source. It >> appears to be a part of the source rather than a bundeled dependency and >> the only reference to it that I found was in Googletest, so I would >> assume it is a part of Googletest. Therefore I did not create a >> seperate package for that, but used the one provided in the Googletest >> source tree. With that, the pre-generated headers can be reproduced. >> The attached patch does that. >> >> Thank you! >> > > A couple last things I found, otherwise it looks great. > >> From 7d4aaba39821130ba824c4b2f8bd236e0d1f1639 Mon Sep 17 00:00:00 2001 >> From: Lukas Gradl >> Date: Thu, 2 Jun 2016 10:36:17 -0500 >> Subject: [PATCH] gnu: Add googletest. >> >> * gnu/packages/check.scm (googletest): New variable. >> --- >> gnu/packages/check.scm | 64 >> ++ >> 1 file changed, 64 insertions(+) >> >> diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm >> index 9eef7a9..4971c71 100644 >> --- a/gnu/packages/check.scm >> +++ b/gnu/packages/check.scm >> @@ -5,6 +5,7 @@ >> ;;; Copyright © 2015 Andreas Enge >> ;;; Copyright © 2016 Efraim Flashner >> ;;; Copyright © 2016 Roel Janssen >> +;;; Copyright © 2016 Lukas Gradl >> ;;; >> ;;; This file is part of GNU Guix. >> ;;; >> @@ -24,6 +25,7 @@ >> (define-module (gnu packages check) >>#:use-module (gnu packages) >>#:use-module (gnu packages autotools) >> + #:use-module (gnu packages python) >>#:use-module (guix licenses) >>#:use-module (guix packages) >>#:use-module (guix download) >> @@ -193,3 +195,65 @@ in the code. Cppcheck primarily detects the types of >> bugs that the compilers >> normally do not detect. The goal is to detect only real errors in the code >> (i.e. have zero false positives).") >> (license gpl3+))) >> + >> +(define-public googletest >> + (package >> +(name "googletest") >> +(version "1.7.0") >> +(source >> + (origin >> + (method url-fetch) >> + (uri >> +(string-append >> + "https://github.com/google/googletest/archive/release-"; >> + version ".tar.gz")) > > add:(file-name (string-append name "-" version ".tar.gz")) I added it. > >> + (sha256 >> +(base32 >> + "1k0nf1l9cb3prdmsvaajl5i31bx86c1mw0d5jgzykz7rzm36afpp" >> +(build-system gnu-build-system) >> +(native-inputs >> + `(("python-2" ,python-2) >> + ("autoconf" ,autoconf) >> + ("automake" ,automake) >> + ("libtool" ,libtool))) >> +(arguments >> + `( #:phases >> + (modify-phases %standard-phases >> + (add-before 'configure 'autoconf >> + (lambda _ >> +(zero? (system* "autoreconf" "-vfi" >> + (add-before 'autoconf 'generate-headers >> + (lambda _ >> + (begin >> + (delete-file "include/gtest/gtest-param-test.h") >> + (system* "python2" "scripts/pump.py" >> +"include/gtest/gtest-param-test.h.pump") >> + (delete-file "include/gtest/internal/gtest-tuple.h") >> + (system* "python2" "scripts/pump.py" >> +"include/gtest//internal/gtest-tuple.h.pump") >> + (delete-file >> +"include/gtest/internal/gtest-param-util-generated.h") >> + (system* >> +"python2" "scripts/pump.py" >> +"include/gtest/internal/gtest-param-util-generated.h.pump") >> + (delete-file "include/gtest/internal/gtest-type-util.h") >> + (system* "python2" "scripts/pump.py" >> +"include/gtest/internal/gtest-type-util.h.pump" >> + (replace 'install >> + (lambda _ >> + (let ((out (assoc-ref %outputs "out")) >> + (version version)) > ^ > you don't actually use this variable -- > Oh, Sorry about that. This is a leftover from a previous attemt that I forgot to remove. >> + (begin >> + (install-file "lib/.libs/libgtest_main.a" >> + (string-append out "/lib")) >> + (install-file "lib/.libs/libgtest.a" >> + (string-append out "/lib")) >> + (copy-recursively >> + (string-append "include") > > don't need the string-append above > Same here. >> + (string-append out "/include") >> +(home-page "https://github.com/google/googletest/";) >> +(synopsis "Test discovery and XUnit test framework") >> +(description "Google Test features an XUnit test framework, automated >> test >> +discovery, death tests, assertions, parameterized tests and XML test report >> +generation.") >> +
Re: [PATCH] Add googletest
On Thu, Jun 02, 2016 at 10:44:58AM -0500, Lukas Gradl wrote: > > Hi Leo, > ... > > The tool "pump.py" is distributed with the Googletest source. It > appears to be a part of the source rather than a bundeled dependency and > the only reference to it that I found was in Googletest, so I would > assume it is a part of Googletest. Therefore I did not create a > seperate package for that, but used the one provided in the Googletest > source tree. With that, the pre-generated headers can be reproduced. > The attached patch does that. > > Thank you! > A couple last things I found, otherwise it looks great. > From 7d4aaba39821130ba824c4b2f8bd236e0d1f1639 Mon Sep 17 00:00:00 2001 > From: Lukas Gradl > Date: Thu, 2 Jun 2016 10:36:17 -0500 > Subject: [PATCH] gnu: Add googletest. > > * gnu/packages/check.scm (googletest): New variable. > --- > gnu/packages/check.scm | 64 > ++ > 1 file changed, 64 insertions(+) > > diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm > index 9eef7a9..4971c71 100644 > --- a/gnu/packages/check.scm > +++ b/gnu/packages/check.scm > @@ -5,6 +5,7 @@ > ;;; Copyright © 2015 Andreas Enge > ;;; Copyright © 2016 Efraim Flashner > ;;; Copyright © 2016 Roel Janssen > +;;; Copyright © 2016 Lukas Gradl > ;;; > ;;; This file is part of GNU Guix. > ;;; > @@ -24,6 +25,7 @@ > (define-module (gnu packages check) >#:use-module (gnu packages) >#:use-module (gnu packages autotools) > + #:use-module (gnu packages python) >#:use-module (guix licenses) >#:use-module (guix packages) >#:use-module (guix download) > @@ -193,3 +195,65 @@ in the code. Cppcheck primarily detects the types of > bugs that the compilers > normally do not detect. The goal is to detect only real errors in the code > (i.e. have zero false positives).") > (license gpl3+))) > + > +(define-public googletest > + (package > +(name "googletest") > +(version "1.7.0") > +(source > + (origin > + (method url-fetch) > + (uri > +(string-append > + "https://github.com/google/googletest/archive/release-"; > + version ".tar.gz")) add:(file-name (string-append name "-" version ".tar.gz")) > + (sha256 > +(base32 > + "1k0nf1l9cb3prdmsvaajl5i31bx86c1mw0d5jgzykz7rzm36afpp" > +(build-system gnu-build-system) > +(native-inputs > + `(("python-2" ,python-2) > + ("autoconf" ,autoconf) > + ("automake" ,automake) > + ("libtool" ,libtool))) > +(arguments > + `( #:phases > + (modify-phases %standard-phases > + (add-before 'configure 'autoconf > + (lambda _ > +(zero? (system* "autoreconf" "-vfi" > + (add-before 'autoconf 'generate-headers > + (lambda _ > + (begin > + (delete-file "include/gtest/gtest-param-test.h") > + (system* "python2" "scripts/pump.py" > +"include/gtest/gtest-param-test.h.pump") > + (delete-file "include/gtest/internal/gtest-tuple.h") > + (system* "python2" "scripts/pump.py" > +"include/gtest//internal/gtest-tuple.h.pump") > + (delete-file > +"include/gtest/internal/gtest-param-util-generated.h") > + (system* > +"python2" "scripts/pump.py" > +"include/gtest/internal/gtest-param-util-generated.h.pump") > + (delete-file "include/gtest/internal/gtest-type-util.h") > + (system* "python2" "scripts/pump.py" > +"include/gtest/internal/gtest-type-util.h.pump" > + (replace 'install > + (lambda _ > + (let ((out (assoc-ref %outputs "out")) > + (version version)) ^ you don't actually use this variable -- > + (begin > + (install-file "lib/.libs/libgtest_main.a" > + (string-append out "/lib")) > + (install-file "lib/.libs/libgtest.a" > + (string-append out "/lib")) > + (copy-recursively > + (string-append "include") don't need the string-append above > + (string-append out "/include") > +(home-page "https://github.com/google/googletest/";) > +(synopsis "Test discovery and XUnit test framework") > +(description "Google Test features an XUnit test framework, automated > test > +discovery, death tests, assertions, parameterized tests and XML test report > +generation.") > +(license bsd-3))) > -- > 2.7.4 > -- Efraim Flashner אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted signature.asc Description: PGP signature
Re: [PATCH] Add googletest
Hi Leo, Leo Famulari writes: > On Wed, Jun 01, 2016 at 09:57:53AM -0500, Lukas Gradl wrote: >> Leo Famulari writes: >> > I noticed in the README.md that upstream suggests use of GNU Make unless >> > building from a Git checkout. Did you try that? >> >> I have not tried that. Their README.md left me under the impression >> that tests are not supported useing GNU Make, but I just looked at their >> Makefile and there are test targets, so I can try using the >> gnu-build-system, if that is preferred? > > All else being equal, my personal preference is to use GNU tools. But > otherwise, we should use the build system that upstream supports most > fully. Especially, we should use the one that runs the tests. Sounds good. I switched to the gnu-build-system + autotools. This also made the "install" phase somewhat easier. >> It seems they generate c++ header files using pump.py. I will look into >> that more. Python is also needed for tests. > > Thanks for looking into that. If the header files are generated by other > code, then we prefer to build them ourselves rather than use the > "pre-compiled" headers. The tool "pump.py" is distributed with the Googletest source. It appears to be a part of the source rather than a bundeled dependency and the only reference to it that I found was in Googletest, so I would assume it is a part of Googletest. Therefore I did not create a seperate package for that, but used the one provided in the Googletest source tree. With that, the pre-generated headers can be reproduced. The attached patch does that. Thank you! >From 7d4aaba39821130ba824c4b2f8bd236e0d1f1639 Mon Sep 17 00:00:00 2001 From: Lukas Gradl Date: Thu, 2 Jun 2016 10:36:17 -0500 Subject: [PATCH] gnu: Add googletest. * gnu/packages/check.scm (googletest): New variable. --- gnu/packages/check.scm | 64 ++ 1 file changed, 64 insertions(+) diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm index 9eef7a9..4971c71 100644 --- a/gnu/packages/check.scm +++ b/gnu/packages/check.scm @@ -5,6 +5,7 @@ ;;; Copyright © 2015 Andreas Enge ;;; Copyright © 2016 Efraim Flashner ;;; Copyright © 2016 Roel Janssen +;;; Copyright © 2016 Lukas Gradl ;;; ;;; This file is part of GNU Guix. ;;; @@ -24,6 +25,7 @@ (define-module (gnu packages check) #:use-module (gnu packages) #:use-module (gnu packages autotools) + #:use-module (gnu packages python) #:use-module (guix licenses) #:use-module (guix packages) #:use-module (guix download) @@ -193,3 +195,65 @@ in the code. Cppcheck primarily detects the types of bugs that the compilers normally do not detect. The goal is to detect only real errors in the code (i.e. have zero false positives).") (license gpl3+))) + +(define-public googletest + (package +(name "googletest") +(version "1.7.0") +(source + (origin + (method url-fetch) + (uri +(string-append + "https://github.com/google/googletest/archive/release-"; + version ".tar.gz")) + (sha256 +(base32 + "1k0nf1l9cb3prdmsvaajl5i31bx86c1mw0d5jgzykz7rzm36afpp" +(build-system gnu-build-system) +(native-inputs + `(("python-2" ,python-2) + ("autoconf" ,autoconf) + ("automake" ,automake) + ("libtool" ,libtool))) +(arguments + `( #:phases + (modify-phases %standard-phases + (add-before 'configure 'autoconf + (lambda _ +(zero? (system* "autoreconf" "-vfi" + (add-before 'autoconf 'generate-headers + (lambda _ + (begin + (delete-file "include/gtest/gtest-param-test.h") + (system* "python2" "scripts/pump.py" +"include/gtest/gtest-param-test.h.pump") + (delete-file "include/gtest/internal/gtest-tuple.h") + (system* "python2" "scripts/pump.py" +"include/gtest//internal/gtest-tuple.h.pump") + (delete-file +"include/gtest/internal/gtest-param-util-generated.h") + (system* +"python2" "scripts/pump.py" +"include/gtest/internal/gtest-param-util-generated.h.pump") + (delete-file "include/gtest/internal/gtest-type-util.h") + (system* "python2" "scripts/pump.py" +"include/gtest/internal/gtest-type-util.h.pump" + (replace 'install + (lambda _ + (let ((out (assoc-ref %outputs "out")) + (version version)) + (begin + (install-file "lib/.libs/libgtest_main.a" + (string-append out "/lib")) + (install-file "lib/.libs/libgtest.a" + (string-append out "/lib")) + (copy-recursively + (string-append "include") + (string-append out "/
Re: [PATCH] Add googletest
Hi Efraim, Thank you for your review! Efraim Flashner writes: > On Tue, May 31, 2016 at 05:44:00PM -0400, Leo Famulari wrote: >> On Tue, May 31, 2016 at 08:53:17AM -0500, Lukas Gradl wrote: >> > * gnu/packages/check.scm (googletest): New variable. >> >> Awesome, thanks for this patch! >> >> > +(build-system cmake-build-system) >> >> I noticed in the README.md that upstream suggests use of GNU Make unless >> building from a Git checkout. Did you try that? >> >> > +(native-inputs >> > + `(("python-2" ,python-2))) >> >> The README also suggests that this is only necessary for building from >> Git, although it's worth finding out what they mean by "re-generating >> certain source files from templates". We prefer to re-build "generated" >> source files since they are not really "source files" in many cases. >> >> > + (replace 'install >> > +(lambda _ >> > + (let ((out (assoc-ref %outputs "out"))) >> > +(and >> > + (mkdir-p (string-append out "/lib")) >> > + (mkdir-p (string-append out "/include")) >> > + (zero? >> > + (system* "cp" "-r" >> > + "../googletest-release-1.7.0/include" >> > + out)) >> > + (zero? (system* "cp" "libgtest.a" >> > "libgtest_main.a" >> > + (string-append out >> > "/lib")) >> >> I think these uses of (system*) could be replaced by (copy-recursively) >> and (install-file), respectively. >> > > I haven't tried building this package yet > > (and -> you can probably switch this for a (begin OK, this is done. > googletest-release-1.7.0 -> > (string-append "../google-release-" version "/include") I seem to have some trouble with that. The package passes all phases except for "install". When it reaches install I get the following error: ---8<--- cut here start --->8--- Start 40: gtest_xml_outfiles_test 40/41 Test #40: gtest_xml_outfiles_test Passed0.19 sec Start 41: gtest_xml_output_unittest 41/41 Test #41: gtest_xml_output_unittest .. Passed0.12 sec 100% tests passed, 0 tests failed out of 41 Total Test time (real) = 4.68 sec phase `check' succeeded after 6.7 seconds starting phase `install' Backtrace: In ice-9/boot-9.scm: 157: 13 [catch #t # ...] In unknown file: ?: 12 [apply-smob/1 #] In ice-9/boot-9.scm: 63: 11 [call-with-prompt prompt0 ...] In ice-9/eval.scm: 432: 10 [eval # #] In ice-9/boot-9.scm: 2401: 9 [save-module-excursion #] 4050: 8 [#] 1724: 7 [%start-stack load-stack #] 1729: 6 [#] In unknown file: ?: 5 [primitive-load "/gnu/store/yjwm79h6b6fswb9wdy2ak8b1bdz6s7hw-googletest-1.7.0-guile-builder"] In ice-9/eval.scm: 387: 4 [eval # ()] In srfi/srfi-1.scm: 830: 3 [every1 # ...] In /gnu/store/915kgcfqqasrnf18rhznc77z96i01i6d-module-import/guix/build/gnu-build-system.scm: 593: 2 [# #] In ice-9/eval.scm: 387: 1 [eval # #] In unknown file: ?: 0 [string-append "../googletest-release-" # "/include"] ERROR: In procedure string-append: ERROR: In procedure string-append: Wrong type (expecting string): # builder for `/gnu/store/ljnpvx73fqdg455nh1kj2ac2i3k9jy3r-googletest-1.7.0.drv' failed with exit code 1 @ build-failed /gnu/store/ljnpvx73fqdg455nh1kj2ac2i3k9jy3r-googletest-1.7.0.drv - 1 builder for `/gnu/store/ljnpvx73fqdg455nh1kj2ac2i3k9jy3r-googletest-1.7.0.drv' failed with exit code 1 guix build: error: build failed: build of `/gnu/store/ljnpvx73fqdg455nh1kj2ac2i3k9jy3r-googletest-1.7.0.drv' failed lukas@serenity$ ---8<--- cut here end ->8--- The patch that created this error is attached. I am not sure what is happening here. Is all of the code in the argument of modify phases executed in a different scope where "version" is something else? I do not quite understand this. > (zero? (system* "cp" "libgtest.a" "libgtest_main.a" > won't this just copy libgtest.a to libgtest_main.a ? I am not sure, but if it behaves like invoking "cp" on the command line, then it should copy both of these to the directory that is on the next line: + (zero? (system* "cp" "libgtest.a" "libgtest_main.a" + (string-append out "/lib")) I replaced all these statements with (install-file ...) and (copy-recursively ...) as mentioned by Leo. Thank you for your help! Best, Lukas >From c0cea38f6ccb281aa105aba00b3da3e4a5c8f7ef Mon Sep 17 00:00:00 2001 From: Lukas Gradl Date: Thu, 2 Jun 2016 08:38:23 -0500 Subject: [PATCH] gnu: Add googletest. * gnu/packages/check.scm (googletest): New variable. --- gnu/packages/check.scm | 44 1 file changed, 44 insertions(+) diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm index 9eef7
Re: [PATCH] Add googletest
On Wed, Jun 01, 2016 at 09:57:53AM -0500, Lukas Gradl wrote: > Leo Famulari writes: > > I noticed in the README.md that upstream suggests use of GNU Make unless > > building from a Git checkout. Did you try that? > > I have not tried that. Their README.md left me under the impression > that tests are not supported useing GNU Make, but I just looked at their > Makefile and there are test targets, so I can try using the > gnu-build-system, if that is preferred? All else being equal, my personal preference is to use GNU tools. But otherwise, we should use the build system that upstream supports most fully. Especially, we should use the one that runs the tests. > It seems they generate c++ header files using pump.py. I will look into > that more. Python is also needed for tests. Thanks for looking into that. If the header files are generated by other code, then we prefer to build them ourselves rather than use the "pre-compiled" headers.
Re: [PATCH] Add googletest
Thank you for your review! Leo Famulari writes: > On Tue, May 31, 2016 at 08:53:17AM -0500, Lukas Gradl wrote: >> * gnu/packages/check.scm (googletest): New variable. > > Awesome, thanks for this patch! > >> +(build-system cmake-build-system) > > I noticed in the README.md that upstream suggests use of GNU Make unless > building from a Git checkout. Did you try that? I have not tried that. Their README.md left me under the impression that tests are not supported useing GNU Make, but I just looked at their Makefile and there are test targets, so I can try using the gnu-build-system, if that is preferred? > >> +(native-inputs >> + `(("python-2" ,python-2))) > > The README also suggests that this is only necessary for building from > Git, although it's worth finding out what they mean by "re-generating > certain source files from templates". We prefer to re-build "generated" > source files since they are not really "source files" in many cases. > It seems they generate c++ header files using pump.py. I will look into that more. Python is also needed for tests. >> + (replace 'install >> +(lambda _ >> + (let ((out (assoc-ref %outputs "out"))) >> +(and >> + (mkdir-p (string-append out "/lib")) >> + (mkdir-p (string-append out "/include")) >> + (zero? >> + (system* "cp" "-r" >> + "../googletest-release-1.7.0/include" >> + out)) >> + (zero? (system* "cp" "libgtest.a" "libgtest_main.a" >> + (string-append out "/lib")) > > I think these uses of (system*) could be replaced by (copy-recursively) > and (install-file), respectively. These indeed sound very useful. I will send an updated patch shortly. Thank you! Best, Lukas
Re: [PATCH] Add googletest
Leo Famulari skribis: > On Tue, May 31, 2016 at 08:53:17AM -0500, Lukas Gradl wrote: [...] >> + (replace 'install >> +(lambda _ >> + (let ((out (assoc-ref %outputs "out"))) >> +(and >> + (mkdir-p (string-append out "/lib")) >> + (mkdir-p (string-append out "/include")) >> + (zero? >> + (system* "cp" "-r" >> + "../googletest-release-1.7.0/include" >> + out)) >> + (zero? (system* "cp" "libgtest.a" "libgtest_main.a" >> + (string-append out "/lib")) > > I think these uses of (system*) could be replaced by (copy-recursively) > and (install-file), respectively. s/could/should/ even ;-) Ludo’.
Re: [PATCH] Add googletest
On Tue, May 31, 2016 at 05:44:00PM -0400, Leo Famulari wrote: > On Tue, May 31, 2016 at 08:53:17AM -0500, Lukas Gradl wrote: > > * gnu/packages/check.scm (googletest): New variable. > > Awesome, thanks for this patch! > > > +(build-system cmake-build-system) > > I noticed in the README.md that upstream suggests use of GNU Make unless > building from a Git checkout. Did you try that? > > > +(native-inputs > > + `(("python-2" ,python-2))) > > The README also suggests that this is only necessary for building from > Git, although it's worth finding out what they mean by "re-generating > certain source files from templates". We prefer to re-build "generated" > source files since they are not really "source files" in many cases. > > > + (replace 'install > > +(lambda _ > > + (let ((out (assoc-ref %outputs "out"))) > > +(and > > + (mkdir-p (string-append out "/lib")) > > + (mkdir-p (string-append out "/include")) > > + (zero? > > + (system* "cp" "-r" > > + "../googletest-release-1.7.0/include" > > + out)) > > + (zero? (system* "cp" "libgtest.a" > > "libgtest_main.a" > > + (string-append out > > "/lib")) > > I think these uses of (system*) could be replaced by (copy-recursively) > and (install-file), respectively. > I haven't tried building this package yet (and -> you can probably switch this for a (begin googletest-release-1.7.0 -> (string-append "../google-release-" version "/include") (zero? (system* "cp" "libgtest.a" "libgtest_main.a" won't this just copy libgtest.a to libgtest_main.a ? -- Efraim Flashner אפרים פלשנר GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted signature.asc Description: PGP signature
Re: [PATCH] Add googletest
On Tue, May 31, 2016 at 08:53:17AM -0500, Lukas Gradl wrote: > * gnu/packages/check.scm (googletest): New variable. Awesome, thanks for this patch! > +(build-system cmake-build-system) I noticed in the README.md that upstream suggests use of GNU Make unless building from a Git checkout. Did you try that? > +(native-inputs > + `(("python-2" ,python-2))) The README also suggests that this is only necessary for building from Git, although it's worth finding out what they mean by "re-generating certain source files from templates". We prefer to re-build "generated" source files since they are not really "source files" in many cases. > + (replace 'install > +(lambda _ > + (let ((out (assoc-ref %outputs "out"))) > +(and > + (mkdir-p (string-append out "/lib")) > + (mkdir-p (string-append out "/include")) > + (zero? > + (system* "cp" "-r" > + "../googletest-release-1.7.0/include" > + out)) > + (zero? (system* "cp" "libgtest.a" "libgtest_main.a" > + (string-append out "/lib")) I think these uses of (system*) could be replaced by (copy-recursively) and (install-file), respectively.
Re: [PATCH] Add googletest
Hello, Sorry, I accidently attached the wrong patch. Please disregard the patch in my previous email, attached is an updated one. Sorry about that! Best, Lukas >From f5f229eafcf2ebf24703e6becc291b8378dff2e0 Mon Sep 17 00:00:00 2001 From: Lukas Gradl Date: Tue, 31 May 2016 08:48:29 -0500 Subject: [PATCH] gnu: check: Add googletest. * gnu/packages/check.scm (googletest): New variable. --- gnu/packages/check.scm | 42 ++ 1 file changed, 42 insertions(+) diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm index 9eef7a9..fca2e98 100644 --- a/gnu/packages/check.scm +++ b/gnu/packages/check.scm @@ -24,6 +24,7 @@ (define-module (gnu packages check) #:use-module (gnu packages) #:use-module (gnu packages autotools) + #:use-module (gnu packages python) #:use-module (guix licenses) #:use-module (guix packages) #:use-module (guix download) @@ -193,3 +194,44 @@ in the code. Cppcheck primarily detects the types of bugs that the compilers normally do not detect. The goal is to detect only real errors in the code (i.e. have zero false positives).") (license gpl3+))) + + +(define-public googletest + (package +(name "googletest") +(version "1.7.0") +(source + (origin + (method url-fetch) + (uri +(string-append + "https://github.com/google/googletest/archive/release-"; + version ".tar.gz")) + (file-name (string-append name "-" version ".tar.gz")) + (sha256 +(base32 + "1k0nf1l9cb3prdmsvaajl5i31bx86c1mw0d5jgzykz7rzm36afpp" +(build-system cmake-build-system) +(native-inputs + `(("python-2" ,python-2))) +(arguments + `(#:configure-flags '("-Dgtest_build_tests=ON") + #:phases (modify-phases %standard-phases + (replace 'install +(lambda _ + (let ((out (assoc-ref %outputs "out"))) +(and + (mkdir-p (string-append out "/lib")) + (mkdir-p (string-append out "/include")) + (zero? + (system* "cp" "-r" + "../googletest-release-1.7.0/include" + out)) + (zero? (system* "cp" "libgtest.a" "libgtest_main.a" + (string-append out "/lib")) +(home-page "https://github.com/google/googletest/";) +(synopsis "Test discovery and XUnit test framework") +(description "Google Test (GTest) features an XUnit test framework, +automated test discovery, death tests, assertions, parameterized tests and XML +test report generation.") +(license bsd-3))) -- 2.7.4
[PATCH] Add googletest
Hello Guix, This patch adds googletest, a native-input for the c/c++ version of msgpack (coming soon). There is no "install" target. At the moment the patch copies the build results to out/lib and the include directory of the source tree to out/include. I am not sure if this is enough or if something else needs to be installed. Opinions are welcome! Thank you! >From 0560038dd1c529cca46e31c6afbb5cd16808db43 Mon Sep 17 00:00:00 2001 From: Lukas Gradl Date: Mon, 30 May 2016 15:35:10 -0500 Subject: [PATCH] gnu: check: Add googletest. * gnu/packages/check.scm (googletest): New variable. --- gnu/packages/check.scm | 41 + 1 file changed, 41 insertions(+) diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm index 9eef7a9..f6b586a 100644 --- a/gnu/packages/check.scm +++ b/gnu/packages/check.scm @@ -5,6 +5,7 @@ ;;; Copyright © 2015 Andreas Enge ;;; Copyright © 2016 Efraim Flashner ;;; Copyright © 2016 Roel Janssen +;;; Copyright © 2016 Lukas Gradl ;;; ;;; This file is part of GNU Guix. ;;; @@ -24,6 +25,7 @@ (define-module (gnu packages check) #:use-module (gnu packages) #:use-module (gnu packages autotools) + #:use-module (gnu packages python) #:use-module (guix licenses) #:use-module (guix packages) #:use-module (guix download) @@ -193,3 +195,42 @@ in the code. Cppcheck primarily detects the types of bugs that the compilers normally do not detect. The goal is to detect only real errors in the code (i.e. have zero false positives).") (license gpl3+))) + +(define-public googletest + (package +(name "googletest") +(version "1.7.0") +(source + (origin + (method url-fetch) + (uri +(string-append + "https://github.com/google/googletest/archive/release-"; + version ".tar.gz")) + (sha256 +(base32 + "1k0nf1l9cb3prdmsvaajl5i31bx86c1mw0d5jgzykz7rzm36afpp" +(build-system cmake-build-system) +(native-inputs + `(("python-2" ,python-2))) +(arguments + `(#:tests? #f + ;#:configure-flags '("-Dgtest_build_tests=ON") + #:phases (modify-phases %standard-phases + (replace 'install +(lambda _ + (let ((out (assoc-ref %outputs "out"))) +(and + (mkdir-p (string-append out "/lib")) + (mkdir-p (string-append out "/include")) + (zero? (system* "cp" "libgtest.a" "libgtest_main.a" + (string-append out "/lib"))) + (system* "ls") + (zero? (system* "ls" "..") +(home-page "https://github.com/google/googletest/";) +(synopsis "Test discovery and XUnit test framework") +(description "Google Test features an XUnit test framework, automated test +discovery, death tests, assertions, parameterized tests and XML test report +generation.") +(license bsd-3))) + -- 2.7.4