Skip to content

Remove dead special case for defunct branch COBALT #4850

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

Merged
merged 11 commits into from
Feb 6, 2025

Conversation

dahlstrom-g
Copy link
Contributor

… and inline local variable gcs_path_suffix.

Issue: 365150653

Remove currently unsupported release packaging builds (evergreen) and
simplify logic for supported release packaging builds. Remove deprecated
code to improve readability.

b/393447640
The required files were manually determined. This moves the release
packging logic to the builder side rather than the snapshot side
allowing better developer access and maintaince.

b/393447640
… and inline local variable gcs_path_suffix.

Issue: 365150653
Reviewed-on: youtube#4850
isarkis and others added 6 commits February 5, 2025 10:34
Copy chromecast::media::VideoGeometrySetterService to Cobalt folder, as
this is needed for youtube#4810.

The PR doesn't have any functional changes.

b/391938746
The required files were manually determined. This moves the release
packging logic to the builder side rather than the snapshot side
allowing better developer access and maintaince.

b/393447640
Create new matrix job for each test target that prints the logs and
handles results per target.

b/354062546
Remove currently unsupported release packaging builds (evergreen) and
simplify logic for supported release packaging builds by removing
redundant function "create_and_upload_nightly_archive". Remove
deprecated code "simple_packager" to improve readability.

b/393447640
b/393160172

Set is_starboard = false globally and true for linux based cobalt and
starboard toolchains.
Remove current toolchain info from is_cobalt_hermetic_build which makes
it easier to be replaced by is_starboard
niranjanyardi and others added 2 commits February 5, 2025 23:04
…ain args (youtube#4852)

b/393160172

Make is_cobalt_hermetic_build a global variable so that it can be easily
replaced by is_starboard and we can avoid some unnecessary includes
… and inline local variable gcs_path_suffix.

Issue: 365150653
@dahlstrom-g dahlstrom-g requested a review from a team as a code owner February 6, 2025 04:00
@dahlstrom-g dahlstrom-g merged commit 6d680b8 into youtube:main Feb 6, 2025
1 check passed
kaidokert pushed a commit to kaidokert/cobalt that referenced this pull request Jun 15, 2025
… and inline local variable gcs_path_suffix.

Issue: 365150653
briantting pushed a commit that referenced this pull request Jul 1, 2025
… and inline local variable gcs_path_suffix.

Issue: 365150653

original-hexsha: 6d680b8
briantting pushed a commit that referenced this pull request Jul 11, 2025
… and inline local variable gcs_path_suffix.

Issue: 365150653

original-hexsha: 6d680b8
sideb0ard pushed a commit to sideb0ard/cobalt that referenced this pull request Jul 25, 2025
… and inline local variable gcs_path_suffix.

Issue: 365150653

original-hexsha: 6d680b8
sideb0ard pushed a commit to sideb0ard/cobalt that referenced this pull request Jul 26, 2025
… and inline local variable gcs_path_suffix.

Issue: 365150653

original-hexsha: 6d680b8
sideb0ard pushed a commit to sideb0ard/cobalt that referenced this pull request Jul 30, 2025
… and inline local variable gcs_path_suffix.

Issue: 365150653

original-hexsha: 6d680b8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants