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

Move FreeBSD CI to Cirrus #43

Merged
merged 1 commit into from
Sep 1, 2022
Merged

Move FreeBSD CI to Cirrus #43

merged 1 commit into from
Sep 1, 2022

Conversation

giordano
Copy link
Member

I'm missing how to setup Cirrus for the repository/organisation. CC: @ararslan

@giordano giordano requested a review from ararslan August 31, 2022 22:34
@ararslan
Copy link
Member

ararslan commented Aug 31, 2022

Cirrus was like half-installed for this organization; I just had to edit the installation from the GitHub Marketplace to tell it whether it's allowed to access all repos or just selected ones. Should be all set now.

As an aside, we could also run CI on platforms like ARM and Alpine with Cirrus "soon".

@giordano
Copy link
Member Author

Ok, I can hold off this PR until we can move also away from Drone. Thanks!

@giordano giordano marked this pull request as draft August 31, 2022 23:04
@ararslan
Copy link
Member

I'd recommend just going ahead with this PR and adding other platforms on Cirrus once I have it nailed down. The green status checks in my PR I linked are a bit misleading as CI for that repo doesn't cover the mechanism by which other repos install CirrusCI.jl. (sh -c "$(fetch ...)" works fine for FreeBSD but fetch is FreeBSD-specific and I don't want people to have to separate out the installation step into a matrix just so they can install curl or whatever a bunch of different ways. If you have thoughts or ideas on that front, feel free to comment on that PR!)

@giordano giordano closed this Aug 31, 2022
@giordano giordano reopened this Aug 31, 2022
@giordano
Copy link
Member Author

Cirrus still doesn't start here.

@ararslan
Copy link
Member

Maybe because it's in draft mode?

@giordano giordano marked this pull request as ready for review August 31, 2022 23:46
@giordano giordano closed this Aug 31, 2022
@giordano giordano reopened this Aug 31, 2022
@ararslan
Copy link
Member

wtf, https://cirrus-ci.com/github/JuliaPackaging only shows BinaryProvider.

@ararslan
Copy link
Member

I changed the org settings for Cirrus from access to all repos to only this one, no idea whether that will do anything

@giordano
Copy link
Member Author

giordano commented Sep 1, 2022

Sounds like I had to push a new commit, I guess it doesn't rebuild on a new merge commit (when you close/reopen the PR)?

@ararslan
Copy link
Member

ararslan commented Sep 1, 2022

Huh, I guess not, weird. I wonder if that's configurable.

@giordano giordano merged commit cb51595 into master Sep 1, 2022
@giordano giordano deleted the mg/cirrus-ci branch September 1, 2022 16:08
contra-bit pushed a commit to contra-bit/JLLWrappers.jl that referenced this pull request Jul 10, 2024
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