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

fix: Roll back depot_tools, bypass vpython #1045

Merged
merged 1 commit into from
Mar 9, 2022

Conversation

joeyparrish
Copy link
Member

@joeyparrish joeyparrish commented Mar 8, 2022

Using the latest depot_tools no longer works. depot_tools also wants
to auto-update itself, which must now be disabled.

We also need to disable the copy of python (vpython) included in
depot_tools, since for some distros, it has dependencies on system
libraries that no longer exist.

Finally, we need to force some distros to use python 2, because our
build system is ancient and needs to be ripped out and replaced some
day soon.

This fixes build issues in our CI, our Dockerfiles, and in general on
certain platforms or distros.

Closes #1023

@joeyparrish joeyparrish force-pushed the depot_tools branch 2 times, most recently from 274c7a5 to b16d706 Compare March 8, 2022 20:09
@joeyparrish joeyparrish changed the title fix: Fix build failure by rolling back to an older version of depot_tools fix: Roll back depot_tools, bypass vpython Mar 8, 2022
@joeyparrish joeyparrish requested a review from theodab March 8, 2022 20:10
@joeyparrish joeyparrish force-pushed the depot_tools branch 3 times, most recently from e13bd4d to 30d95a9 Compare March 8, 2022 20:37
Using the latest depot_tools no longer works.  depot_tools also wants
to auto-update itself, which must now be disabled.

We also need to disable the copy of python (vpython) included in
depot_tools, since for some distros, it has dependencies on system
libraries that no longer exist.

Finally, we need to force some distros to use python 2, because our
build system is ancient and needs to be ripped out and replaced some
day soon.

This fixes build issues in our CI, our Dockerfiles, and in general on
certain platforms or distros.

Closes shaka-project#1023
@joeyparrish joeyparrish requested review from theodab and removed request for theodab March 8, 2022 21:00
@joeyparrish
Copy link
Member Author

Windows builds are still failing, but the rest are working now. PTAL!

@joeyparrish joeyparrish merged commit 3fd538a into shaka-project:main Mar 9, 2022
@joeyparrish joeyparrish deleted the depot_tools branch March 9, 2022 00:46
sr1990 pushed a commit to sr1990/shaka-packager that referenced this pull request Feb 18, 2023
Using the latest depot_tools no longer works.  depot_tools also wants
to auto-update itself, which must now be disabled.

We also need to disable the copy of python (vpython) included in
depot_tools, since for some distros, it has dependencies on system
libraries that no longer exist.

Finally, we need to force some distros to use python 2, because our
build system is ancient and needs to be ripped out and replaced some
day soon.

This fixes build issues in our CI, our Dockerfiles, and in general on
certain platforms or distros.

Closes shaka-project#1023
sr1990 pushed a commit to sr1990/shaka-packager that referenced this pull request Feb 18, 2023
Using the latest depot_tools no longer works.  depot_tools also wants
to auto-update itself, which must now be disabled.

We also need to disable the copy of python (vpython) included in
depot_tools, since for some distros, it has dependencies on system
libraries that no longer exist.

Finally, we need to force some distros to use python 2, because our
build system is ancient and needs to be ripped out and replaced some
day soon.

This fixes build issues in our CI, our Dockerfiles, and in general on
certain platforms or distros.

Closes shaka-project#1023
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Jul 24, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker build fails on the v2.5.0 tag
2 participants