search

putnam / binmerge

Tool to merge multiple bin/cue tracks into one. Great for redump.
GNU General Public License v2.0
340 stars 44 forks source link

Invalid escape sequence \d #18

Closed AlexMax closed 10 months ago

AlexMax commented 10 months ago

I'm getting this message when attempting to run master with Python 3.12.1 on Windows

PS ...> binmerge.py
...\binmerge.py:122: SyntaxWarning: invalid escape sequence '\d'
  m = re.search('TRACK (\d+) ([^\s]*)', line)
...\binmerge.py:128: SyntaxWarning: invalid escape sequence '\d'
  m = re.search('INDEX (\d+) (\d+:\d+:\d+)', line)
...\binmerge.py:169: SyntaxWarning: invalid escape sequence '\d'
  m = re.match("(\d+):(\d+):(\d+)", stamp)
usage: binmerge.py [-h] [-l] [-v] [-s] [-o OUTDIR] cuefile basename
binmerge.py: error: the following arguments are required: cuefile, basename

Seems like some of these strings are missing r prefixes.

putnam commented 10 months ago

Thanks for the report. Looks like 3.12 raised this warning, which was previously a (hidden) DeprecationWarning.

The app should run fine regardless, but I will fix those regex strings.

putnam commented 10 months ago

Fixed in 5576322