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

Refresh cobalt_evergreen_overview.md for Cobalt 24 #1117

Merged
merged 2 commits into from
Aug 4, 2023

Conversation

hlwarriner
Copy link
Contributor

The changes are all to the build section and include:

  • Instructions for building crashpad_handler w/ native_target toolchain
  • Recommendation to use gn gen directly instead of the wrapper script
  • Instructions for setting Starboard version in build commands
  • Removal of gyp references

b/270858365

Change-Id: I6a5270d9ce2afcc444637cf0be81300f38215da6

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #1117 (6d608c0) into main (4358730) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1117      +/-   ##
==========================================
- Coverage   56.54%   56.53%   -0.01%     
==========================================
  Files        1899     1899              
  Lines       94144    94144              
==========================================
- Hits        53237    53228       -9     
- Misses      40907    40916       +9     

see 9 files with indirect coverage changes

Copy link
Contributor

@yuying-y yuying-y left a comment

Choose a reason for hiding this comment

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

Lgtm with a nit

starboard/doc/evergreen/cobalt_evergreen_overview.md Outdated Show resolved Hide resolved
The changes are all to the build section and include:
* Instructions for building crashpad_handler w/ native_target toolchain
* Recommendation to use `gn gen` directly instead of the wrapper script
* Instructions for setting Starboard version in build commands
* Removal of gyp references

b/270858365

Change-Id: I6a5270d9ce2afcc444637cf0be81300f38215da6
b/270858365

Change-Id: I5419405a0bfffc339c916c475dd9309ddbc5ea73
@hlwarriner hlwarriner merged commit 0da921a into youtube:main Aug 4, 2023
333 checks passed
@hlwarriner hlwarriner added the cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch label Aug 4, 2023
cobalt-github-releaser-bot pushed a commit that referenced this pull request Aug 4, 2023
The changes are all to the build section and include:
* Instructions for building crashpad_handler w/ native_target toolchain
* Recommendation to use `gn gen` directly instead of the wrapper script
* Instructions for setting Starboard version in build commands
* Removal of gyp references

b/270858365

Change-Id: I6a5270d9ce2afcc444637cf0be81300f38215da6
(cherry picked from commit 0da921a)
hlwarriner added a commit that referenced this pull request Aug 4, 2023
… 24 (#1148)

Refer to the original PR: #1117

The changes are all to the build section and include:
* Instructions for building crashpad_handler w/ native_target toolchain
* Recommendation to use `gn gen` directly instead of the wrapper script
* Instructions for setting Starboard version in build commands
* Removal of gyp references

b/270858365

Change-Id: I6a5270d9ce2afcc444637cf0be81300f38215da6

Co-authored-by: Holden Warriner <hwarriner@google.com>
hlwarriner added a commit to hlwarriner/cobalt that referenced this pull request Nov 14, 2023
This a smaller change than
youtube#1117 and mostly just cleans
up old GYP references.

b/62677273

Change-Id: Idc9e51463099babf77a92f9fb5c830f9def01084
hlwarriner added a commit that referenced this pull request Nov 14, 2023
This a smaller change than
#1117 and mostly just cleans up
old GYP references.

b/62677273

Change-Id: Idc9e51463099babf77a92f9fb5c830f9def01084
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants