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-08-07 19:53:51 +03:00
parent 272012ea5c
commit 925bc84427
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/jbuild.ml
View File

@ -51,7 +51,7 @@ end = struct
type t = string
let error_message =
"Error: invalid library name\n\
"invalid library name.\n\
Hint: library names must be non-empty and composed only of \
the following characters: 'A'..'Z', 'a'..'z', '_' or '0'..'9'"
@ -934,8 +934,8 @@ module Library = struct
of_sexp_errorf loc
"%s.\n\
Public library names don't have this restriction. \
You can either change this public name to be a valid \
library name or add a `name` field with a valid library name."
You can either change this public name to be a valid library \
name or add a \"name\" field with a valid library name."
Lib_name.error_message
else
of_sexp_error loc "name field cannot be omitted before version \

4
test/blackbox-tests/test-cases/no-name-field/run.t
View File

@ -22,7 +22,7 @@ there's only a public name but it's invalid as a name
$ dune build --root public-name-invalid-name
File "dune", line 1, characters 22-28:
Error: Error: invalid library name
Error: invalid library name.
Hint: library names must be non-empty and composed only of the following characters: 'A'..'Z', 'a'..'z', '_' or '0'..'9'.
Public library names don't have this restriction. You can either change this public name to be a valid library name or add a `name` field with a valid library name.
Public library names don't have this restriction. You can either change this public name to be a valid library name or add a "name" field with a valid library name.
[1]