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

[NOMERGE] Test macOS CI workflow without Roswell #53

Closed
wants to merge 1 commit into from

Conversation

ak-coram
Copy link
Owner

No description provided.

@ak-coram ak-coram force-pushed the macos-no-roswell branch 4 times, most recently from 2512816 to 5f7ba4d Compare September 17, 2023 15:29
@ak-coram ak-coram force-pushed the macos-no-roswell branch 18 times, most recently from 08a097c to 67bc91c Compare September 23, 2023 08:55
@catap
Copy link
Collaborator

catap commented Sep 23, 2023

FYI: ivmai/bdwgc#576

@catap
Copy link
Collaborator

catap commented Sep 25, 2023

@catap
Copy link
Collaborator

catap commented Sep 25, 2023

This fix allows to pass CI a few time in the row

Co-authored-by: Kirill A. Korinsky <kirill@korins.ky>
@catap
Copy link
Collaborator

catap commented Sep 25, 2023

Fix inside ecl develop branch... Updated the code.

@catap
Copy link
Collaborator

catap commented Sep 25, 2023

Seems quite stable now. Thanks to discover how to reproduce it. I do have encountered it before but haven't able to reproduce it.

@ak-coram
Copy link
Owner Author

@catap thanks for the investigation and the fixes! Let me know if you still need this PR, otherwise I'll close it.

@catap
Copy link
Collaborator

catap commented Sep 25, 2023

The fix were merged into upstream => nothing to do with that on my side.

but it is sadly that such issue were discovered after release and to be fixed in the released version we must wait quite a long time :(

@ak-coram
Copy link
Owner Author

but it is sadly that such issue were discovered after release and to be fixed in the released version we must wait quite a long time :(

@catap I agree, hopefully it's not two and a half years this time :(

@ak-coram ak-coram closed this Sep 26, 2023
@catap catap deleted the macos-no-roswell branch September 26, 2023 09:07
@catap
Copy link
Collaborator

catap commented Sep 26, 2023

who knows. BTW I've created patch which was merged on 23.9.9 branch which allows me to easy backport it to MacPorts: macports/macports-ports#20581

So, technically you may use https://gitlab.com/embeddable-common-lisp/ecl/-/commit/e8c9355d31e15d15900416e9925db56c54c9f265 as root and enjoy ECL-23.9.9 with that patch at your CI.

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