Build: use short form for action in src/dune (ocaml-ppx#1076)

- `(run)` always sets its first argument as dependency - when the short form is used, targets are inferred from `(with-stdout-to)`
bogdan2412 · Mar 28, 2020 · 3aa188e · 3aa188e
1 parent aef62c3
commit 3aa188e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,5 +1,6 @@
 ### (master)
 
+  + Build: use short form for action in src/dune (#1076) (Etienne Millon)
   + Improve: Implement sequence-blank-line=preserve-one for let bindings (#1077) (Jules Aguillon)
   + Internal: Cleanup sequence_blank_line (#1075) (Jules Aguillon)
   + Improve: set conventional as the default profile (#1060) (Guillaume Petiot)

diff --git a/src/dune b/src/dune
@@ -22,9 +22,7 @@
  (libraries format_ import ocaml-migrate-parsetree odoc.model odoc.parser re uuseg uuseg.string))
 
 (rule
- (targets ocamlformat.1)
- (deps ocamlformat.exe)
- (action (with-stdout-to %{targets} (run ./ocamlformat.exe --help=groff))))
+ (with-stdout-to ocamlformat.1 (run ./ocamlformat.exe --help=groff)))
 
 (install
  (section bin)