Hi Eygene, On Mon, Apr 14, 2008 at 07:44:13PM +0400, Eygene Ryabinkin wrote: > [...] > Testing and feedback are more than welcome. I didn't test your patch but I have a have a few comments about it: In install_new_file(), I don't think you should test for $CMP being an executable file... It is in the base system and the rule of thumb in other rc.d scripts is to use those directly. I'm not sure you should chown/chmod the forwarders file. People may have custom setup that you should not interfere with without a good reason. Also, I would rather let add_new_bind_forwarders() build the "empty" forwarders file, it would make more sense IMHO. You could then put a single call to add_new_bind_forwarders() at the end of the script under a $resolv_build_named_forwarders condition. It makes more sense indeed to test this outside of the function, my personal feeling being that it makes the reading less puzzling. Anyway, thank you very much for your work. I think many people will enjoy it once it will hit the source tree. Best regards, -- Jeremie Le Hen < jeremie at le-hen dot org >< ttz at chchile dot org >Received on Wed Apr 23 2008 - 17:09:28 UTC
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:30 UTC