-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: add Archlinux pacman support (#31) #95
Conversation
* add `setupPacmanPack` * add arch linux docker image * add container tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow. Thanks for the work!
Edit: fixed it!
- Could you run
pnpm format
? Some spelling errors need to be addressed by adding those words tocspell.config.yaml
. -> 895aaca
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Edit: fixed it!
Fortunately, everything passes the docker test except for Doxygen. We can fix or ignore it. Do you know how to fix the issue? What version of Doxygen is available?
Installing doxygen 1.9.4 via pacman
error: target not found: doxygen=1.9.4
error: target not found: doxygen1.9.4
Error: Command failed with exit code 1: pacman -S --noconfirm doxygen1.9.4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the work! This is awesome!
This contains the essential installations. The rest of the apps can be added incrementally. For now, I think this is ready to ship!
setupPacmanPack
ADD
toCOPY
- https://www.geeksforgeeks.org/difference-between-the-copy-and-add-commands-in-a-dockerfile/TODO:
cppcheck
anddoxygen
is not workling, yet.kcov