return parallel_{read,write}_safe true for sphinx #376
Conversation
see #141 on the few projects I have locally it works fine with this change and calling sphinx with -j auto I haven't examined all source code, this needs further testing
|
I must admit that although I have done basic "logic checking" in my head and have run tests on several projects, including the very large zephyrproject-rtos/zephyr#6823, I cannot be 100% certain that all possibilities and edge cases will work. Personally I think it is better to merge this now and perhaps deal with bug reports for edge cases later, especially because the issue has been open since end 2014. |
|
@michaeljones what do you think of @melvinvermeeren's proposal? (to merge and solve for hedge cases if we find some later) I am travelling this week, but I can check this branch against the QuantStack projects when I am back. |
|
Once other open PRs are handled and I have done some issue tracker cleanup with perhaps minor fixes I intend on releasing the next version of Breathe. In case there are no approvals or objections on this PR I will most likely merge this before that release. |
vermeeren
added
enhancement
|
Will be releasing 4.8.0 in a bit. A bit over a third of the issues has been checked at this point. Merging this in time for 4.8.0. |
…nx-j-option
return parallel_{read,write}_safe true for sphinx
closes #141
on the few projects I have locally it works fine with this change
and calling sphinx with -j auto
I haven't examined all source code, this needs further testing