[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
** Changed in: nssbackup Status: Fix Committed = Fix Released -- 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 subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
** Changed in: sbackup (Ubuntu) Status: Fix Committed = Fix Released -- 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 subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
** Changed in: nssbackup Status: Confirmed = Fix Committed Target: None = release0.2 -- 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 subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
This patch is included in 0.10.5. ** Changed in: sbackup Status: Confirmed = Fix Released -- 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 subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
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
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
sorry, I imagine this is not the proper place to ask, but my knowledge of launchpad is superficial at best. I realy like this software, is simple but effective, and I'd like to help, as it seems simple enough for me to give side contributions. I have noticed right now that it have been forked to a new version, so I'd like to know where I should put my efforts, on sbackup or on nssbackup, and in case whether it is better I submit patches to the current version or I download the code from the development trunk and I submit diffs to that code. best regards mc -- 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
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
Hello, Thank you for your patch. I appreciate a lot your motivation and help. I don't maintain Sbackup (at least for now) anymore but as you have submitted a patch, it would be nice to have it included in Ubuntu for other users. There is a plan to re-merge Sbackup and NSsbackup though. But that won't be for Hardy. So it's better having Ubuntu-dev patching Sbackup 0.10.4. As for Launchpad usage, by submitting your patch here, since I have set the Bug for all projects (NSsbackup, Sbackup Upstream and Sbackup (Ubuntu) ), the patch is available for all projects and all developers have been notified. As for the plan on Implementing logging in the software, it has been done on NSsbackup (information available on https://edge.launchpad.net/nssbackup/ ). You can easily install NSsbackup by Using NSsbackup-Team PPA ( https://edge.launchpad.net /~nssbackup-team/+archive ). Take the gutsy version (0.1-6) if you want something stable. Version 0.2 is the development version. I'll review you patch and add it to NSsbackup (I must admit that I was thinking that I've already fixed this bug :-( ). Best regards wattazoum -- 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
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
Hello, Thank you for this bug report. NSsbackup has indeed the same problem. About your question, I recommend you post a patch here and upstream. But I don't think the upstream project is maintained anymore. You'll have more chance of having your patch in Ubuntu by posting it here. Best regards wattazoum ** Also affects: nssbackup Importance: Undecided Status: New ** Changed in: nssbackup Importance: Undecided = High Assignee: (unassigned) = Oumar Aziz OUATTARA (wattazoum) Status: New = Confirmed -- 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
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
I confirm the Bug for NSsbackup also. ** Changed in: sbackup (Ubuntu) Status: New = Confirmed -- 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
[Bug 191127] Re: simple-backup-config does not check for validity of regex exclusion rules
oops, I meant Sbackup ** Also affects: sbackup Importance: Undecided Status: New ** Changed in: sbackup Status: New = Confirmed -- 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
