Skip to content

Conversation

@nicola-lunghi
Copy link
Contributor

Add arm build to use this on mac with M1 chip.
Also add the alpine version to all targets and automatically get the build list from the buildx cofnig in the ci

@sonarqubecloud
Copy link

Copy link
Collaborator

@shenxianpeng shenxianpeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it needs to add too much code here

https://github.com/cpp-linter/clang-tools-docker/pull/102/files#diff-870f6fe23fc034f008f5203ee1e628d7bfa65a49095d5a4688db498328449ed2R21-R202

It should be something like

tgt = ["16-alpine", "17-alpine", "18-alpine", "19-alpine", "20-alpine"]

@shenxianpeng shenxianpeng added the enhancement New feature or request label Aug 14, 2025
@shenxianpeng
Copy link
Collaborator

build:
needs: extract-targets
runs-on: ubuntu-latest
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
runs-on: ubuntu-latest

strategy:
matrix:
target: [ "all", "20", "20-alpine", "19", "19-alpine", "18", "18-alpine", "17", "17-alpine", "16", "16-alpine", "15", "14", "13", "12", "11", "10", "9", "8", "7"]
target: ${{ fromJson(needs.extract-targets.outputs.targets) }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this idea.

@shenxianpeng
Copy link
Collaborator

Since I’ve added arm64 support and some Alpine images #101, it might be a good idea to start working from the master branch.

@nicola-lunghi
Copy link
Contributor Author

nicola-lunghi commented Aug 14, 2025

Ok thanks I'll have a look
that was actually my main issue. Do you want me just to add the CI part?

@nicola-lunghi
Copy link
Contributor Author

I'll close this and add another PR for the CI. I think you already implemented the rest on master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants