Skip to content

Update ubuntu for arm compiler crashes#151

Merged
clux merged 7 commits intomainfrom
try-update
Feb 24, 2025
Merged

Update ubuntu for arm compiler crashes#151
clux merged 7 commits intomainfrom
try-update

Conversation

@clux
Copy link
Owner

@clux clux commented Feb 5, 2025

red CI on arm with compiler errors and core dumps for 3 weeks. resolved by:

The only big change here should curl removal, which at this point SHOULD be reasonable. I doubt many people use curl crate over hyper at this point, and if you are, you probably shouldn't. Anyway, reach out on #96 if you disagree.

Signed-off-by: clux <sszynrae@gmail.com>
@SimenB
Copy link

SimenB commented Feb 22, 2025

Would love to see a fix here so we can use 1.85 and 2024 edition 🙏

clux added 5 commits February 24, 2025 09:21
Signed-off-by: clux <sszynrae@gmail.com>
also remove a (now-default) removed flag.

Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux
Copy link
Owner Author

clux commented Feb 24, 2025

it's been a weird one. it's failing only on arm and the error is literally a core dump or compiler error in the middle of compilation.

have gotten it mostly working locally by sidestepping it with an ubuntu upgrade, but curl is acting up. i might actually kill curl for now (also noted in #96 and no one has said anything, and i don't think it's heavily relied on). hopefully will get a release through with/without curl.

also add nicer separation of local dev and ci entrypoints

Signed-off-by: clux <sszynrae@gmail.com>
@clux clux changed the title try updating deps Update ubuntu for arm compiler crashes Feb 24, 2025
@clux clux marked this pull request as ready for review February 24, 2025 17:53
@clux
Copy link
Owner Author

clux commented Feb 24, 2025

Finally, green, merging as is. curl be damned.

@clux clux merged commit 69bcbd3 into main Feb 24, 2025
6 checks passed
@clux clux deleted the try-update branch February 24, 2025 17:56
@clux clux mentioned this pull request Feb 24, 2025
@SimenB
Copy link

SimenB commented Feb 28, 2025

thank you!!

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.

2 participants