-
Notifications
You must be signed in to change notification settings - Fork 142
Conversation
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.
@palytoxin: thanks for the contribution, appreciate your trying to make a build.
You should always test the PR by running ./build package-docker archlinux-amd64 WKHTMLTOPDF-SRC-DIR
and testing that the generated package works on your machine.
|
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.
Looks good! Were you able to install the packages? I got an error in a clean docker environment:
error: failed to commit transaction (conflicting files)
wkhtmltox: /usr/local/share/man exists in filesystem (owned by filesystem)
Errors occurred, no packages were upgraded.
I don't use ArchLinux, don't know if you can still proceed with this?
@ashkulz Can you merge this pr if it works fine? |
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.
Looks good, thanks!
Thanks for the contribution, @palytoxin! |
Added to the downloads page 🎉 |
@foutrelis you might want to use this package in |
I'm trying to create archlinux build