Skip to content

PR#7007, PR#7311: document OCAMLPARAM and ocaml_compiler_internal_params#879

Merged
damiendoligez merged 2 commits intoocaml:4.04from
damiendoligez:document-param-modifiers
Nov 3, 2016
Merged

PR#7007, PR#7311: document OCAMLPARAM and ocaml_compiler_internal_params#879
damiendoligez merged 2 commits intoocaml:4.04from
damiendoligez:document-param-modifiers

Conversation

@damiendoligez
Copy link
Copy Markdown
Member

Here is my first draft. Comments and suggestions are welcome.


\noindent
On native Windows, the following environment variable is also consulted:
\paragraph{contextual control of command-line options}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capital C


The compiler command line can be modified ``from the outside''
with the following mechanisms. These are experimental
and subject to change, and they are not intended for normal use.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about "and subject to change. They should be used only for experimental and development work---never in released packages." ?

configured value. Primarily used for bootstrapping.
\end{options}

\section{Common errors}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we "include" this section from a separate file maybe?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a lot of duplicate text between comp.etex and native.etex. Factoring it out may be a good idea, but I won't attempt it in this PR.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Concerning the duplication between comp.etex and native.etex, I had proposed to merge these files together (with top.etex) and handle the differences with tex macros in #491 . If there are some interests in this solution, I could rebase this PR; or create a version merging only comp.etex and native.etex − which would decrease the complexity of the resulting tex file.

Copy link
Copy Markdown
Contributor

@alainfrisch alainfrisch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I don't think that the information is enough to use either ocaml_compiler_internal_params or OCAMLPARAM, but the text carry the intented message ("do not use!").


\noindent
On native Windows, the following environment variable is also consulted:
\paragraph{contextual control of command-line options}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would use a capitalized title here ("Contextual").

@dra27
Copy link
Copy Markdown
Member

dra27 commented Oct 30, 2016

+1 - looks good to me

@gasche
Copy link
Copy Markdown
Member

gasche commented Nov 1, 2016

That's a good minimal first step. As a user when reading this documentation I would have liked to see examples of the syntax, with the following questions in mind:

  • for OCAMLPARAM, what is evaluated before and after command-line parsing?
  • for ocaml_internal_params, the documentation says that specific file paths/names are mapped to options, but are those relative or absolute names?
  • I suppose there is a wildcard syntax to add an option to all files, what is its syntax?

@damiendoligez damiendoligez added this to the 4.04 milestone Nov 2, 2016
@damiendoligez
Copy link
Copy Markdown
Member Author

@gasche The idea (for the moment) is to document their existence so that users who see their compiler behaving weirdly will have a chance to figure out what is going on. Do you agree to merge this minimal first step for 4.04, then work on expanding the documentation in trunk?

@gasche
Copy link
Copy Markdown
Member

gasche commented Nov 2, 2016

Yep, sure.

@damiendoligez damiendoligez merged commit 60aa30f into ocaml:4.04 Nov 3, 2016
dra27 pushed a commit to dra27/ocaml that referenced this pull request Dec 10, 2016
dra27 pushed a commit to dra27/ocaml that referenced this pull request Dec 10, 2016
dra27 pushed a commit to dra27/ocaml that referenced this pull request Dec 11, 2016
dra27 pushed a commit to dra27/ocaml that referenced this pull request Dec 11, 2016
dra27 pushed a commit to dra27/ocaml that referenced this pull request Dec 12, 2016
dra27 pushed a commit to dra27/ocaml that referenced this pull request Dec 14, 2016
@damiendoligez damiendoligez deleted the document-param-modifiers branch June 28, 2017 12:05
camlspotter pushed a commit to camlspotter/ocaml that referenced this pull request Oct 17, 2017
stedolan pushed a commit to stedolan/ocaml that referenced this pull request Oct 25, 2022
EmileTrotignon pushed a commit to EmileTrotignon/ocaml that referenced this pull request Jan 12, 2024
* Sort books at build

* Formatting

---------

Co-authored-by: Cuihtlauac ALVARADO <cuihtmlauac@tarides.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants