Comparison

makefile @ 11748:88ba05494d17 0.11

makefile: fix prosody.version target POSIX is quite explicit regarding the precedence of AND-OR lists [0]: > The operators "&&" and "||" shall have equal precedence and shall be > evaluated with left associativity. For example, both of the following > commands write solely `bar` to standard output: > false && echo foo || echo bar > true || echo foo && echo bar Given that, `prosody.version` target behaves as ((((((test -f prosody.release && cp ...) || test -f ...) && sed ...) || test -f ...) && hexdump ...) || echo unknown > $@) In the case of release tarballs, `prosody.release` does exist, so the first AND pair is executed. Given that it's successful, then the first `test -f` in the OR pair is ignored, and instead the `sed` in the AND pair is executed. `sed` success, as `.hg_archival.txt` exists, making the second `test -f` in the OR pair ignored, and `hexdump` in the AND pair is executed. Now, given that `.hg` doesn't exist, it fails, so the last `echo` is run, overwriting `prosody.version` with `unknown`. This can be worked around placing `()` around the AND pairs. Decided to use conditionals instead, as I think they better communicate the intention of the block. [0]: https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_09_03
author Lucas <lucas@sexy.is>
date Sun, 15 Aug 2021 04:10:36 +0000
parent 8593:c4222e36333c
child 11750:a8760562a096
comparison
equal deleted inserted replaced
11746:68faaf936f6d 11748:88ba05494d17
88 88
89 prosody.cfg.lua.install: prosody.cfg.lua.dist 89 prosody.cfg.lua.install: prosody.cfg.lua.dist
90 sed 's|certs/|$(INSTALLEDCONFIG)/certs/|' prosody.cfg.lua.dist > $@ 90 sed 's|certs/|$(INSTALLEDCONFIG)/certs/|' prosody.cfg.lua.dist > $@
91 91
92 prosody.version: 92 prosody.version:
93 test -f prosody.release && \ 93 if [ -f prosody.release ]; then \
94 cp prosody.release $@ || \ 94 cp prosody.release $@; \
95 test -f .hg_archival.txt && \ 95 elif [ -f .hg_archival.txt ]; then \
96 sed -n 's/^node: \(............\).*/\1/p' .hg_archival.txt > $@ || \ 96 sed -n 's/^node: \(............\).*/\1/p' .hg_archival.txt > $@; \
97 test -f .hg/dirstate && \ 97 elif [ -f .hg/dirstate ]; then \
98 hexdump -n6 -e'6/1 "%02x"' .hg/dirstate > $@ || \ 98 hexdump -n6 -e'6/1 "%02x"' .hg/dirstate > $@; \
99 echo unknown > $@ 99 else \
100 100 echo unknown > $@; \
101 101 fi