search

useblocks / sphinx-simplepdf

A simple PDF builder for Sphinx documentations
https://sphinx-simplepdf.readthedocs.io
MIT License
35 stars 17 forks source link

Adding retry option if weasyprint process gets timed out #70

Closed kreuzberger closed 1 year ago

kreuzberger commented 1 year ago

Related to #48 we have problems to get it stable running in your CI System on windows. This affects all up to the newest version of weasyprint. Workaround by limiting the number of process to one for sphinx_simplepdf builds limits us in CI speed and also leads to failed builds sometimes.

So i decided to add a retry configuration option. This could help us in case of timeouts in the pdf generation to retry. I currently use 3 times and builds are not broken any more.

This only works when using weasyprint as executable, not as module. But only in this case the timeout option is also usable.

kreuzberger commented 1 year ago

Severe Error in the PR: Using the default configuration value for retries 0, simplepdf never outputs an pdf without any hint. I think setting retry default value to 1 should fix the problem or setting the range value to retries + 1.