Vagrant Cascadian wrote 2 weeks ago
(address . guix-patches@gnu.org)
87jz9c73wo.fsf@wireframe
Reprotest fails to run correctly due to files that are not marked
executable:
reprotest --verbose --vary=-fileordering,-domain_host,-user_group,-time -c 'date > date' . date -- null
WARNING:reprotest:The control build runs on 1 CPU by default, give --min-cpus to increase this.
WARNING:reprotest:diffoscope not available, falling back to regular diff
INFO:reprotest:STARTING VIRTUAL SERVER ['/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/virt/autopkgtest-virt-null']
Traceback (most recent call last):
File "/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/__init__.py", line 862, in run
return 0 if check_func(*check_args) else 1
File "/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/__init__.py", line 370, in check
proc = test_args._replace(result_dir=result_dir).corun_builds(testbed_args)
File "/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/__init__.py", line 114, in start
next(cr)
File "/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/__init__.py", line 322, in corun_builds
with start_testbed(virtual_server_args, temp_dir, no_clean_on_error,
File "/gnu/store/50047r6f9chb6dwd01f9hai90y9arwb4-python-3.10.7/lib/python3.10/contextlib.py", line 135, in __enter__
return next(self.gen)
File "/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/__init__.py", line 85, in start_testbed
testbed.start()
File "/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/lib/adt_testbed.py", line 134, in start
self.sp = subprocess.Popen(self.vserver_argv,
File "/gnu/store/50047r6f9chb6dwd01f9hai90y9arwb4-python-3.10.7/lib/python3.10/subprocess.py", line 969, in __init__
self._execute_child(args, executable, preexec_fn, close_fds,
File "/gnu/store/50047r6f9chb6dwd01f9hai90y9arwb4-python-3.10.7/lib/python3.10/subprocess.py", line 1845, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
PermissionError: [Errno 13] Permission denied: '/gnu/store/7cglj9r39y4hfbwzd3rbh2rgl1yg343v-reprotest-0.7.28/lib/python3.10/site-packages/reprotest/virt/autopkgtest-virt-null'
The reprotest version 0.7.21 included in guix v1.4.0 did have these
files marked executable, but for some reason in recent versions (maybe
changes in pyproject-build-system?), they are shipped without the
execute bit set. I have not git bisected between all the versions of
guix to see exactly where it starts failing.
CCed members of the python team if they have ideas about changes in the
build system which might cause the issue.
The attached patch marks the necessary files are executable with an
added phase. Open to refactorring the patch, but my guile skills are
quite limited and this works!
live well,
vagrant
From 5ed6bdda83e9e06f5984faac3d347b91438e9b04 Mon Sep 17 00:00:00 2001
From: Vagrant Cascadian <vagrant@reproducible-builds.org>
Date: Wed, 26 Feb 2025 11:49:48 -0800
Subject: [PATCH] gnu: reprotest: Add phase fixing executable bit on virt
files.
* gnu/packages/diffoscope.scm (reprotest): Add 'make-virt-files-executable
phase.
---
gnu/packages/diffoscope.scm | 7 +++++++
1 file changed, 7 insertions(+)
Toggle diff (22 lines)
diff --git a/gnu/packages/diffoscope.scm b/gnu/packages/diffoscope.scm
index d0e911363d..90be6e68ec 100644
--- a/gnu/packages/diffoscope.scm
+++ b/gnu/packages/diffoscope.scm
@@ -273,6 +273,13 @@ (define-public reprotest
#:tests? #f
#:phases
#~(modify-phases %standard-phases
+ (add-after 'compress-documentation 'make-virt-files-executable
+ ;; The autopkgtest-virt- files need to be marked executable for
+ ;; reprotest to function correctly.
+ (lambda _
+ (for-each (lambda (file)
+ (invoke "chmod" "+x" file))
+ (find-files #$output "autopkgtest-virt-.*"))))
(add-after 'install 'install-doc
(lambda _
(let* ((mandir1 (string-append
base-commit: 54ff5d33739ae95c19f4ed5bea38d2613f6f7d3c
--
2.39.5
-----BEGIN PGP SIGNATURE-----
iHUEARYKAB0WIQRlgHNhO/zFx+LkXUXcUY/If5cWqgUCZ7+ReAAKCRDcUY/If5cW
qn3WAP9KkbpEZlSTUpQw5apcFCtQkhhj7GS/BGqfqw4xZWguowEA+9TPtpJ0Gqxi
5Z00KPmo3Nwqx9d6iaaviYHT4p5utQg=
=h8ZM
-----END PGP SIGNATURE-----