-
Notifications
You must be signed in to change notification settings - Fork 699
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
{tools}[dummy/] wkhtmltopdf v0.12.3 #3098
{tools}[dummy/] wkhtmltopdf v0.12.3 #3098
Conversation
Test report by @boegel |
lgtm |
this one has many system deps: http://wkhtmltopdf.org/downloads.html
|
@pescobar should we list them in |
@boegel I added os deps for redhat/ubuntu |
source_urls = ['http://download.gna.org/wkhtmltopdf/%(version_major_minor)s/%(version)s/'] | ||
sources = ['wkhtmltox-%(version)s_linux-generic-amd64.tar.xz'] | ||
|
||
osdependencies = [('zlib', 'zlib1g'), 'fontconfig', ('freetype', 'libfreetype6'), ('libX11', 'libx11-6'), ('libXext', 'libxext6'), ('libXrender', 'libxrender1')] |
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.
@pescobar line too long, let's go with one per line?
osdependencies = [
('zlib', 'zlib1g'),
'fontconfig',
('freetype', 'libfreetype6'),
('libX11', 'libx11-6'),
('libXext', 'libxext6'),
('libXrender', 'libxrender1'),
]
@pescobar no need to close/open on every push you do, Travis is picking up new commits correctly now? |
lgtm |
Test report by @boegel |
Going in, thanks @pescobar! |
(created using
eb --new-pr
)