Skip to content

Commit

Permalink
Add test for no warning vendor
Browse files Browse the repository at this point in the history
Signed-off-by: Ulysse Gérard <[email protected]>
  • Loading branch information
voodoos committed Nov 16, 2020
1 parent 51a6532 commit 88f3e83
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/blackbox-tests/test-cases/c-flags.t/dune
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,5 @@
(name test)
(foreign_stubs (language c) (names foo))
(foreign_stubs (language c) (names bar) (flags)))

(vendored_dirs vendor)
5 changes: 5 additions & 0 deletions test/blackbox-tests/test-cases/c-flags.t/run.t
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,10 @@ future_c_and_cxx_flags_handling = default (false)
effectively prevent Dune from silently adding c-flags to the compiler
arguments which is the new recommended behaviour.

No warning in vendored subfolder

$ dune build vendor/barv.o

Ocamlc_cflags are duplicated if the :standard set is kept:
$ cat out_foo | grep -ce "${O_CCF} ${O_CCPPF} ${O_CCF}"
1
Expand Down Expand Up @@ -99,3 +103,4 @@ future_c_and_cxx_flags_handling = true but dune < 2.8
of the dune language. Please update your dune-project file to have (lang dune
2.8).
[1]

Empty file.
3 changes: 3 additions & 0 deletions test/blackbox-tests/test-cases/c-flags.t/vendor/dune
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
(library
(name testv)
(foreign_stubs (language c) (names barv) (flags)))

0 comments on commit 88f3e83

Please sign in to comment.