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.58.0.93 -> 8.59.0.77 #86256
Conversation
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.
Unless there is a good reason not to, I would prioritize the mirror.cs.uchicago.edu
mirror over the web.archive.org
as it is much faster (factor 30x at least).
I tested that all mirrors are able to equally retrieve the deb bundle and resolve to the same sha:
$ version="8.59.0.77"
$ nix-prefetch-url "https://repo.skype.com/deb/pool/main/s/skypeforlinux/skypeforlinux_${version}_amd64.deb"
[77.1 MiB DL]
path is '/nix/store/94v5ixlpdnbsrw3m5s7y5m7mwycv9wpg-skypeforlinux_8.59.0.77_amd64.deb'
1cnlwlp84942ywji3x60zvhijavazdxfym3rfzq3ysky28b6mn6i
$ nix-prefetch-url "https://web.archive.org/web/https://repo.skype.com/deb/pool/main/s/skypeforlinux/skypeforlinux_${version}_amd64.deb"
[77.1 MiB DL]
path is '/nix/store/94v5ixlpdnbsrw3m5s7y5m7mwycv9wpg-skypeforlinux_8.59.0.77_amd64.deb'
1cnlwlp84942ywji3x60zvhijavazdxfym3rfzq3ysky28b6mn6i
$ nix-prefetch-url "https://mirror.cs.uchicago.edu/skype/pool/main/s/skypeforlinux/skypeforlinux_${version}_amd64.deb"[77.1 MiB DL]
path is '/nix/store/94v5ixlpdnbsrw3m5s7y5m7mwycv9wpg-skypeforlinux_8.59.0.77_amd64.deb'
1cnlwlp84942ywji3x60zvhijavazdxfym3rfzq3ysky28b6mn6i
Binary still to be tested.
Binary compiles and runs fine: $ gh pr checkout bachp:skypeforlinux-8.59.0.77
# ..
Switched to branch 'skypeforlinux-8.59.0.77'
$ nix-shell -I nixpkgs=. -p skypeforlinux
these derivations will be built:
/nix/store/271mhjwb997wxvyk3mc6lln1g3zyng4w-skypeforlinux-8.59.0.77.drv
these paths will be fetched (9.60 MiB download, 47.14 MiB unpacked):
# ..
$ which skypeforlinux
/nix/store/wklixy1636hby73ah2s6f0k8i6n4w2lc-skypeforlinux-8.59.0.77/bin/skypeforlinux
$ skypeforlinux
Fontconfig warning: "/etc/fonts/fonts.conf", line 86: unknown element "blank"
(electron) 'setZoomLevel function' is deprecated and will be removed. Please use 'zoomLevel property' instead.
(electron) 'getZoomLevel function' is deprecated and will be removed. Please use 'zoomLevel property' instead.
(electron) 'setBadgeCount function' is deprecated and will be removed. Please use 'badgeCount property' instead.
ioctl failed for wlan0, errno = 19 (No such device)
ioctl failed for wlp2s0, errno = 19 (No such device)
ioctl failed for wlp3s0, errno = 19 (No such device)
|
LGTM. Remains to see what we do with the mirror ordering (see above recommendation). |
I have tested and this works fine. |
The university of chigaco keeps the binaries of old releases. This reduces the change of NixOS#81868 and NixOS#85724 happening again in the future.
0375529
to
e4aab9c
Compare
Mirror order changed. |
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.
Excellent. Can merge.
Thanks! :) |
Motivation for this change
Update to latest skype version.
In addition the university of chicago is added as an additional mirror to try to fetch
the binary from. This reduces the change of #81868 and #85724 happening again in the
future. This might also be a good candidate to backport to stable releases.
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)