Skip to content

Commit

Permalink
Editorial changes
Browse files Browse the repository at this point in the history
Editorial changes to doc/sites.rst,
test/blackbox-tests/test-cases/toplevel-plugin-fail.t, and
test/blackbox-tests/test-cases/toplevel-plugin-fail2.t.

Co-authored-by: Christine Rose <[email protected]>
Signed-off-by: Richard L Ford <[email protected]>
  • Loading branch information
richardlford and christinerose committed Mar 7, 2023
1 parent 3483d56 commit 102b609
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion doc/sites.rst
Original file line number Diff line number Diff line change
Expand Up @@ -258,7 +258,7 @@ The generated module `sites` depends here also on the library
If the executable being created is an OCaml toplevel, then the
``libraries`` stanza needs to also include the ``dune-site.toplevel``
library. This causes the loading to use the toplevel's normal loading
mechanism rather than Dynload.loadfile (which is not allowed in
mechanism rather than ``Dynload.loadfile`` (which is not allowed in
toplevels).

- The module ``registration.ml`` of the library ``app.registration``:
Expand Down
4 changes: 2 additions & 2 deletions test/blackbox-tests/test-cases/toplevel-plugin-fail.t
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Testsuite for (toplevel that loads plugins). This version
uses dune-site.dynlink which uses Dynlink.loadfile.
This is not allowed in top-levels, so it fails.
uses ``dune-site.dynlink`` which uses ``Dynlink.loadfile``.
This is not allowed in toplevels, so it fails.

$ cat > dune-project <<EOF
> (lang dune 3.7)
Expand Down
6 changes: 3 additions & 3 deletions test/blackbox-tests/test-cases/toplevel-plugin-fail2.t
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
Testsuite for (toplevel that loads plugins). This version
still uses dune-site.dynlink, but only implicitly as it is
still uses ``dune-site.dynlink``, but only implicitly as it is
the default implementation of the virtual library.
It uses Dynlink.loadfile.
This is not allowed in top-levels, so it fails.
It uses ``Dynlink.loadfile``.
This is not allowed in toplevels, so it fails.

$ cat > dune-project <<EOF
> (lang dune 3.7)
Expand Down

0 comments on commit 102b609

Please sign in to comment.