Open melissalinkert opened 5 months ago
Two quick thoughts:
As discussed earlier today, the next steps here are:
- keep the exception by default, but add an option to warn instead
- make the readme changes a separate section (so we can link to it), and explain the issue a little more
- delete the "check" file right away instead of on exit
- include in a 0.10.0 (minor, not patch) release and make sure release notes flag this as a potentially breaking change
All except the actual release should be covered by 9dca8a9, cc2ec93, and 862e4ac.
Updates readme to reference the known
noexec
issue, and tries to detect and fail conversion quickly if that is the case. The check is very heavy-handed, as it should always fail ifjava.io.tmpdir
isnoexec
- independent of the input format or conversion options. We could make that more lenient if any strong opinions.