Software /
code /
prosody
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 |