Delete removed dependencies from CI package list

latex3 / hyperref

Hypertext support for LaTeX

169 stars 36 forks source link

Delete removed dependencies from CI package list #319

Closed muzimuzhi closed 1 year ago

muzimuzhi commented 1 year ago

Dependencies

etexcmds was removed in 549ee5d (Merge pull request #276 from muzimuzhi/drop-etexcmds, 2023-02-19)
infwarerr was removed in 908e303 (remove infwarerr (#317), 2023-11-26)
ltxcmds and pdftexcmds were removed in 396ba2b (remove ltxcmds dependency (#312), 2023-11-26)

BTW, from a successful CI run, it seems that ltxmisc is not needed too.

u-fischer commented 1 year ago

ltxcmds, pdftexcmds and infwarerr are still needed as they are loaded by pdfescape (and infwarerr is still in hyperref anyway because only the command names were changed but the \RequirePackage has been forgotten). If removing them from the list doesn't break the run, it means they are loaded somewhere else. etexcmds isn't used directly but it is still in some other packages (like e.g. zref) and so I would keep it. I will try what happens if I remove ltxmisc.

muzimuzhi commented 1 year ago

You're right.

Deleting all Actions caches and rerunning the workflow, all the packages this PR proposes to remove are still installed, most likely due to the package dependency info provided by TeX Live, see for instance the dependency list of hyperref currently recorded in TeX Live SVN repo.

I'm closing this PR.

Update: The only exception ltxmisc has been removed in https://github.com/latex3/hyperref/commit/79a3832ff2309d69e87b0cbba0c265f236599444.