From f1f95cda37cf2502aa3ea4974c1dd13a64e8e7fe Mon Sep 17 00:00:00 2001 From: Jeremie Dimino Date: Fri, 26 May 2017 12:33:55 +0100 Subject: [PATCH] Clarify --only-packages --- bin/main.ml | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/bin/main.ml b/bin/main.ml index 64df5880..182136e4 100644 --- a/bin/main.ml +++ b/bin/main.ml @@ -151,9 +151,11 @@ let common = & opt (some string) None & info ["only-packages"] ~docs ~docv:"PACKAGES" ~doc:{|Ignore stanzas referring to a package that is not in $(b,PACKAGES). - $(b,PACKAGES) is a coma-separated list of package name. You need to - use this option in your $(i,.opam) file if your project - contains several packages.|} + $(b,PACKAGES) is a coma-separated list of package name. + Note that this has the same effect as deleting the relevant stanzas + from jbuild files. It is mostly meant for releases. + During development, it is likely that what you want instead is to + build a particular $(b,.install) target.|} ) in let ddep_path = @@ -202,7 +204,10 @@ let common = Arg.(value & opt (some string) None & info ["p"; for_release] ~docs ~docv:"PACKAGES" - ~doc:{|Shorthand for $(b,--root . --only-packages PACKAGE).|}) + ~doc:{|Shorthand for $(b,--root . --only-packages PACKAGE). You must use + this option in your $(i,.opam) files, in order to build + only what's necessary when your project contains multiple packages + as well as getting reproducible builds.|}) in let root_and_only_packages = let merge root only_packages release =