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

Compilation fails when cache directory exists (sometimes) #1094

Closed
agross opened this issue Mar 2, 2017 · 7 comments
Closed

Compilation fails when cache directory exists (sometimes) #1094

agross opened this issue Mar 2, 2017 · 7 comments
Milestone

Comments

@agross
Copy link
Contributor

@agross agross commented Mar 2, 2017

Nanoc compilation sometimes fails unless you rm -rf tmp.

Expected behavior

Compilation succeeds.

Actual behavior

Compilation fails.

Details

The item reported in the log below is passed-through, no filters and such. Note the path pointing to a temporary file.

Crash log

https://gist.github.com/agross/9831ab6eb3844783424b19f64bd7831c

@ddfreyne
Copy link
Member

@ddfreyne ddfreyne commented Mar 3, 2017

I’m unsure how to reproduce this. Can you try whether the issue also exists on master, and if so, could you provide a reproduction (a small one is preferred, but not required).

@agross
Copy link
Contributor Author

@agross agross commented Mar 3, 2017

I'll try to. I think I can reproduce it locally pretty reliably, but the project is rather complex.

@sunshineco
Copy link
Contributor

@sunshineco sunshineco commented Mar 4, 2017

This failure looks remarkably similar to the failure mentioned tangentially in #1082. I came up with a reproduction recipe, but it only manifested when some in-progress patches from Denis were applied. However, if the underlying cause is the same, perhaps those patches and the recipe I provided could help track down the problem (assuming it still manifests on master).

@ddfreyne
Copy link
Member

@ddfreyne ddfreyne commented Mar 4, 2017

Any reproductions that work with master would be incredibly valuable!

I know that @gpakosz is also experiencing this issue.

@ddfreyne
Copy link
Member

@ddfreyne ddfreyne commented Mar 5, 2017

I have a reproduction case!

@ddfreyne
Copy link
Member

@ddfreyne ddfreyne commented Mar 5, 2017

A (WIP) fix is in #1109.

@ddfreyne ddfreyne added this to the 4.6.3 milestone Mar 5, 2017
@ddfreyne ddfreyne added this to the 4.6.3 milestone Mar 5, 2017
ddfreyne added a commit that referenced this issue Mar 5, 2017
Do not reuse cached binary content
@agross
Copy link
Contributor Author

@agross agross commented Mar 5, 2017

Works for me! Many thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants