-
Notifications
You must be signed in to change notification settings - Fork 15.1k
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
build: update libcxx filenames #35397
Conversation
publish-linux job here is passing where it was previously failing - here it’s failing on upload, (which is expected, this isn’t a real release) |
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.
Linux failure is:
upload.py: error: unrecognized arguments: --UPLOAD_TO_STORAGE
which is not something i think is directly caused by this PR? that's here:
electron/.circleci/config/base.yml
Line 638 in 97b353a
script/release/uploaders/upload.py --verbose --UPLOAD_TO_STORAGE |
however, i'm not sure why --UPLOAD_TO_STORAGE
is all caps? other usage is lowecase and the file itself indicates that as well:
electron/script/release/uploaders/upload.py
Line 175 in ff804e3
parser.add_argument('-s', '--upload_to_storage', |
I believe it's case-sensitive so maybe we should try changing that @VerteDinde 🤔
@codebytere Good call! You're right, I don't know why that's listed in uppercase here 🤔 I'll try pushing a fix here and seeing if it makes a difference |
No Release Notes |
/trop run backport-to 21-x-y |
The backport process for this PR has been manually initiated - sending your PR to |
I have automatically backported this PR to "21-x-y", please check out #35497 |
* build: update libcxx filenames * build: change upload_to_storage variable
Description of Change
This PR updates the needed libcxx filenames in filenames.libcxx.gn after a recent Chromium bump.
Fixes main
publish-linux
job for nightly.Checklist
npm test
passesRelease Notes
Notes: none