ok, finally I found 5 minutes to fix this. now the config GUI checks for validity of the regex, warns the user and doesn't add the regex if it does not compile. check if this try-except approach fits your programming style.
probably, since there is no logging facility (I read you are going to implement one, how is the progress?) also sbackupd should handle more gracefully bad regexs, probably printing a warning to stderr and going on ignoring the mispelled regex. regex are powerful but quite cryptic to newbies... since this aims at being a simple backup solution, it should be a bit tolerant, as long as it is not issuing meaningful error messages. let me know what are your views about this, I can work it out myself as well. best regards mc ** Attachment added: "patch for regex check" http://launchpadlibrarian.net/12091586/diff-simple-backup-config.py ** Changed in: sbackup (Ubuntu) Status: Confirmed => Fix Committed -- simple-backup-config does not check for validity of regex exclusion rules https://bugs.launchpad.net/bugs/191127 You received this bug notification because you are a member of Ubuntu Bugs, which is the bug contact for Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
