PR#115: Get rid of SyntaxWarning: invalid escape sequence '\d' - centos/centpkg

centos / centpkg

#115 Get rid of SyntaxWarning: invalid escape sequence '\d'

Merged 8 days ago by tdawson. Opened 9 days ago by churchyard.

centos/ churchyard/centpkg escape-sequence into develop

Get rid of SyntaxWarning: invalid escape sequence '\d'

Miro Hrončok • 8 days ago

d624da

src/centpkg/__init__.py

file modified

+1 -1

		`@@ -171,7 +171,7 @@`
		`# For MD5 we want to use the old format of source files, the BSD format`
		`# should only be used when configured for SHA512`
		`self.source_entry_type = "bsd" if self.lookasidehash != "md5" else "old"`
		`- self.branchre = "c\d{1,}(s)?(tream)?\|master"`
		`+ self.branchre = r"c\d{1,}(s)?(tream)?\|master"`

		`@property`
		`def distgitdir(self):`

no initial comment

tdawson commented 9 days ago

Could you give an example of when this is happening, so I can see that it is fixed.

churchyard commented 9 days ago

Every time on Feodra 41+

$ centpkg 
/usr/lib/python3.13/site-packages/centpkg/__init__.py:174: SyntaxWarning: invalid escape sequence '\d'
  self.branchre = "c\d{1,}(s)?(tream)?|master"
usage: centpkg [-h] ...

tdawson commented 9 days ago

I saw this once on rawhide, then did a system update, and I'm not seeing that error again.
But on the other hand, this isn't causing any harm on any of the machines I tested it on.
So I'm ok merging it.