gumyr / cq_warehouse

A cadquery parametric part collection
Apache License 2.0
112 stars 24 forks source link

Patch Script Instructions incorrect for Windows #43

Closed gumyr closed 2 years ago

gumyr commented 2 years ago

The -p parameter is dependent on where the OS and needs to be calculated.

gumyr commented 2 years ago

Fixed with this commit: 5a49b5ef6be8026d2b876004d2889feff9363393

The path depth is calculated from the patch file and shown in the user instructions. A "-l" option is added to cope with CR/LF issues.

Not tested on Windows.

marked23 commented 2 years ago

54

Attempts to address this by formatting all the paths in the patch file as forward slashes... with the drive letter after the first slash.

gumyr commented 2 years ago

Fixed with PR54.