-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
[Backport release-24.05] legcord: init at 1.0.1, armcord: remove, add throw with migration details #347988
[Backport release-24.05] legcord: init at 1.0.1, armcord: remove, add throw with migration details #347988
Conversation
3906a77
to
32536e1
Compare
Not quite sure what’s going on there (source vs. binary package difference?). |
32536e1
to
be8b795
Compare
I must have had something cached locally when I tested, thanks for the catch on this. Not knowing what else we will encounter between a source build on 24.05 vs on master, I am going to just revert to the deb-based packaging that we were doing with armcord/legcord already on 24.05 and let it be retired when 24.05 goes out of support. |
be8b795
to
0e4b0ab
Compare
@emilazy Apologies for the direct ping, could you take a second look and let me know if we need any more changes? Just want to close the loop on the armcord/legcord thing. |
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.
No worries about the ping, I think I was waiting for ofborg and then forgot about it ^^;
LGTM, thank you for dealing with this.
Things done
ArmCord was renamed to Legcord and re-versioned to 1.0.0 upstream, see the release notes for details on this change. Updated to 1.0.1 while waiting on change reviews for this PR.
Context and discussion can be found on the PR to unstable: #347971
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.