-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
[misc] Cleanup for 2024-05 release #9765
Merged
Merged
Changes from 7 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
d0ef065
Use `self.assertWarns`
Grub4K c36cac6
[Makefile] Bring `offlinetest` recipe on par with `devscripts.run_tests`
bashonly b9aeb3e
[Makefile] Add `-Werror` arg to `test` recipe
bashonly ed44e23
Merge branch 'yt-dlp:master' into cleanup/2024-05
bashonly adb1483
[utils] `read_batch_urls`: Fix deprecated `re.split` positional arg (…
bashonly c91b905
[ie] Fix deprecated `re.split` positional arg (3.13)
bashonly da483f2
[docs] Remove `yt_dlp_linux.zip` from README
bashonly 4d72297
Merge branch 'master' into pr/9765
seproDev 97c4c86
[ie/vk] Extract additional potentially existing formats
seproDev b01f22c
[docs] brackets must be quoted in zsh
bashonly 41cef94
Remove deprecated run_tests scripts
seproDev 454bf0b
Remove compat `setup.py` and `pyinst.py`
bashonly 04c5037
Remove deprecated option from example
seproDev 91d5892
Merge branch 'yt-dlp:master' into cleanup/2024-05
bashonly 61a1e15
Changelog override: Suppress "[build] Exclude `requests` from `py2exe`"
bashonly 4101ab3
Merge branch 'yt-dlp:master' into cleanup/2024-05
bashonly 7c044bd
changelog overrides
bashonly 4df35f3
Merge branch 'yt-dlp:master' into cleanup/2024-05
bashonly f1bb4ab
Merge branch 'yt-dlp:master' into cleanup/2024-05
bashonly File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Why are we dropping it?
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.
Mamba build suddenly didnt want to work anymore and so we had to switch to a different strategy (statically linked musl libc builds). I have a plan to potentially bring them back but it will require some research and some trial and error. Bringing back working builds seemed of more immediate importance.
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 have seen the discussion. But the zip build will be the same workflow with
pyinstaller --one-dir
, no? Sure, fixing it is not a priority, but I thought it'd be trivialThere 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.
Here is a good overview of what staticx does: JonathonReinhart/staticx#239 (comment)
It works similarly to pyinstaller
--onefile
; it's just another layer of bundling on top with someRPATH
+INTERP
special sauce. The maintainer also says here that it's not possible to use staticx with pyinstaller--onedir
builds, so we would need to get very inventive to prove them wrong(?)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.
On that note, I think the static exe is a bit problematic since we are doubling the startup time and /tmp usage. I want to try replacing the
linux_arm
job with a CentOS 7 build flow; if that's successful, and if we can't find a way to optimize the static exe, then maybe we could build thelinux_exe
using the CentOS 7 framework as well.