Skip to content

Commit

Permalink
Unvirtualize goblint.build-info and goblint.sites
Browse files Browse the repository at this point in the history
  • Loading branch information
sim642 committed Jul 24, 2024
1 parent a00ca1b commit 5a50fa1
Show file tree
Hide file tree
Showing 11 changed files with 16 additions and 42 deletions.
2 changes: 1 addition & 1 deletion gobview
Submodule gobview updated 1 files
+2 −2 src/dune
6 changes: 0 additions & 6 deletions src/build-info/build_info_dune/dune

This file was deleted.

8 changes: 1 addition & 7 deletions src/build-info/dune
Original file line number Diff line number Diff line change
@@ -1,15 +1,9 @@
(include_subdirs no)

; virtual library to allow js build (for gobview) without dune-build-info
; dune-build-info seems to be incompatible with js_of_ocaml
; File "gobview/src/.App.eobjs/build_info_data.ml-gen", line 1:
; Error: Could not find the .cmi file for interface
; gobview/src/.App.eobjs/build_info_data.ml-gen.
(library
(name goblint_build_info)
(public_name goblint.build-info)
(libraries batteries.unthreaded)
(virtual_modules dune_build_info))
(libraries dune-build-info batteries.unthreaded))

(rule
(target configVersion.ml)
Expand Down
File renamed without changes.
8 changes: 4 additions & 4 deletions src/dune
Original file line number Diff line number Diff line change
Expand Up @@ -88,31 +88,31 @@
(public_names goblint)
(modes byte native) ; https://dune.readthedocs.io/en/stable/dune-files.html#linking-modes
(modules goblint)
(libraries goblint.lib goblint.sites.dune goblint.build-info.dune goblint_std)
(libraries goblint.lib goblint.sites goblint.build-info goblint_std)
(preprocess (pps ppx_deriving.std ppx_deriving_hash ppx_deriving_yojson))
(flags :standard -linkall -open Goblint_std)
)

(executable
(name privPrecCompare)
(modules privPrecCompare)
(libraries goblint.lib goblint.sites.dune goblint.build-info.dune goblint_std)
(libraries goblint.lib goblint.sites goblint.build-info goblint_std)
(preprocess (pps ppx_deriving.std ppx_deriving_hash ppx_deriving_yojson))
(flags :standard -linkall -open Goblint_std)
)

(executable
(name apronPrecCompare)
(modules apronPrecCompare)
(libraries goblint.lib goblint.sites.dune goblint.build-info.dune goblint_std)
(libraries goblint.lib goblint.sites goblint.build-info goblint_std)
(preprocess (pps ppx_deriving.std ppx_deriving_hash ppx_deriving_yojson))
(flags :standard -linkall -open Goblint_std)
)

(executable
(name messagesCompare)
(modules messagesCompare)
(libraries goblint.lib goblint.sites.dune goblint.build-info.dune goblint_std)
(libraries goblint.lib goblint.sites goblint.build-info goblint_std)
(preprocess (pps ppx_deriving.std ppx_deriving_hash ppx_deriving_yojson))
(flags :standard -linkall -open Goblint_std)
)
Expand Down
12 changes: 5 additions & 7 deletions src/sites/dune
Original file line number Diff line number Diff line change
@@ -1,12 +1,10 @@
(include_subdirs no)

; virtual library to allow js build (for gobview) without dune-site
; dune-site seems to be incompatible with js_of_ocaml
; File "gobview/src/.App.eobjs/dune_site_data.ml-gen", line 1:
; Error: Could not find the .cmi file for interface
; gobview/src/.App.eobjs/dune_site_data.ml-gen.
(library
(name goblint_sites)
(public_name goblint.sites)
(virtual_modules goblint_sites)
(libraries fpath))
(libraries dune-site fpath))

(generate_sites_module
(module dunesite)
(sites goblint))
File renamed without changes.
12 changes: 0 additions & 12 deletions src/sites/sites_dune/dune

This file was deleted.

4 changes: 2 additions & 2 deletions tests/regression/cfg/util/dune
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,6 @@
goblint_common
goblint_lib ; TODO: avoid: extract LoopUnrolling and WitnessUtil node predicates from goblint_lib
fpath
goblint.sites.dune
goblint.build-info.dune)
goblint.sites
goblint.build-info)
(preprocess (pps ppx_deriving.std)))
2 changes: 1 addition & 1 deletion tests/unit/dune
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

(test
(name mainTest)
(libraries ounit2 qcheck-ounit goblint.std goblint.common goblint.lib goblint.constraint goblint.solver goblint.cdomain.value goblint.sites.dune goblint.build-info.dune)
(libraries ounit2 qcheck-ounit goblint.std goblint.common goblint.lib goblint.constraint goblint.solver goblint.cdomain.value goblint.sites goblint.build-info)
(preprocess (pps ppx_deriving.std ppx_deriving_hash ppx_deriving_yojson))
(flags :standard -linkall))

Expand Down
4 changes: 2 additions & 2 deletions tests/util/dune
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
goblint_std
goblint_lib
yaml
goblint.sites.dune
goblint.build-info.dune)
goblint.sites
goblint.build-info)
(flags :standard -open Goblint_std)
(preprocess (pps ppx_deriving.std)))

0 comments on commit 5a50fa1

Please sign in to comment.