ypid / latex-packages

My LaTeX Packages.
https://ctan.org/author/schneider-r
25 stars 22 forks source link

doclicense: Specify package order for any dependencies in documentation #79

Closed thbde closed 3 years ago

thbde commented 3 years ago

It would be helpful if the documentation contains a sentence in which order I should declare my packages.

Following https://github.com/ypid/latex-packages/issues/66#issuecomment-698630391, it seems that the order is important.

hyperxmp, loaded by doclicense, states in the documentation https://mirror.informatik.hs-fulda.de/tex-archive/macros/latex/contrib/hyperxmp/hyperxmp.pdf :

To use hyperxmp, merely put a \usepackage{hyperxmp} in your document’s preamble. That line can appear anywhere before the hyperref pdf options are specified (i.e., with either \usepackage[. . . ]{hyperref} or \hypersetup{. . . }).

Conclusively, I would assume doclicensemust appear before hypperref?

It is unclear to me if there is any order dependency to hyperxmpnow. Most likely no, but https://github.com/ypid/latex-packages/issues/66#issuecomment-698630391 is not so clear in this aspect.

ypid commented 3 years ago

Can you give it a try in a pull request? Generally, I prefer when the changes are done directly to the source rather than talking about it :)

thbde commented 3 years ago

In general yes, I can sent a pull request. However, in this case, the specification is not fully clear for me. I can conclude:

  1. doclicense should appear before hyperref if hyperxmp is loaded

It is unclear to me:

  1. Must doclicense appear before hyperxmp? Or after? Or maybe there is no dependency at all to hyperxmp
  2. Anything else?
ypid commented 3 years ago

Don’t worry. I wrote the section. Does #84 solve it? I hope I did not overlook a package load order constraint.

thbde commented 3 years ago

Looks good. The loading order via the hooks will indeed prevent some possible issues if I understand it correctly.

ypid commented 3 years ago

Merged, thanks for all your feedback :) I will stick to my plan to release a new version on 2021-07-10.

thbde commented 3 years ago

Thank you very much for the fast reactions. I have adapted my affected projects to work with the current and the new version.