Add test to show that jbuilder mishandles ${null}
Jbuilder incorrectly infers ${null} to be a target
This commit is contained in:
parent
0571504abb
commit
8c86604ee6
|
@ -480,3 +480,12 @@
|
||||||
(run ${exe:cram.exe} run.t)
|
(run ${exe:cram.exe} run.t)
|
||||||
(diff? run.t run.t.corrected))))))
|
(diff? run.t run.t.corrected))))))
|
||||||
|
|
||||||
|
(alias
|
||||||
|
((name runtest)
|
||||||
|
(deps ((package jbuilder)
|
||||||
|
(files_recursively_in test-cases/null-dep)))
|
||||||
|
(action
|
||||||
|
(chdir test-cases/null-dep
|
||||||
|
(progn
|
||||||
|
(run ${exe:cram.exe} run.t)
|
||||||
|
(diff? run.t run.t.corrected))))))
|
||||||
|
|
|
@ -0,0 +1,5 @@
|
||||||
|
(jbuild_version 1)
|
||||||
|
|
||||||
|
(alias
|
||||||
|
((name runtest)
|
||||||
|
(action (with-stdout-to ${null} (echo "hello world")))))
|
|
@ -0,0 +1,4 @@
|
||||||
|
$ jbuilder runtest --debug-dependency-path
|
||||||
|
File "jbuild", line 5, characters 26-33:
|
||||||
|
Warning: Aliases must not have targets, this target will be ignored.
|
||||||
|
This will become an error in the future.
|
Loading…
Reference in New Issue