guix: Make V=1 more powerful for debugging #18958

dongcarl commented at 2:09 PM on May 12, 2020: member

- Print commands in both unexpanded and expanded forms
- Set VERBOSE=1 for CMake

Ping MarcoFalke hopefully you use V=1 already for the Guix builds on DrahtBot?

guix: Make V=1 more powerful for debugging

- Print commands in both unexpanded and expanded forms
- Set VERBOSE=1 for CMake

85f4a4b082

dongcarl added the label Build system on May 12, 2020

dongcarl added the label Needs Guix build on May 12, 2020

MarcoFalke commented at 2:25 PM on May 12, 2020: member

Thanks! Concept A⃣ C⃣ K⃣

DrahtBot commented at 3:51 PM on May 13, 2020: member

Guix builds

File	commit 8da1e43b63cb36759eeb1fcfd6768163265c44e2<br>(master)	commit af77fea6d47db3221e07650e9e6dce713e5285d7<br>(master and this pull)
*-aarch64-linux-gnu-debug.tar.gz	`f511c202f481e4ad...`	`13552fd073cef426...`
*-aarch64-linux-gnu.tar.gz	`00972e52659dc1cd...`	`7fc18b2fac318e0c...`
*-arm-linux-gnueabihf-debug.tar.gz	`017b9932bfb2133a...`	`126687a5aeedd41e...`
*-arm-linux-gnueabihf.tar.gz	`bb5cf0cb93f33a67...`	`06ed461e9ad9b778...`
*-riscv64-linux-gnu-debug.tar.gz	`11dc54be08c74885...`	`85de6e45432819b6...`
*-riscv64-linux-gnu.tar.gz	`e29ced14e1e25fd3...`	`8c4d3c795ffb839b...`
*-win-unsigned.tar.gz	`52f74fc1807f7eec...`	`17d24ed1bf00b848...`
*-win64-debug.zip	`25cec7e5ea0258e9...`	`bf41f92e2979f12f...`
*-win64-setup-unsigned.exe	`84699a3c542f5032...`	`19f9a353f64e9d36...`
*-win64.zip	`5415d253d272a78e...`	`812b04ffa3bfd94a...`
*-x86_64-linux-gnu-debug.tar.gz	`6cd89e7adbdd879f...`	`7d6057e4365c5fbd...`
*-x86_64-linux-gnu.tar.gz	`6056637d60b40a8c...`	`9fe54f8bb8ed1e7d...`
*.tar.gz	`318a4ac37bfef0ad...`	`6937016b9d1e5e43...`
guix_build.log	`7e594c7df397762e...`	`dc98a200ec14a192...`
guix_build.log.diff		`3a8389bc9941163b...`

DrahtBot removed the label Needs Guix build on May 13, 2020

MarcoFalke added the label Needs Guix build on May 13, 2020

in contrib/guix/libexec/build.sh:6 in 85f4a4b082 outdated

   2 | @@ -3,6 +3,14 @@ export LC_ALL=C
   3 |  set -e -o pipefail
   4 |  export TZ=UTC
   5 |  
   6 | +if [ -n "$V" ]; then

laanwj commented at 5:35 PM on May 14, 2020:

No idea if it's a problem, but won't this detect V=0 as well?

dongcarl commented at 4:40 PM on May 20, 2020:

Right yes, I just wanted to conform with the README which says that we will do thing if "$V" is non-empty. Perhaps I can note in the README that we don't try to interpret the value of "$V" at all.

DrahtBot commented at 7:51 AM on May 16, 2020: member

Guix builds

File	commit e2f6866cca3e0c3c74f94f2df50ed3243158ea32<br>(master)	commit 1e80867eb5c56d2d7dd04568915c1ec759f40717<br>(master and this pull)
*-aarch64-linux-gnu-debug.tar.gz	`b9317425f5ae0772...`	`c16ce9c7d0bfa5ea...`
*-aarch64-linux-gnu.tar.gz	`8547167d635a159b...`	`b9eec63ca8d1f47d...`
*-arm-linux-gnueabihf-debug.tar.gz	`a7442351bad62f78...`	`0f3c69f531b6534e...`
*-arm-linux-gnueabihf.tar.gz	`20084ed9d25bbb7c...`	`87788269d8565a1c...`
*-riscv64-linux-gnu-debug.tar.gz	`559e9188e5298f47...`	`41dbc5e650d4b67e...`
*-riscv64-linux-gnu.tar.gz	`37f7a3b7985894ee...`	`14f3633e3dbf81e5...`
*-win-unsigned.tar.gz	`3704d8e6a872997b...`	`ede2eef05c0c6aa8...`
*-win64-debug.zip	`97843e42b686ff12...`	`3a2eaf65f0cd742e...`
*-win64-setup-unsigned.exe	`63ab03762282c5e7...`	`1594e2f6fb158320...`
*-win64.zip	`dc80c6a8ff8a9987...`	`1cacb740eec6d288...`
*-x86_64-linux-gnu-debug.tar.gz	`2b929524c920119e...`	`802d28cf0b5faa7a...`
*-x86_64-linux-gnu.tar.gz	`9e3d98012136c821...`	`fc20cf2702a53c2f...`
*.tar.gz	`90f83c125c189853...`	`361800d4fafd0877...`
guix_build.log	`7948626ab6c30044...`	`97bd407e9324888b...`
guix_build.log.diff		`728b4d573dc076d2...`

DrahtBot removed the label Needs Guix build on May 16, 2020

MarcoFalke added this to the milestone 0.21.0 on May 16, 2020

MarcoFalke commented at 4:46 PM on May 20, 2020: member

𝒜𝒞𝒦 82f4b5a98e2ab213f02773eed17019df23f731cd

MarcoFalke requested review from fanquake on May 20, 2020

guix: Add clarifying documentation for V env var f852761aec

in contrib/guix/README.md:144 in 82f4b5a98e outdated

 140 | @@ -141,6 +141,11 @@ find output/ -type f -print0 | sort -z | xargs -r0 sha256sum
 141 |  
 142 |    If non-empty, will pass `V=1` to all `make` invocations, making `make` output
 143 |    verbose.
 144 | +

MarcoFalke commented at 4:47 PM on May 20, 2020:

dongcarl force-pushed on May 20, 2020

fanquake approved

fanquake commented at 5:13 AM on May 21, 2020: member

ACK f852761aec81ed23c7b9e4546c08d1ef303f2507. Ran a Windows Guix build and compared the output from master and this PR when using V=1. i.e HOSTS=x86_64-w64-mingw32 PATH="/root/.config/guix/current/bin${PATH:+:}$PATH" V=1 ./contrib/guix/guix-build.sh.

fanquake merged this on May 21, 2020

fanquake closed this on May 21, 2020

sidhujag referenced this in commit 3512bedeed on May 21, 2020

DrahtBot locked this on Feb 15, 2022