Bug 428101
Summary: | BackupPC should depend on par2cmdline | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Joe Bayes <jbayes> |
Component: | BackupPC | Assignee: | Johan Cwiklinski <fedora> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | low | Docs Contact: | |
Priority: | low | ||
Version: | 8 | ||
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-07-27 22:51:38 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Joe Bayes
2008-01-09 08:21:14 UTC
The default configuration file has 'par' disabled ($Conf{ArchivePar} = 0;). As it is not enabled by default at install time, I'm not sure including it as a Require is a good thing. Also, with this default configuration, using web interface to change configuration works correctly. Well, if I change the par2 config line in the web interface (under the server tab) from /usr/bin/par2 to /usr/bin/qux (which does not exist) and click "save", the page says Error: No save due to errors Error: ParPath must be a valid executable path and config.pl is unmodified. This is a problem because the user may have changed other, more important settings that will not get saved. I suppose the user could work around it by typing /bin/false (or any other executable) into the par2 configuration line, as long as they don't plan on using the ArchivePar feature. The Right Thing To Do is probably to have the web interface not require a working par2 unless ArchivePar==1. I'll try to reproduce these error and see what I could do ;) $Conf{ParPath} now defaults to null (not sure it was previously the case). If you put it to a path that does not exists, simply blanking the input should allow you to store config.pl. If problem is still present, feel free to reopen this bug. It works now. Thanks. |