matthieu
/
dune
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix error message 

Signed-off-by: Rudi Grinberg <rudi.grinberg@gmail.com>
This commit is contained in:
Rudi Grinberg 2018-07-06 19:32:46 +07:00
parent 998ff8ce8c
commit 1a37977f62
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/super_context.ml
View File

@ -95,7 +95,8 @@ let expand_var_no_root t loc syntax_version var =
if syntax_version < (1, 0) then
v
else
Loc.fail loc "Uppercase variables are removed in dune files. Use: %%{%s}"
Loc.fail loc "Uppercase variables are removed in dune files.@.\
Hint: Did you mean %%{%s} instead?"
(String.lowercase var)
end

3
test/blackbox-tests/test-cases/dune-jbuild-var-case/run.t
View File

@ -6,7 +6,8 @@ All builtin variables are lower cased in Dune:
$ dune runtest --root dune-upper
Entering directory 'dune-upper'
File "dune", line 3, characters 41-46:
Error: Uppercase variables are removed in dune files. Use: %{make}
Error: Uppercase variables are removed in dune files.
Hint: Did you mean %{make} instead?
[1]
jbuild files retain the the old names: