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
skypeforlinux: 8.51.0.72 -> 8.54.0.85 #72850
Conversation
Hello @fishi0x01 and welcome around here since it seems to be your first contribution to nixpkgs :) As per our contribution guidelines, your commit message should be written like this: |
2f48732
to
a4dd9d7
Compare
Thanks for pointing this out @c0bw3b 🙇♂️ I adjusted the commit message as you suggested :) |
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.
I've tested it locally on x64 NixOS and it runs as expected.
Thank you @fishi0x01 for this update. 🚀 |
I think this could be a candidate for backporting to 19.09, as currently this seems broken. I think in such a case a version upgrade of such a leaf package is probably legitimate for the stable release. |
It was updated in #72822 for r19.09 |
@c0bw3b: It does not exist anymore: https://repo.skype.com/deb/pool/main/s/skypeforlinux/
|
Quick fix in e6d584f Wish upstream repo would be more stable. |
Motivation for this change
Source package got removed https://repo.skype.com/deb/pool/main/s/skypeforlinux/skypeforlinux_8.51.0.72_amd64.deb
Updated to latest
x.y.0.z
release, i.e.,8.54.0.85
.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @PanAeon @jraygauthier