-
Notifications
You must be signed in to change notification settings - Fork 415
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test: missing opam file now causes build failure (#8662)
Signed-off-by: Etienne Millon <[email protected]>
- Loading branch information
Showing
1 changed file
with
21 additions
and
0 deletions.
There are no files selected for viewing
21 changes: 21 additions & 0 deletions
21
test/blackbox-tests/test-cases/missing-opam-generated-file.t
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
This shows a difference since #8518: we set up a project with a generated opam | ||
file. If the resulting file is not present in the repository (or if it has been | ||
renamed to just `opam` as in this example), we now fail. | ||
|
||
A related issue happens when `pkg.opam` exists but is not up to date. The | ||
instruction in the opam file would previously consider the original version but | ||
will overwrite `pkg.opam`. Now it does not update the file. | ||
|
||
$ cat > dune-project << EOF | ||
> (lang dune 1.10) | ||
> (generate_opam_files) | ||
> (package | ||
> (name pkg)) | ||
> EOF | ||
|
||
$ touch opam | ||
|
||
$ dune build -p pkg @install | ||
Error: No rule found for pkg.opam | ||
-> required by alias install | ||
[1] |