Hi Nathan Nice to hear from you. Yes this looks like a good patch. Some comments/questions below.
On Mon, Nov 03, 2008 at 05:29:18PM -0600, Nathan Handler wrote: ... > In Ubuntu, we've applied the attached patch to achieve the following: > > * etc/bash_completion.d/vzctl.sh.in: > Use "/bin/ls" instead of "ls", so an existing alias for "ls" > does not get used. This sounds like a good patch. However I find it unlikely that root have any aliases. But it could of course be so. How critical did you find this one? > * src/lib/ub.c: Fix typo: s/successefully/successfully/ Ok good. I assume that this is not a critical thing. > * src/lib/config.c, src/lib/util.c: Explicitly include limits.h What is the reason for this? I assume that it helps you to compile on some platform but I want to check first. > We thought you might be interested in doing the same. Yes very interesting. However I think you are aware that we are close to release which makes me reluctant to do any changes right now. They will be applied after the release of lenny though. One more question. Have you submitted this upstream, or do you want me to do that? Best regards, // Ola -- --------------------- Ola Lundqvist --------------------------- / [EMAIL PROTECTED] Annebergsslingan 37 \ | [EMAIL PROTECTED] 654 65 KARLSTAD | | http://inguza.com/ +46 (0)70-332 1551 | \ gpg/f.p.: 7090 A92B 18FE 7994 0C36 4FE4 18A1 B1CF 0FE5 3DD9 / --------------------------------------------------------------- -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
