For the most part, this looks good but I do have a few questions: 1. Could you please clarify how we can confirm that your patch fixes the problem? 2. In your debdiff, you don't follow the existing indentation sometimes. 3. Is it necessary for you to remove attribution comments at the top of tools/slapd.backup? 4. In the same file, you remove DEFAULTS="/etc/default/slapd" but I see that variable is referenced later in the file. Is this intentional? 5. In the same file, you seem to blow away an entire function and replace it with a few lines which does not set IFS to EOL like the original script did. Can you confirm that this intended and/or required?
-- Current eBox packages in intrepid don't work at all https://bugs.launchpad.net/bugs/273486 You received this bug notification because you are a member of Ubuntu Server Team, which is a direct subscriber. -- Ubuntu-server-bugs mailing list Ubuntu-server-bugs@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-server-bugs
