Conversation
|
Builds and works for me (tested on a few latex files). |
|
This PR is still marked as draft, but pretty much LGTM from my side. Is there anything that you still want to do? |
This used to be broken, but recently was fixed.
The ppc64 condition is based on previous reports and was not tested on actual hardware.
Also change default package version from texlive year to a revision. The problem with using texlive year is that it will change on each of our update even if the package itself was not updated. The texlive SVN revision number should not change unless the package is updated. This will allow to share more store paths between different texlive versions.
d36db9b to
10438ba
Compare
|
@gebner Just pushed a new changeset. Main difference is a change of default versioning scheme for unversioned packages, it now uses SVN revision number instead of the texlive year. There are also minor tweaks for luajit (cc @CrystalGamma for ppc64 changes). We usually had major texlive updates sit for a while before getting merged, but we could try merging sooner this time. |
|
@GrahamcOfBorg build tests.texlive texlive.combined.scheme-full |
I know, in 2017 I merged the texlive update after it had been sitting around for over a month with no activity, and we only got bug reports after it was merged. We don't gain much by letting this PR sit longer than necessary. If it builds, then I think we should merge it. |
|
I ran into one regression, the new biber 2.15 doesn't like authors such as |
| }) | ||
| ]; | ||
|
|
||
| # remove when removing synctex-missing-header.patch |
There was a problem hiding this comment.
Think this should've been removed before, per:comment?
Motivation for this change
Provide minor fixes for some items in
texlive.bin. Update to the latest texlive snapshot.Closes: #91994
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)