New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
texlive: 2019 -> 2020 #100121
texlive: 2019 -> 2020 #100121
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 |
stripLen = 2; | ||
}) | ||
]; | ||
|
||
# remove when removing synctex-missing-header.patch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Think this should've been removed before, per:comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Motivation for this change
Provide minor fixes for some items in
texlive.bin
. Update to the latest texlive snapshot.Closes: #91994
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)