all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
blob f5fbb0a1111dfedf463204e785d62c3b604f9711 4417 bytes (raw)
name: gnu/packages/patches/flashrom-fix-building-on-aarch64.patch 	 # note: path name is non-authoritative(*)

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
 
commit da6b3b70cb852dd8e9f9e21aef95fa83e7f7ab0d
Author: Pyry Kontio <pyry.kontio@drasa.eu>
Date:   Mon Jul 6 12:57:35 2020 +0900

    Makefile: Fix building on AArch64 NixOS
    
    The parsing of the output of archtest.c produced an unexpected
    value on AArch64 NixOS. For example, the make variable ARCH was set to:
    
    ```
    bit outside of fd_set selected
    arm
    ```
    
    This made the arch and OS checks fail.
    
    This commit simplifies the parsing, making it more robust.
    
    The C files archtest.c, endiantest.c and os.h used to set the
    TARGET_OS, ARCH and ENDIAN variables, respectively, output
    the result of the test as the final line, so just extracting
    the final line and removing double quoting is enough.
    
    This commit also fixes a bug with debug_shell lacking escaping
    single quotes, which prevented using the single quote in the
    debug_shell calls. It used to work by accident before this fix;
    the line in the call happened to contain a balanced pair of double
    quotes and lacked other characters that needed escaping, which
    didn't break the debug_shell, but this was accidental and very
    brittle.
    
    Signed-off-by: Pyry Kontio <pyry.kontio@drasa.eu>
    Change-Id: Iaa4477a71e758cf9ecad2c22f3b77bc6508a3510
    Reviewed-on: https://review.coreboot.org/c/flashrom/+/43140
    Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
    Reviewed-by: Angel Pons <th3fanbus@gmail.com>

diff --git a/Makefile b/Makefile
index f3f7717e..e475cbdb 100644
--- a/Makefile
+++ b/Makefile
@@ -83,7 +83,8 @@ dummy_for_make_3_80:=$(shell printf "Build started on %s\n\n" "$$(date)" >$(BUIL
 
 # Provide an easy way to execute a command, print its output to stdout and capture any error message on stderr
 # in the build details file together with the original stdout output.
-debug_shell = $(shell export LC_ALL=C ; { echo 'exec: export LC_ALL=C ; { $(1) ; }' >&2;  { $(1) ; } | tee -a $(BUILD_DETAILS_FILE) ; echo >&2 ; } 2>>$(BUILD_DETAILS_FILE))
+debug_shell = $(shell export LC_ALL=C ; { echo 'exec: export LC_ALL=C ; { $(subst ','\'',$(1)) ; }' >&2; \
+    { $(1) ; } | tee -a $(BUILD_DETAILS_FILE) ; echo >&2 ; } 2>>$(BUILD_DETAILS_FILE))
 
 ###############################################################################
 # General OS-specific settings.
@@ -106,7 +107,8 @@ endif
 # IMPORTANT: The following line must be placed before TARGET_OS is ever used
 # (of course), but should come after any lines setting CC because the line
 # below uses CC itself.
-override TARGET_OS := $(strip $(call debug_shell,$(CC) $(CPPFLAGS) -E os.h 2>/dev/null | grep -v '^\#' | grep '"' | cut -f 2 -d'"'))
+override TARGET_OS := $(strip $(call debug_shell,$(CC) $(CPPFLAGS) -E os.h 2>/dev/null \
+    | tail -1 | cut -f 2 -d'"'))
 
 ifeq ($(TARGET_OS), Darwin)
 override CPPFLAGS += -I/opt/local/include -I/usr/local/include
@@ -490,8 +492,10 @@ endif
 # IMPORTANT: The following line must be placed before ARCH is ever used
 # (of course), but should come after any lines setting CC because the line
 # below uses CC itself.
-override ARCH := $(strip $(call debug_shell,$(CC) $(CPPFLAGS) -E archtest.c 2>/dev/null | grep -v '^\#' | grep '"' | cut -f 2 -d'"'))
-override ENDIAN := $(strip $(call debug_shell,$(CC) $(CPPFLAGS) -E endiantest.c 2>/dev/null | grep -v '^\#'))
+override ARCH := $(strip $(call debug_shell,$(CC) $(CPPFLAGS) -E archtest.c 2>/dev/null \
+    | tail -1 | cut -f 2 -d'"'))
+override ENDIAN := $(strip $(call debug_shell,$(CC) $(CPPFLAGS) -E endiantest.c 2>/dev/null \
+    | tail -1))
 
 # Disable the internal programmer on unsupported architectures (everything but x86 and mipsel)
 ifneq ($(ARCH)-little, $(filter $(ARCH),x86 mips)-$(ENDIAN))
@@ -1299,12 +1303,12 @@ compiler: featuresavailable
 	@printf "Target arch is "
 	@# FreeBSD wc will output extraneous whitespace.
 	@echo $(ARCH)|wc -w|grep -q '^[[:blank:]]*1[[:blank:]]*$$' ||	\
-		( echo "unknown. Aborting."; exit 1)
+		( echo "unknown (\"$(ARCH)\"). Aborting."; exit 1)
 	@printf "%s\n" '$(ARCH)'
 	@printf "Target OS is "
 	@# FreeBSD wc will output extraneous whitespace.
 	@echo $(TARGET_OS)|wc -w|grep -q '^[[:blank:]]*1[[:blank:]]*$$' ||	\
-		( echo "unknown. Aborting."; exit 1)
+		( echo "unknown (\"$(TARGET_OS)\"). Aborting."; exit 1)
 	@printf "%s\n" '$(TARGET_OS)'
 ifeq ($(TARGET_OS), libpayload)
 	@$(CC) --version 2>&1 | grep -q coreboot || \

debug log:

solving f5fbb0a111 ...
found f5fbb0a111 in https://yhetil.org/guix/20221217181205.23795-1-simon@simonsouth.net/

applying [1/1] https://yhetil.org/guix/20221217181205.23795-1-simon@simonsouth.net/
diff --git a/gnu/packages/patches/flashrom-fix-building-on-aarch64.patch b/gnu/packages/patches/flashrom-fix-building-on-aarch64.patch
new file mode 100644
index 0000000000..f5fbb0a111

1:12: trailing whitespace.
    
1:15: trailing whitespace.
    
1:20: trailing whitespace.
    
1:22: trailing whitespace.
    
1:24: trailing whitespace.
    
Checking patch gnu/packages/patches/flashrom-fix-building-on-aarch64.patch...
Applied patch gnu/packages/patches/flashrom-fix-building-on-aarch64.patch cleanly.
warning: squelched 15 whitespace errors
warning: 20 lines add whitespace errors.

index at:
100644 f5fbb0a1111dfedf463204e785d62c3b604f9711	gnu/packages/patches/flashrom-fix-building-on-aarch64.patch

(*) Git path names are given by the tree(s) the blob belongs to.
    Blobs themselves have no identifier aside from the hash of its contents.^

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.