RFR: Convert path separators in patch file to unix explicitly
Erik Duveblad via github.com
duke at openjdk.java.net
Thu Aug 22 11:50:04 UTC 2019
On Thu, 1 Aug 2019 19:30:30 GMT, JornVernee via github.com <duke at openjdk.java.net> wrote:
> I ran into trouble when trying to apply a patch file generated with Windows Git to a mercurial repository through cygwin. It wasn't recognizing the files since the paths in the patch file were using Windows path separators.
> The spec doesn't seem to mention path separators: https://www.gnu.org/software/diffutils/manual/html_node/Detailed-Unified.html#Detailed-Unified
> But, both the Windows Git and Mercurial clients I have installed are handling unix path separators just fine. So, in this PR I've changed the patch file generators to use unix path separators explicitly, which solves the problem.
> - cefd6ae1: Convert path separators in patch file to unix explicitly
> Pull request:
> Fetch command:
> git fetch https://github.com/openjdk/skara.git pull/35/head:pull/35
This PR has been reviewed by Erik Duveblad via github.com - changes are approved. Review comment:
Sorry for taking so long to review this @JornVernee, the patch looks good! I executed `sh gradlew test` on Linux x86_64 and all tests passed!
More information about the skara-dev