Skip to content

add alpine linux compatibility #940

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

Closed
wants to merge 1 commit into from

Conversation

ajbt200128
Copy link

Hey y'all, this adds support for alpine when running this action.

support running action on alpine linux
@smorimoto
Copy link
Member

Adding features that are too specialized for the various distributions used by self-hosted runners is out-of-scope for this action. The installation of system packages can be accomplished by running in the previous step of this action.

@smorimoto smorimoto closed this Mar 5, 2025
@ajbt200128
Copy link
Author

@smorimoto do you mind if I create an issue then? This action will always run apt to install packages if the OS is linux, making it unusable on self hosted runners

@ajbt200128
Copy link
Author

(I also don't mind fixing this myself)

@ajbt200128
Copy link
Author

ajbt200128 commented Mar 20, 2025

this example doesn't work if the container isn't ubuntu/debian, as when initializeOpam is called, if the container is linux, it will always try to use apt

@smorimoto
Copy link
Member

@ajbt200128 The fix is ​​included in the latest release.

@ajbt200128
Copy link
Author

thanks!

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