Merge pull request #834 from rgrinberg/quoting-test-quoted-case
Add quoted case to quoting test
This commit is contained in:
commit
d708cc714d
|
@ -0,0 +1,3 @@
|
||||||
|
(rule
|
||||||
|
((targets (x y))
|
||||||
|
(action (with-stdout-to ${@} (echo foo)))))
|
|
@ -1,3 +0,0 @@
|
||||||
(rule
|
|
||||||
((targets (x y))
|
|
||||||
(action (with-stdout-to ${@} (echo foo)))))
|
|
|
@ -0,0 +1,3 @@
|
||||||
|
(rule
|
||||||
|
((targets (s t))
|
||||||
|
(action (with-stdout-to "${@}" (echo foo)))))
|
|
@ -1,9 +1,15 @@
|
||||||
This behavior is surprising, we should get an error about the fact
|
This behavior is surprising, we should get an error about the fact
|
||||||
that ${@} is not quoted and doesn't contain exactly 1 element
|
that ${@} is not quoted and doesn't contain exactly 1 element
|
||||||
|
|
||||||
$ dune build x
|
$ dune build --root bad x 2>&1 | grep -v Entering
|
||||||
Error: Rule failed to generate the following targets:
|
Error: Rule failed to generate the following targets:
|
||||||
- x
|
- x
|
||||||
- y
|
- y
|
||||||
[1]
|
|
||||||
|
|
||||||
|
|
||||||
|
The targets should only be interpreted as a single path when quoted
|
||||||
|
|
||||||
|
$ dune build --root good s 2>&1 | grep -v Entering
|
||||||
|
Error: Rule failed to generate the following targets:
|
||||||
|
- s
|
||||||
|
- t
|
||||||
|
|
Loading…
Reference in New Issue