math/R: Revert documentation changes

[jrm]

Summary

math/R: Revert documentation changes

In an attempt to simplify the handling of documentation for users, changes
were made in the last commit. When the documentation knob was off,
(almost) no HTML, PDF, etc. files or their sources were installed, and when
the knob was on, all these files were installed. Because some of these
documentation files depended on some very large TeX ports, despite the
usual convention, the knob was turned off by default. Unfortunately, this
caused problems for a few R-dependent ports, because they copy
documentation files from math/R when they are installed.

[mat]

If other ports *need* the documentation to be installed, then it should not be optional here.

Now that I said that, I have a feeling the correct solution is to fix the other ports to not copy R's documentation when installing.

[jrm]

math/R/Makefile

147–153 ↗

(On Diff #21963)

This was the part that made those files optional. It's been removed, so they are again installed regardless what the knob is set to. That being said, it does seems strange to copy files from another port. The offending ports are finance/R-cran-timeSeries, finance/R-cran-tseries, math/R-cran-car, and textproc/R-cran-R2HTML. To prevent the error

Warning in file.copy(file.path(R.home("doc"), "html", "R.css"), outman) :
  problem copying /usr/local/share/doc/R/html/R.css to /wrkdirs/usr/ports/finance/R-cran-timeSeries/work/stage/usr/local/lib/R/library/timeSeries/html/R.css: No such file or directory`

they can break the convention of installing documentation by default by using --no-docs and --no-html, although this might be unexpected given these lines in cran.mk.

.if defined(NOPORTDOCS)
R_POSTCMD_INSTALL_OPTIONS+=	--no-docs --no-html
.endif

[swills]

Approved

[mat]

math/R/Makefile

147–153 ↗

(On Diff #21963)

Mmm, ok, but then, do not make the DOCS option install all of texlive, keep the ALLDOCS option.

The idea is that the DOCS option is lightweight, if it brings texlive, it is not any more and it should not be used for it.

[jrm]

But then we would circle back to the original issue of having multiple knobs for documentation (aside from INFO). The only consolation is that it's a build dependency.

[mat]

In D8429#175486, @jrm wrote:

But then we would circle back to the original issue of having multiple knobs for documentation (aside from INFO). The only consolation is that it's a build dependency.

Yes, multiple knobs is good, DOCS for the lightweight one, and <WHATEVERYOUWANT> for the one nobody will ever use :-)