@ferdnyc commented on this pull request.


> +Typographic conventions:
+- Put option aliases on separate lines
+- Put option arguments with pre-determined values into <> (see below)
+
+*-d*
+*--do-this*
+       Do Y instead of default action.
+
+*-o* _ARG_,
+*--output*=_ARG_
+       Use file _ARG_ for output.
+
+*--format*=<*aa*|*bb*>
+       Use format *aa* or *bb* for output.
+
+*--compress*=<_method_>

I meant to mention, also, that CMake never uses all-caps `PLACEHOLDER` values 
-- they use `<placeholder>` exclusively, and (except for `--log-level`) take 
pains to avoid any situations where 'placeholder' would be a literal value. For 
some argument types, that means breaking out the possible option forms 
completely, e.g.:

(man page)
<pre><code>
<b>-Werror=&lt;what&gt;</b>
       Treat CMake warnings as errors. <b>&lt;what&gt;</b> must be one of the 
follow‐
       ing:

       <b>dev</b>    Make developer warnings errors.

       <b>deprecated</b>
              Make deprecated macro and function warnings errors.
</code></pre>

(`--help` output)
```text
  -Werror=dev         = Make developer warnings errors.
  -Werror=deprecated  = Make deprecated macro and function warnings
                        errors.
```


-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3639#discussion_r2003629350
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3639/review/[email protected]>
_______________________________________________
Rpm-maint mailing list
[email protected]
https://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to