Skip to content
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

Fixing dependencies of gallery RSS generation. #1633

Merged
merged 1 commit into from Mar 9, 2015
Merged

Conversation

@felixfontein
Copy link
Contributor

@felixfontein felixfontein commented Mar 8, 2015

The current gallery RSS rendering task requires the input images and the generated thumbnails to be there, but not the generated output images. When the task is executed, it tries to retrieve the size of the output images (which are copied from the input images).
When running Nikola in parallel mode, this can yield errors when the RSS generator tries to access a file which hasn't been copied to the destination yet.
This PR fixes the dependencies of the RSS generator task to require the output images to be there.

felixfontein added a commit that referenced this pull request Mar 9, 2015
Fixing dependencies of gallery RSS generation.
@felixfontein felixfontein merged commit 1cf4899 into master Mar 9, 2015
3 checks passed
3 checks passed
continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@felixfontein felixfontein deleted the gallery-rss-fix branch Mar 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant