borisveytsman / acmart

ACM consolidated LaTeX styles
595 stars 252 forks source link

Loading order of hyperref and hyperxmp should be reversed #510

Open iincer opened 11 months ago

iincer commented 11 months ago

Summary

The current version of acmart raises an error saying that hyperxml must be loaded before hyperxmp.

Reason

hyperxmp v5.12 (released on 9/10/23) introduced the following requirement:

New in v5.12 hyperref must be loaded before hyperxmp.
This is because recent changes in hyperref and the LATEX
kernel prevent hyperxmp from hooking into hyperref’s internals
if hyperref is loaded first.

The following code shows how acmart loads hyperref and hyperxmp:

https://github.com/borisveytsman/acmart/blob/ec4b7b08252c312afb523f7162938b9e439c5b1d/acmart.dtx#L3474-L3489

Solution

Reverse the loading order of hyperref and hyperxmp.

krono commented 11 months ago

This is already tracked i #505