Number regex doesn't work in get_highest_z()

[sej7278]

make_fcp_x3g wrapper script seems to use \d instead of [0-9] for digits which errors with:

WARNING: could not find highest Z coordinate. If this is a valid G-code file, the make_fcp_x3g script needs updating.

The fix is (PR seemed like overkill):

diff --git a/make_fcp_x3g b/make_fcp_x3g
index 108d41a..6bc219e 100755
--- a/make_fcp_x3g
+++ b/make_fcp_x3g
@@ -267,7 +267,7 @@ function get_highest_z
        # Extract highest Z value in a G1 command from the last 2048 lines of a file.
        local gcode=$1

-       tail -n 2048 "${gcode}" | grep -oE '^G1 [^;]*Z\d*\.?\d+' | \
+       tail -n 2048 "${gcode}" | grep -oE '^G1 [^;]*Z[0-9]*\.?[0-9]+' | \
            awk '{print $2}' | cut -b 2- | sort -g | tail -n 1
 }

[DrLex0]

Thanks for catching this. It seems this is a difference between the OS X version of grep (recognizes \d) and the one in Linux (doesn't).

Cool, GitHub has auto-closed this after I pushed my commit…

[sej7278]

yeah i've seen the osx vs linux thing before - you get the same problem with sed too.