GNU bug report logs -
#72106
[PATCH 0/3] gnu: klee: Enable test suite
Previous Next
Reported by: soeren <at> soeren-tempel.net
Date: Sun, 14 Jul 2024 10:38:02 UTC
Severity: normal
Tags: patch
Done: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 72106 in the body.
You can then email your comments to 72106 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 10:38:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
soeren <at> soeren-tempel.net
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Sun, 14 Jul 2024 10:38:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
The KLEE version packaged added in #71634 did not enable the KLEE
test suite. This patchset re-adds the changes from #68296 to get
the test suite to pass. Additionally, while working on this, I
noticed that the current KLEE package does not propagate the Python
dependencies needed by klee-stats, which I fixed as well. Lastly,
I also passed a custom #:strip-directories argument to avoid stripping
of LLVM bitcode which (prior to this patchset) caused strip(1) to
emit several warnings in the build log.
P.S.: I also think it might be worthwhile to only enable the KLEE
package on x86_64 Linux <https://issues.guix.gnu.org/71925#18>. Let me
know if I should add that as well :-)
Sören Tempel (3):
gnu: klee: Propagate Python dependencies needed by klee-stats.
gnu: klee: Enable the test suite.
gnu: klee: Only strip bin directory.
gnu/packages/check.scm | 32 +++++++++++++++++++++++++++++---
1 file changed, 29 insertions(+), 3 deletions(-)
base-commit: c1d367f57e89c3f2efd964e6d638bd89b0b1df97
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 10:43:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
Without these dependencies, the klee-stats Python script,
which is shipped by the klee package, does not work correctly.
* gnu/packages/check.scm (klee): Add propagated Python inputs.
---
gnu/packages/check.scm | 2 ++
1 file changed, 2 insertions(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 3bfcf5a37e..f31e1bdb36 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1087,6 +1087,8 @@ (define-public klee
(string-append "-DKLEE_UCLIBC_PATH="
(search-input-file %build-inputs "/lib/klee/libc.a"))
"-DENABLE_POSIX_RUNTIME=ON")))
+ ;; klee-stats tool (shipped in /bin) requires Python.
+ (propagated-inputs (list python python-tabulate))
(native-inputs (list clang-13 llvm-13 python-lit))
(inputs (list bash-minimal klee-uclibc gperftools sqlite z3))
(build-system cmake-build-system)
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 10:43:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Enable all tests.
[arguments]: Add phase to patch lit configuration, set #:test-target.
<#:configure-flags?>: Enable system and unit tests, configure gtest.
[inputs]: Add googletest and python-lit.
---
gnu/packages/check.scm | 29 ++++++++++++++++++++++++++---
1 file changed, 26 insertions(+), 3 deletions(-)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index f31e1bdb36..14d7124908 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,13 +1063,23 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:test-target "check"
#:phases
#~(modify-phases %standard-phases
- (add-after 'unpack 'patch
+ (add-after 'unpack 'patch-cmake
(lambda _
(substitute* "CMakeLists.txt"
(("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
"${KLEE_UCLIBC_PATH}"))))
+ (add-after 'unpack 'patch-lit-config
+ (lambda _
+ ;; Make sure that we retain the value of the GUIX_PYTHONPATH
+ ;; environment variable in the test environmented created by
+ ;; python-lit. Otherwise, the test scripts won't be able to
+ ;; find the python-tabulate dependency, causing test failures.
+ (substitute* "test/lit.cfg"
+ (("addEnv\\('PWD'\\)" env)
+ (string-append env "\n" "addEnv('GUIX_PYTHONPATH')")))))
(add-after 'install 'wrap-hooks
(lambda* (#:key inputs outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
@@ -1080,7 +1090,13 @@ (define-public klee
`("KLEE_RUNTIME_LIBRARY_PATH" =
(,(string-append lib "/klee/runtime/"))))))))
#:configure-flags
- #~(list (string-append "-DLLVMCC="
+ #~(list "-DENABLE_UNIT_TESTS=ON"
+ "-DENABLE_SYSTEM_TESTS=ON"
+ (string-append "-DGTEST_SRC_DIR="
+ (assoc-ref %build-inputs "googletest"))
+ (string-append "-DGTEST_INCLUDE_DIR="
+ (assoc-ref %build-inputs "googletest") "/googletest/include")
+ (string-append "-DLLVMCC="
(search-input-file %build-inputs "/bin/clang"))
(string-append "-DLLVMCXX="
(search-input-file %build-inputs "/bin/clang++"))
@@ -1090,7 +1106,14 @@ (define-public klee
;; klee-stats tool (shipped in /bin) requires Python.
(propagated-inputs (list python python-tabulate))
(native-inputs (list clang-13 llvm-13 python-lit))
- (inputs (list bash-minimal klee-uclibc gperftools sqlite z3))
+ (inputs
+ `(("bash-minimal" ,bash-minimal)
+ ("klee-uclibc" ,klee-uclibc)
+ ("gperftools" ,gperftools)
+ ("sqlite" ,sqlite)
+ ("z3" ,z3)
+ ("python-lit" ,python-lit)
+ ("googletest" ,(package-source googletest))))
(build-system cmake-build-system)
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 10:43:03 GMT)
Full text and
rfc822 format available.
Message #14 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
This avoid several warnings to be emitted by strip(1) when attempting
to strip .bca (i.e. LLVM bitcode) files shipped in /lib/klee/runtime/.
* gnu/packages/check.scm (klee): Set #:strip-directories.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 14d7124908..47e99a6be5 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,6 +1063,7 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:strip-directories #~(list "bin") ;don't strip LLVM bitcode in /lib
#:test-target "check"
#:phases
#~(modify-phases %standard-phases
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 11:45:01 GMT)
Full text and
rfc822 format available.
Message #17 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Sonntag, dem 14.07.2024 um 12:42 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> Without these dependencies, the klee-stats Python script,
> which is shipped by the klee package, does not work correctly.
>
> * gnu/packages/check.scm (klee): Add propagated Python inputs.
Can we use a wrapper instead?
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 16:19:01 GMT)
Full text and
rfc822 format available.
Message #20 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Sonntag, dem 14.07.2024 um 12:36 +0200 schrieb
soeren <at> soeren-tempel.net:
> Lastly, I also passed a custom #:strip-directories argument to avoid
> stripping of LLVM bitcode which (prior to this patchset) caused
> strip(1) to emit several warnings in the build log.
There is only .bca files in there, right?
> P.S.: I also think it might be worthwhile to only enable the KLEE
> package on x86_64 Linux <https://issues.guix.gnu.org/71925#18>. Let
> me know if I should add that as well :-)
Ah, yes, good catch.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:26:01 GMT)
Full text and
rfc822 format available.
Message #23 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Wrap klee-stats.
---
gnu/packages/check.scm | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 3bfcf5a37e..6b0ea0aaa8 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1070,7 +1070,15 @@ (define-public klee
(substitute* "CMakeLists.txt"
(("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
"${KLEE_UCLIBC_PATH}"))))
- (add-after 'install 'wrap-hooks
+ (add-after 'install 'wrap-klee-stats
+ (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+ (bin (string-append out "/bin")))
+ (wrap-program (string-append bin "/klee-stats")
+ `("GUIX_PYTHONPATH" ":" prefix
+ ,(search-path-as-string->list
+ (getenv "GUIX_PYTHONPATH")))))))
+ (add-after 'install 'wrap-klee
(lambda* (#:key inputs outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
(bin (string-append out "/bin"))
@@ -1088,7 +1096,7 @@ (define-public klee
(search-input-file %build-inputs "/lib/klee/libc.a"))
"-DENABLE_POSIX_RUNTIME=ON")))
(native-inputs (list clang-13 llvm-13 python-lit))
- (inputs (list bash-minimal klee-uclibc gperftools sqlite z3))
+ (inputs (list bash-minimal klee-uclibc gperftools sqlite z3 python python-tabulate))
(build-system cmake-build-system)
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
base-commit: c1d367f57e89c3f2efd964e6d638bd89b0b1df97
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:26:02 GMT)
Full text and
rfc822 format available.
Message #26 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Enable all tests.
[arguments]: Add phase to patch lit configuration, set #:test-target.
<#:configure-flags?>: Enable system and unit tests, configure gtest.
[inputs]: Add googletest and python-lit.
---
gnu/packages/check.scm | 30 +++++++++++++++++++++++++++---
1 file changed, 27 insertions(+), 3 deletions(-)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 6b0ea0aaa8..552cb39de5 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,13 +1063,23 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:test-target "check"
#:phases
#~(modify-phases %standard-phases
- (add-after 'unpack 'patch
+ (add-after 'unpack 'patch-cmake
(lambda _
(substitute* "CMakeLists.txt"
(("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
"${KLEE_UCLIBC_PATH}"))))
+ (add-after 'unpack 'patch-lit-config
+ (lambda _
+ ;; Make sure that we retain the value of the GUIX_PYTHONPATH
+ ;; environment variable in the test environmented created by
+ ;; python-lit. Otherwise, the test scripts won't be able to
+ ;; find the python-tabulate dependency, causing test failures.
+ (substitute* "test/lit.cfg"
+ (("addEnv\\('PWD'\\)" env)
+ (string-append env "\n" "addEnv('GUIX_PYTHONPATH')")))))
(add-after 'install 'wrap-klee-stats
(lambda* (#:key outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
@@ -1088,7 +1098,13 @@ (define-public klee
`("KLEE_RUNTIME_LIBRARY_PATH" =
(,(string-append lib "/klee/runtime/"))))))))
#:configure-flags
- #~(list (string-append "-DLLVMCC="
+ #~(list "-DENABLE_UNIT_TESTS=ON"
+ "-DENABLE_SYSTEM_TESTS=ON"
+ (string-append "-DGTEST_SRC_DIR="
+ (assoc-ref %build-inputs "googletest"))
+ (string-append "-DGTEST_INCLUDE_DIR="
+ (assoc-ref %build-inputs "googletest") "/googletest/include")
+ (string-append "-DLLVMCC="
(search-input-file %build-inputs "/bin/clang"))
(string-append "-DLLVMCXX="
(search-input-file %build-inputs "/bin/clang++"))
@@ -1096,7 +1112,15 @@ (define-public klee
(search-input-file %build-inputs "/lib/klee/libc.a"))
"-DENABLE_POSIX_RUNTIME=ON")))
(native-inputs (list clang-13 llvm-13 python-lit))
- (inputs (list bash-minimal klee-uclibc gperftools sqlite z3 python python-tabulate))
+ (inputs
+ `(("bash-minimal" ,bash-minimal)
+ ("klee-uclibc" ,klee-uclibc)
+ ("gperftools" ,gperftools)
+ ("sqlite" ,sqlite)
+ ("z3" ,z3)
+ ("python", python)
+ ("python-tabulate" ,python-tabulate)
+ ("googletest" ,(package-source googletest))))
(build-system cmake-build-system)
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:26:02 GMT)
Full text and
rfc822 format available.
Message #29 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
This avoid several warnings to be emitted by strip(1) when attempting
to strip .bca (i.e. LLVM bitcode) files shipped in /lib/klee/runtime/.
* gnu/packages/check.scm (klee): Set #:strip-directories.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 552cb39de5..3f3cc64e8e 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,6 +1063,7 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:strip-directories #~(list "bin") ;don't strip LLVM bitcode in /lib
#:test-target "check"
#:phases
#~(modify-phases %standard-phases
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:26:02 GMT)
Full text and
rfc822 format available.
Message #32 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Add supported-systems.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 3f3cc64e8e..3fc88b078f 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1123,6 +1123,7 @@ (define-public klee
("python-tabulate" ,python-tabulate)
("googletest" ,(package-source googletest))))
(build-system cmake-build-system)
+ (supported-systems '("x86_64-linux"))
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
(description "KLEE is a symbolic virtual machine built on top of the LLVM
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:26:03 GMT)
Full text and
rfc822 format available.
Message #35 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Add supported-systems.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 3fc88b078f..e48d392251 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1006,6 +1006,7 @@ (define-public klee-uclibc
(sha256
(base32 "12fnr5mq80cxwvv09gi844mi31jgi8067swagxnlxlhxj4mi125j"))))
(build-system gnu-build-system)
+ (supported-systems '("x86_64-linux"))
(arguments
`(#:tests? #f ;upstream uClibc tests do not work in the fork
#:strip-directories '() ;only ships a static library, so don't strip anything.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:30:02 GMT)
Full text and
rfc822 format available.
Message #38 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Hi Liliana,
Thanks for your feedback, I added a v2 to incorporate your feedback.
Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:
> Can we use a wrapper instead?
Sure, good idea! I modified the patch to wrap klee-stats instead.
Haven't done that so far so please check if I did it correctly :)
> > Lastly, I also passed a custom #:strip-directories argument to avoid
> > stripping of LLVM bitcode which (prior to this patchset) caused
> > strip(1) to emit several warnings in the build log.
>
> There is only .bca files in there, right?
Only .bca files and one shared library (libkleeRuntest.so) are in /lib.
Does stripping of shared libraries do anything? If so: Should we just
live with the warnings or should we strip that file individually?
> > P.S.: I also think it might be worthwhile to only enable the KLEE
> > package on x86_64 Linux <https://issues.guix.gnu.org/71925#18>. Let
> > me know if I should add that as well :-)
>
> Ah, yes, good catch.
I added two additional commits to set supported-systems for KLEE.
Best,
Sören
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:56:01 GMT)
Full text and
rfc822 format available.
Message #41 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Sonntag, dem 14.07.2024 um 20:25 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> * gnu/packages/check.scm (klee): Enable all tests.
> [arguments]: Add phase to patch lit configuration, set #:test-target.
> <#:configure-flags?>: Enable system and unit tests, configure gtest.
> [inputs]: Add googletest and python-lit.
> ---
> gnu/packages/check.scm | 30 +++++++++++++++++++++++++++---
> 1 file changed, 27 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
> index 6b0ea0aaa8..552cb39de5 100644
> --- a/gnu/packages/check.scm
> +++ b/gnu/packages/check.scm
> @@ -1063,13 +1063,23 @@ (define-public klee
> (base32
> "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
> (arguments
> (list
> + #:test-target "check"
> #:phases
> #~(modify-phases %standard-phases
> - (add-after 'unpack 'patch
> + (add-after 'unpack 'patch-cmake
> (lambda _
> (substitute* "CMakeLists.txt"
>
> (("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
> "${KLEE_UCLIBC_PATH}"))))
> + (add-after 'unpack 'patch-lit-config
> + (lambda _
> + ;; Make sure that we retain the value of
> the GUIX_PYTHONPATH
> + ;; environment variable in the test
> environmented created by
> + ;; python-lit. Otherwise, the test scripts
> won't be able to
> + ;; find the python-tabulate dependency,
> causing test failures.
> + (substitute* "test/lit.cfg"
> + (("addEnv\\('PWD'\\)" env)
> + (string-append env "\n"
> "addEnv('GUIX_PYTHONPATH')")))))
If this is a test, then we should use native-inputs below.
> (add-after 'install 'wrap-klee-stats
> (lambda* (#:key outputs #:allow-other-keys)
> (let* ((out (assoc-ref outputs "out"))
> @@ -1088,7 +1098,13 @@ (define-public klee
> `("KLEE_RUNTIME_LIBRARY_PATH" =
> (,(string-append lib
> "/klee/runtime/"))))))))
> #:configure-flags
> - #~(list (string-append "-DLLVMCC="
> + #~(list "-DENABLE_UNIT_TESTS=ON"
> + "-DENABLE_SYSTEM_TESTS=ON"
> + (string-append "-DGTEST_SRC_DIR="
> + (assoc-ref %build-inputs "googletest"))
> + (string-append "-DGTEST_INCLUDE_DIR="
> + (assoc-ref %build-inputs "googletest")
> "/googletest/include")
> + (string-append "-DLLVMCC="
> (search-input-file %build-inputs
> "/bin/clang"))
> (string-append "-DLLVMCXX="
> (search-input-file %build-inputs
> "/bin/clang++"))
> @@ -1096,7 +1112,15 @@ (define-public klee
> (search-input-file %build-inputs
> "/lib/klee/libc.a"))
> "-DENABLE_POSIX_RUNTIME=ON")))
> (native-inputs (list clang-13 llvm-13 python-lit))
> - (inputs (list bash-minimal klee-uclibc gperftools sqlite z3
> python python-tabulate))
> + (inputs
> + `(("bash-minimal" ,bash-minimal)
> + ("klee-uclibc" ,klee-uclibc)
> + ("gperftools" ,gperftools)
> + ("sqlite" ,sqlite)
> + ("z3" ,z3)
> + ("python", python)
> + ("python-tabulate" ,python-tabulate)
> + ("googletest" ,(package-source googletest))))
Why the package source and not the compiled package? Can we make it so
that we can use a prebuilt compiled one?
Cheers
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:56:02 GMT)
Full text and
rfc822 format available.
Message #44 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Sonntag, dem 14.07.2024 um 20:25 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> * gnu/packages/check.scm (klee): Add supported-systems.
> ---
> gnu/packages/check.scm | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
> index 3f3cc64e8e..3fc88b078f 100644
> --- a/gnu/packages/check.scm
> +++ b/gnu/packages/check.scm
> @@ -1123,6 +1123,7 @@ (define-public klee
> ("python-tabulate" ,python-tabulate)
> ("googletest" ,(package-source googletest))))
> (build-system cmake-build-system)
> + (supported-systems '("x86_64-linux"))
> (home-page "https://klee-se.org/")
> (synopsis "Symbolic execution engine")
> (description "KLEE is a symbolic virtual machine built on top of
> the LLVM
LGTM
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 18:57:01 GMT)
Full text and
rfc822 format available.
Message #47 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Sonntag, dem 14.07.2024 um 20:25 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> * gnu/packages/check.scm (klee): Add supported-systems.
Should be klee-uclibc.
> ---
> gnu/packages/check.scm | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
> index 3fc88b078f..e48d392251 100644
> --- a/gnu/packages/check.scm
> +++ b/gnu/packages/check.scm
> @@ -1006,6 +1006,7 @@ (define-public klee-uclibc
> (sha256
> (base32
> "12fnr5mq80cxwvv09gi844mi31jgi8067swagxnlxlhxj4mi125j"))))
> (build-system gnu-build-system)
> + (supported-systems '("x86_64-linux"))
> (arguments
> `(#:tests? #f ;upstream uClibc tests do not work in the fork
> #:strip-directories '() ;only ships a static library, so
> don't strip anything.
Cheers
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sun, 14 Jul 2024 20:05:01 GMT)
Full text and
rfc822 format available.
Message #50 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Hello Liliana,
Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:
> Why the package source and not the compiled package? Can we make it so
> that we can use a prebuilt compiled one?
Unfortunately, this does not seem to be possible. You can only point the
KLEE build system to a googletest source [1]. This seems to be related
to some peculiarity of googletest as other Guix package do the same
thing [2] [3] [4] (there are more grep for "package-source googletest").
> If this is a test, then we should use native-inputs below.
What exact dependency are you referring to? python-lit is already
declared as a dependency through native-inputs.
> Should be klee-uclibc.
Good catch! I can send a revision fixing the commit message.
Is there anything else I should adjust in a v3 revision?
Cheers,
Sören
[1]: https://github.com/klee/klee/blob/master/README-CMake.md?plain=1#L60-L62
[2]: https://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/crypto.scm?id=eb508e32d2d359c94d2cabebfe90dc32ca5dcf4f#n336
[3]: https://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/benchmark.scm?id=eb508e32d2d359c94d2cabebfe90dc32ca5dcf4f#n241
[4]: https://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/chemistry.scm?id=eb508e32d2d359c94d2cabebfe90dc32ca5dcf4f#n407
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sat, 20 Jul 2024 09:58:02 GMT)
Full text and
rfc822 format available.
Message #53 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Hello Sören,
soeren <at> soeren-tempel.net skribis:
> (,(string-append lib "/klee/runtime/"))))))))
> #:configure-flags
> - #~(list (string-append "-DLLVMCC="
> + #~(list "-DENABLE_UNIT_TESTS=ON"
> + "-DENABLE_SYSTEM_TESTS=ON"
> + (string-append "-DGTEST_SRC_DIR="
> + (assoc-ref %build-inputs "googletest"))
> + (string-append "-DGTEST_INCLUDE_DIR="
> + (assoc-ref %build-inputs "googletest") "/googletest/include")
I would replace (assoc-ref …) by:
#+(package-source googletest)
> - (inputs (list bash-minimal klee-uclibc gperftools sqlite z3 python python-tabulate))
> + (inputs
> + `(("bash-minimal" ,bash-minimal)
> + ("klee-uclibc" ,klee-uclibc)
> + ("gperftools" ,gperftools)
> + ("sqlite" ,sqlite)
> + ("z3" ,z3)
> + ("python", python)
> + ("python-tabulate" ,python-tabulate)
> + ("googletest" ,(package-source googletest))))
… and leave ‘googletest’ out of ‘inputs’ entirely, keeping the concise
input list without labels. (I think we shouldn’t reintroduce input
labels; the goal has always been to remove them.)
WDYT?
Besides, to answer Liliana, Googletest is often used as a “source
library” like you’re doing here; there are quite a few other packages
that do that in Guix already, as you explained. That’s OK.
Thanks,
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Thu, 25 Jul 2024 20:41:01 GMT)
Full text and
rfc822 format available.
Message #56 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Wrap klee-stats.
---
gnu/packages/check.scm | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 412b94569c..3c5ed7a3c3 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1070,7 +1070,15 @@ (define-public klee
(substitute* "CMakeLists.txt"
(("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
"${KLEE_UCLIBC_PATH}"))))
- (add-after 'install 'wrap-hooks
+ (add-after 'install 'wrap-klee-stats
+ (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+ (bin (string-append out "/bin")))
+ (wrap-program (string-append bin "/klee-stats")
+ `("GUIX_PYTHONPATH" ":" prefix
+ ,(search-path-as-string->list
+ (getenv "GUIX_PYTHONPATH")))))))
+ (add-after 'install 'wrap-klee
(lambda* (#:key inputs outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
(bin (string-append out "/bin"))
@@ -1088,7 +1096,7 @@ (define-public klee
(search-input-file %build-inputs "/lib/klee/libc.a"))
"-DENABLE_POSIX_RUNTIME=ON")))
(native-inputs (list clang-13 llvm-13 python-lit))
- (inputs (list bash-minimal klee-uclibc gperftools sqlite z3))
+ (inputs (list bash-minimal klee-uclibc gperftools sqlite z3 python python-tabulate))
(build-system cmake-build-system)
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
base-commit: 03062c7a9fd74d625639e1a325e9cb58d1cd74e3
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Thu, 25 Jul 2024 20:41:02 GMT)
Full text and
rfc822 format available.
Message #59 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Enable all tests.
[arguments]: Add phase to patch lit configuration, set #:test-target.
<#:configure-flags?>: Enable system and unit tests, configure gtest.
[inputs]: Add googletest and python-lit.
---
gnu/packages/check.scm | 20 ++++++++++++++++++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 3c5ed7a3c3..48e8ae3aef 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,13 +1063,23 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:test-target "check"
#:phases
#~(modify-phases %standard-phases
- (add-after 'unpack 'patch
+ (add-after 'unpack 'patch-cmake
(lambda _
(substitute* "CMakeLists.txt"
(("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
"${KLEE_UCLIBC_PATH}"))))
+ (add-after 'unpack 'patch-lit-config
+ (lambda _
+ ;; Make sure that we retain the value of the GUIX_PYTHONPATH
+ ;; environment variable in the test environmented created by
+ ;; python-lit. Otherwise, the test scripts won't be able to
+ ;; find the python-tabulate dependency, causing test failures.
+ (substitute* "test/lit.cfg"
+ (("addEnv\\('PWD'\\)" env)
+ (string-append env "\n" "addEnv('GUIX_PYTHONPATH')")))))
(add-after 'install 'wrap-klee-stats
(lambda* (#:key outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
@@ -1088,7 +1098,13 @@ (define-public klee
`("KLEE_RUNTIME_LIBRARY_PATH" =
(,(string-append lib "/klee/runtime/"))))))))
#:configure-flags
- #~(list (string-append "-DLLVMCC="
+ #~(list "-DENABLE_UNIT_TESTS=ON"
+ "-DENABLE_SYSTEM_TESTS=ON"
+ (string-append "-DGTEST_SRC_DIR="
+ #+(package-source googletest))
+ (string-append "-DGTEST_INCLUDE_DIR="
+ #+(package-source googletest) "/googletest/include")
+ (string-append "-DLLVMCC="
(search-input-file %build-inputs "/bin/clang"))
(string-append "-DLLVMCXX="
(search-input-file %build-inputs "/bin/clang++"))
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Thu, 25 Jul 2024 20:41:02 GMT)
Full text and
rfc822 format available.
Message #62 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
This avoid several warnings to be emitted by strip(1) when attempting
to strip .bca (i.e. LLVM bitcode) files shipped in /lib/klee/runtime/.
* gnu/packages/check.scm (klee): Set #:strip-directories.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 48e8ae3aef..1b7381cd36 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,6 +1063,7 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:strip-directories #~(list "bin") ;don't strip LLVM bitcode in /lib
#:test-target "check"
#:phases
#~(modify-phases %standard-phases
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Thu, 25 Jul 2024 20:41:03 GMT)
Full text and
rfc822 format available.
Message #65 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Add supported-systems.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 1b7381cd36..75393ca179 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1115,6 +1115,7 @@ (define-public klee
(native-inputs (list clang-13 llvm-13 python-lit))
(inputs (list bash-minimal klee-uclibc gperftools sqlite z3 python python-tabulate))
(build-system cmake-build-system)
+ (supported-systems '("x86_64-linux"))
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
(description "KLEE is a symbolic virtual machine built on top of the LLVM
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Thu, 25 Jul 2024 20:41:03 GMT)
Full text and
rfc822 format available.
Message #68 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee-uclibc): Add supported-systems.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 75393ca179..493829082b 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1006,6 +1006,7 @@ (define-public klee-uclibc
(sha256
(base32 "12fnr5mq80cxwvv09gi844mi31jgi8067swagxnlxlhxj4mi125j"))))
(build-system gnu-build-system)
+ (supported-systems '("x86_64-linux"))
(arguments
`(#:tests? #f ;upstream uClibc tests do not work in the fork
#:strip-directories '() ;only ships a static library, so don't strip anything.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Thu, 25 Jul 2024 20:44:02 GMT)
Full text and
rfc822 format available.
Message #71 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Ludovic Courtès <ludo <at> gnu.org> wrote:
> Hello Sören,
Hi Ludo,
> … and leave ‘googletest’ out of ‘inputs’ entirely, keeping the concise
> input list without labels. (I think we shouldn’t reintroduce input
> labels; the goal has always been to remove them.)
>
> WDYT?
Thanks for this suggestion, this sounds very good to me!
I just send a v3 which implements this and also fixes the commit message
typo that Liliana pointed out. Let me know if there is anything else
that needs to be done :)
Best,
Sören
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Fri, 26 Jul 2024 18:39:02 GMT)
Full text and
rfc822 format available.
Message #74 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Donnerstag, dem 25.07.2024 um 22:39 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> * gnu/packages/check.scm (klee): Wrap klee-stats.
> ---
> gnu/packages/check.scm | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
> index 412b94569c..3c5ed7a3c3 100644
> --- a/gnu/packages/check.scm
> +++ b/gnu/packages/check.scm
> @@ -1070,7 +1070,15 @@ (define-public klee
> (substitute* "CMakeLists.txt"
>
> (("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
> "${KLEE_UCLIBC_PATH}"))))
> - (add-after 'install 'wrap-hooks
> + (add-after 'install 'wrap-klee-stats
> + (lambda* (#:key outputs #:allow-other-keys)
> + (let* ((out (assoc-ref outputs "out"))
> + (bin (string-append out "/bin")))
> + (wrap-program (string-append bin "/klee-
> stats")
> + `("GUIX_PYTHONPATH" ":" prefix
> + ,(search-path-as-string->list
> + (getenv "GUIX_PYTHONPATH")))))))
> + (add-after 'install 'wrap-klee
> (lambda* (#:key inputs outputs #:allow-
> other-keys)
> (let* ((out (assoc-ref outputs "out"))
> (bin (string-append out "/bin"))
Why not consolidate these into a single 'wrap phase?
Cheers
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Fri, 26 Jul 2024 18:40:02 GMT)
Full text and
rfc822 format available.
Message #77 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Donnerstag, dem 25.07.2024 um 22:40 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> This avoid several warnings to be emitted by strip(1) when attempting
> to strip .bca (i.e. LLVM bitcode) files shipped in
> /lib/klee/runtime/.
>
> * gnu/packages/check.scm (klee): Set #:strip-directories.
> ---
LGTM
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Fri, 26 Jul 2024 18:43:02 GMT)
Full text and
rfc822 format available.
Message #80 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Am Donnerstag, dem 25.07.2024 um 22:40 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> * gnu/packages/check.scm (klee): Add supported-systems.
> ---
To recall, i386 does not work?
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sat, 27 Jul 2024 09:12:02 GMT)
Full text and
rfc822 format available.
Message #83 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Wrap klee-stats.
---
gnu/packages/check.scm | 11 ++++++++---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 412b94569c..347bd2e1aa 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1070,12 +1070,17 @@ (define-public klee
(substitute* "CMakeLists.txt"
(("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
"${KLEE_UCLIBC_PATH}"))))
- (add-after 'install 'wrap-hooks
+ (add-after 'install 'wrap-programs
(lambda* (#:key inputs outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
(bin (string-append out "/bin"))
(lib (string-append out "/lib")))
- ;; Ensure that KLEE finds runtime libraries (e.g. uclibc).
+ ;; Ensure that klee-stats finds its Python dependencies.
+ (wrap-program (string-append bin "/klee-stats")
+ `("GUIX_PYTHONPATH" ":" prefix
+ ,(search-path-as-string->list
+ (getenv "GUIX_PYTHONPATH"))))
+ ;; Ensure that klee finds runtime libraries (e.g. uclibc).
(wrap-program (string-append bin "/klee")
`("KLEE_RUNTIME_LIBRARY_PATH" =
(,(string-append lib "/klee/runtime/"))))))))
@@ -1088,7 +1093,7 @@ (define-public klee
(search-input-file %build-inputs "/lib/klee/libc.a"))
"-DENABLE_POSIX_RUNTIME=ON")))
(native-inputs (list clang-13 llvm-13 python-lit))
- (inputs (list bash-minimal klee-uclibc gperftools sqlite z3))
+ (inputs (list bash-minimal klee-uclibc gperftools sqlite z3 python python-tabulate))
(build-system cmake-build-system)
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
base-commit: c6ff1d6ff761af0bd9bac5403fd834f04a14a192
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sat, 27 Jul 2024 09:12:02 GMT)
Full text and
rfc822 format available.
Message #86 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Enable all tests.
[arguments]: Patch lit configuration and set #:test-target.
<#:configure-flags?>: Enable system and unit tests, configure gtest.
---
gnu/packages/check.scm | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 347bd2e1aa..aeaa2e807f 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,13 +1063,22 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:test-target "check"
#:phases
#~(modify-phases %standard-phases
(add-after 'unpack 'patch
(lambda _
+ ;; Allow specification of an absolute full path to uclibc.
(substitute* "CMakeLists.txt"
(("\\$\\{KLEE_UCLIBC_PATH\\}/lib/libc\\.a")
- "${KLEE_UCLIBC_PATH}"))))
+ "${KLEE_UCLIBC_PATH}"))
+ ;; Make sure that we retain the value of the GUIX_PYTHONPATH
+ ;; environment variable in the test environmented created by
+ ;; python-lit. Otherwise, the test scripts won't be able to
+ ;; find the python-tabulate dependency, causing test failures.
+ (substitute* "test/lit.cfg"
+ (("addEnv\\('PWD'\\)" env)
+ (string-append env "\n" "addEnv('GUIX_PYTHONPATH')")))))
(add-after 'install 'wrap-programs
(lambda* (#:key inputs outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
@@ -1085,7 +1094,13 @@ (define-public klee
`("KLEE_RUNTIME_LIBRARY_PATH" =
(,(string-append lib "/klee/runtime/"))))))))
#:configure-flags
- #~(list (string-append "-DLLVMCC="
+ #~(list "-DENABLE_UNIT_TESTS=ON"
+ "-DENABLE_SYSTEM_TESTS=ON"
+ (string-append "-DGTEST_SRC_DIR="
+ #+(package-source googletest))
+ (string-append "-DGTEST_INCLUDE_DIR="
+ #+(package-source googletest) "/googletest/include")
+ (string-append "-DLLVMCC="
(search-input-file %build-inputs "/bin/clang"))
(string-append "-DLLVMCXX="
(search-input-file %build-inputs "/bin/clang++"))
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sat, 27 Jul 2024 09:12:03 GMT)
Full text and
rfc822 format available.
Message #89 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
This avoid several warnings to be emitted by strip(1) when attempting
to strip .bca (i.e. LLVM bitcode) files shipped in /lib/klee/runtime/.
* gnu/packages/check.scm (klee): Set #:strip-directories.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index aeaa2e807f..250206886f 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1063,6 +1063,7 @@ (define-public klee
(base32 "1nma6dqi8chjb97llsa8mzyskgsg4dx56lm8j514j5wmr8vkafz6"))))
(arguments
(list
+ #:strip-directories #~(list "bin") ;don't strip LLVM bitcode in /lib
#:test-target "check"
#:phases
#~(modify-phases %standard-phases
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sat, 27 Jul 2024 09:12:03 GMT)
Full text and
rfc822 format available.
Message #92 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee): Add supported-systems.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 250206886f..ae3ec90944 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1111,6 +1111,7 @@ (define-public klee
(native-inputs (list clang-13 llvm-13 python-lit))
(inputs (list bash-minimal klee-uclibc gperftools sqlite z3 python python-tabulate))
(build-system cmake-build-system)
+ (supported-systems '("x86_64-linux"))
(home-page "https://klee-se.org/")
(synopsis "Symbolic execution engine")
(description "KLEE is a symbolic virtual machine built on top of the LLVM
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sat, 27 Jul 2024 09:12:04 GMT)
Full text and
rfc822 format available.
Message #95 received at 72106 <at> debbugs.gnu.org (full text, mbox):
From: Sören Tempel <soeren <at> soeren-tempel.net>
* gnu/packages/check.scm (klee-uclibc): Add supported-systems.
---
gnu/packages/check.scm | 1 +
1 file changed, 1 insertion(+)
diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index ae3ec90944..b8fbe3f6a8 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -1006,6 +1006,7 @@ (define-public klee-uclibc
(sha256
(base32 "12fnr5mq80cxwvv09gi844mi31jgi8067swagxnlxlhxj4mi125j"))))
(build-system gnu-build-system)
+ (supported-systems '("x86_64-linux"))
(arguments
`(#:tests? #f ;upstream uClibc tests do not work in the fork
#:strip-directories '() ;only ships a static library, so don't strip anything.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#72106
; Package
guix-patches
.
(Sat, 27 Jul 2024 09:15:02 GMT)
Full text and
rfc822 format available.
Message #98 received at 72106 <at> debbugs.gnu.org (full text, mbox):
Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:
> To recall, i386 does not work?
Yes, i386 does not work as KLEE assumes a 64-bit pointer representation.
Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:
> Why not consolidate these into a single 'wrap phase?
Good idea, I consolidated everything into a single 'wrap phase.
Additionally, I also consolidated the patching into a single
'patch phase.
Best,
Sören
Reply sent
to
Liliana Marie Prikler <liliana.prikler <at> gmail.com>
:
You have taken responsibility.
(Sat, 27 Jul 2024 17:30:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
soeren <at> soeren-tempel.net
:
bug acknowledged by developer.
(Sat, 27 Jul 2024 17:30:02 GMT)
Full text and
rfc822 format available.
Message #103 received at 72106-done <at> debbugs.gnu.org (full text, mbox):
Am Samstag, dem 27.07.2024 um 11:10 +0200 schrieb
soeren <at> soeren-tempel.net:
> From: Sören Tempel <soeren <at> soeren-tempel.net>
>
> * gnu/packages/check.scm (klee-uclibc): Add supported-systems.
> ---
Pushed as 46a64c7fdd057283063aae6df058579bb07c4b6a.
Cheers
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Sun, 25 Aug 2024 11:24:05 GMT)
Full text and
rfc822 format available.
This bug report was last modified 197 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.