Re: [zypp-devel] Proposal of a new option to disable check for deleted files
On 09/24/2012 12:25 PM, Michael Andres wrote: > On Thursday 20 September 2012 10:54:19 Andrea Turrini wrote: >> Now, since this is the first time I look at zypper code, I know that >> my changes have to be improved before submitting them, so some >> question follows: >> - do you think it is reasonable to have the variable in the conf file? > > Frankly, I wonder if it actually needs more then a .conf file option. I doubt > that there is much need to turn this on and off on the command line. Since I think it is safer to have the check for deleted files enabled (even if I will disable it in my system by using the .conf file), I think a command line option may be useful when you do not want to disable "permanently" the check, as happens with the config variable, but just for some interaction and then to come back to the standard safer check. >> If so, is it ok to put it in the main section? > > I'd introduce an [install] or [commit] section for options like this. I added a [commit] section since the check is used mainly when updating/deleting packages. Attached you find the patch for the code as contained in the zypper-1.7.3-1.2.2.src.rpm from oS-12.2 source repository. Please tell me where to improve the code and then how to submit it since I never used git (since in the development page in the wiki I read it is used to manage zypper). Best, Andrea deletedFilesCheck.patch.gz Description: GNU Zip compressed data
Re: [zypp-devel] Proposal of a new option to disable check for deleted files
On Thursday 20 September 2012 10:54:19 Andrea Turrini wrote: > Now, since this is the first time I look at zypper code, I know that > my changes have to be improved before submitting them, so some > question follows: > - do you think it is reasonable to have the variable in the conf file? Frankly, I wonder if it actually needs more then a .conf file option. I doubt that there is much need to turn this on and off on the command line. > If so, is it ok to put it in the main section? I'd introduce an [install] or [commit] section for options like this. -- cu, Michael Andres +--+ Key fingerprint = 2DFA 5D73 18B1 E7EF A862 27AC 3FB8 9E3A 27C6 B0E4 +--+ Michael Andres SUSE LINUX Products GmbH, Development, m...@suse.de GF:Jeff Hawn,Jennifer Guild,Felix Imendörffer, HRB16746(AG Nürnberg) Maxfeldstrasse 5, D-90409 Nuernberg, Germany, ++49 (0)911 - 740 53-0 +--+ -- To unsubscribe, e-mail: zypp-devel+unsubscr...@opensuse.org To contact the owner, e-mail: zypp-devel+ow...@opensuse.org