Skip to content
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

youtube-dl: 2019.05.11 -> 2019.05.20 #61785

Merged
merged 1 commit into from May 23, 2019

Conversation

xrelkd
Copy link
Contributor

@xrelkd xrelkd commented May 21, 2019

Motivation for this change

Update youtube-dl to the latest release.
https://github.com/ytdl-org/youtube-dl/releases/tag/2019.05.20

NOTE: Updates to youtube-dl should be backported, thanks!

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Copy link
Member

@mmahut mmahut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nix-review fails to build this pull request:

$ nix build --no-link --keep-going --max-jobs 1 --option build-use-sandbox true -f /root/.cache/nix-review/pr-61785-2/build.nix
builder for '/nix/store/sl8xr3m91mgvjs5bkpx10ni5l6p6849a-env.drv' failed with exit code 1; last 10 log lines:
  no Makefile, doing nothing
  glibPreInstallPhase
  installing
  glibPreFixupPhase
  post-installation fixup
  shrinking RPATHs of ELF executables and libraries in /nix/store/kjcvr0mwbjzjnqkqsxqscc7rbnygvwx9-env
  strip is /nix/store/2dfjlvp38xzkyylwpavnh61azi0d168b-binutils-2.31.1/bin/strip
  patching script interpreter paths in /nix/store/kjcvr0mwbjzjnqkqsxqscc7rbnygvwx9-env
  checking for references to /build/ in /nix/store/kjcvr0mwbjzjnqkqsxqscc7rbnygvwx9-env...
  mkdir: cannot create directory '/nix/store/kjcvr0mwbjzjnqkqsxqscc7rbnygvwx9-env': Not a directory
[0 built (1 failed), 0.0 MiB DL]
error: build of '/nix/store/sl8xr3m91mgvjs5bkpx10ni5l6p6849a-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/61785
21 package were build:
bomi curseradio deepin.deepin-movie-reborn gnome-mpv kodi-cli lollypop mopidy-youtube mpc-qt mpv mpv-with-scripts persepolis plex-media-player python27Packages.mpv python27Packages.pafy python27Packages.youtube-dl python27Packages.youtube-dl-light python37Packages.mps-youtube python37Packages.mpv python37Packages.pafy youtubeDL youtube-dl-light

$ nix-shell /root/.cache/nix-review/pr-61785-2/shell.nix

[nix-shell:~/.cache/nix-review/pr-61785-2]# 

[nix-shell:~/.cache/nix-review/pr-61785-2]# ./results/youtubeDL/bin/youtube-dl https://www.youtube.com/watch?v=O446XppZzXA
Fatal Python error: initsite: Failed to import the site module
Traceback (most recent call last):
  File "/nix/store/hm6iw7j16i1r7ppvfagsyzzrb9zjbq7x-python2.7-setuptools-41.0.1/lib/python2.7/site-packages/site.py", line 73, in <module>
    __boot()
  File "/nix/store/hm6iw7j16i1r7ppvfagsyzzrb9zjbq7x-python2.7-setuptools-41.0.1/lib/python2.7/site-packages/site.py", line 26, in __boot
    import imp  # Avoid import loop in Python 3
  File "/nix/store/s5f3vpmig33nk4zyk228q55wdydd3pc2-python3-3.7.3/lib/python3.7/imp.py", line 27, in <module>
    import tokenize
  File "/nix/store/s5f3vpmig33nk4zyk228q55wdydd3pc2-python3-3.7.3/lib/python3.7/tokenize.py", line 33, in <module>
    import re
  File "/nix/store/s5f3vpmig33nk4zyk228q55wdydd3pc2-python3-3.7.3/lib/python3.7/re.py", line 143, in <module>
    class RegexFlag(enum.IntFlag):
AttributeError: module 'enum' has no attribute 'IntFlag'

[nix-shell:~/.cache/nix-review/pr-61785-2]# 

@worldofpeace
Copy link
Contributor

Because you used nix-review which made a nix-shell mixing python interpreter versions it causes that error.
If you were to execute just youtube-dl outside of that it should work properly.

❯ ./result/bin/youtube-dl "https://www.youtube.com/watch?v=O446XppZzXA"                                                                         
[youtube] O446XppZzXA: Downloading webpage
[youtube] O446XppZzXA: Downloading video info webpage
[download] Destination: DíSA - New World Coming - Miss Peregrine's Home For Peculiar Children Trailer song-O446XppZzXA.f243.webm
[download] 100% of 2.19MiB in 00:01
[download] Destination: DíSA - New World Coming - Miss Peregrine's Home For Peculiar Children Trailer song-O446XppZzXA.f251.webm
[download] 100% of 2.75MiB in 00:01
[ffmpeg] Merging formats into "DíSA - New World Coming - Miss Peregrine's Home For Peculiar Children Trailer song-O446XppZzXA.webm"
Deleting original file DíSA - New World Coming - Miss Peregrine's Home For Peculiar Children Trailer song-O446XppZzXA.f243.webm (pass -k to keep)
Deleting original file DíSA - New World Coming - Miss Peregrine's Home For Peculiar Children Trailer song-O446XppZzXA.f251.webm (pass -k to keep)

@worldofpeace worldofpeace merged commit e57bb43 into NixOS:master May 23, 2019
@worldofpeace
Copy link
Contributor

backported in e1cd559

@xrelkd xrelkd deleted the update/youtube-dl branch May 24, 2019 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants