Skip to content

Commit

Permalink
Merge pull request #148 from coopdevs/fix/wkhtmltox_version_by_distro
Browse files Browse the repository at this point in the history
fix: conditional wkhtmltox version
  • Loading branch information
oyale authored Jan 31, 2023
2 parents e62751d + eb34689 commit dfdb46d
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
5 changes: 5 additions & 0 deletions defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -111,3 +111,8 @@ odoo_role_logrotate_odoo:
compress
delaycompress
}
# wkhtmltopdf version
wkhtmltox_version_jammy: "0.12.6.1-2"
wkhtmltox_version_bionic: "0.12.6-1"
wkhtmltox_version: "{% if ansible_distribution_release in ['jammy', 'bullseye'] %}{{ wkhtmltox_version_jammy }}{% else %}{{wkhtmltox_version_bionic}}{% endif %}"
2 changes: 1 addition & 1 deletion tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@

- name: Download and install wkhtmltopdf only if not already present at any version
apt:
deb: "https://github.com/wkhtmltopdf/packaging/releases/download/0.12.6.1-2/wkhtmltox_0.12.6.1-2.{{ ansible_distribution_release }}_amd64.deb"
deb: "https://github.com/wkhtmltopdf/packaging/releases/download/{{ wkhtmltox_version }}/wkhtmltox_{{ wkhtmltox_version }}.{{ ansible_distribution_release }}_amd64.deb"
when: wkhtmltox_installed.rc == 1

- import_tasks: pyenv.yml
Expand Down

0 comments on commit dfdb46d

Please # to comment.