[patch] Match pattern is case-sensitive

[manu.blot@…]

from branches/experimental/trac-0.11

When adding a match pattern for an OS, for example, Bitten is case-sensitive, which is a bit of a hassle to match an OS, for example:

• Linux is a match
• linux is not a match

[anatoly techtonik <techtonik@…>]

No test cases yet. Can't run them out of the box. Need pointer to instructions how to run tests from checkout.

bitten/queue.py

@@ -184 +184 @@
                 for propname, pattern in ifilter(None, platform.rules):
                     try:
                         propvalue = properties.get(propname)
-                        if not propvalue or not re.match(pattern, propvalue):
+                        if (not propvalue or not re.match(pattern,
+                                                          propvalue, re.I)):
                             match = False
                             break
                     except re.error:

[osimons]

$> python setup.py test

[anatoly techtonik <techtonik@…>]

Added test case.

[osimons]

Thanks! Committed in [993] (trunk) and merged to 0.6.1dev in [994].

[anatoly techtonik <techtonik@…>]

Nice, but I can't close the ticket. There is only "new" option.

[osimons]

Heh, I forgot to close :-)