-
Notifications
You must be signed in to change notification settings - Fork 49
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
Upgrade BoringSSL #111
Labels
Comments
I would like to help on this bug. |
I have plans to upgrade boringssl, but haven't had the time.
Last I tried it turned out that the python script had changed a lot. So
we'll need to change how we generate lists of sources... And possibly how
we build too.
In any case, yes, it's something we want to do.
@bhushangitfull, you are most welcome to help.
I'd suggest making a fork, and trying to fix:
https://github.com/google/webcrypto.dart/blob/master/tool/update-boringssl.py
You could also try to the steps manually, and we can find out how to
automatically roll it later.
But I do thing we want rolls to be scripted, we don't want this package
laggy behind boringssl.
If you get something even partially working, please share as a separate
branch. I'm currently on parental leave, but back next week and will
hopefully find time to help with this eventually.
tors. 16. jan. 2025, 04.11 skrev Buddhbhushan Kamble <
***@***.***>:
… I would like to help on this bug.
—
Reply to this email directly, view it on GitHub
<#111 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABERZAWHIFXFTRGR6X2LDT2K4PM7AVCNFSM6AAAAABUM67TZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOJUGM4DGMJTGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I will give my best |
Hi @jonasfj, What do you think about using gith hub action workflow for making the rolling process automatic? |
I don't mind, but let's get the rolling process working first :D |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We need to upgrade BoringSSL, however, their build scripts have changed a bit. In particular how ASM files are added, so we might need to make some changes.
The logic is in:
https://github.com/google/webcrypto.dart/blob/master/tool/update-boringssl.py
We need to bump
BORINGSSL_REVISION
.And then fix our build script generator logic, since
asm_outputs
parameter isn't give toWriteFiles
anymore.I tried playing with this, but we need to do more than simple remove the
asm_outputs
parameter.There is some documentation in: https://boringssl.googlesource.com/boringssl/+/refs/heads/master/INCORPORATING.md
The rest is reading BoringSSL code and build system 🤣
The text was updated successfully, but these errors were encountered: