[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 29 May 2013 11:55:45 +0530
From: Sayantan Datta <std2048@...il.com>
To: john-dev@...ts.openwall.com
Subject: Re: pre-commit hooks (was: Unify internal form of mscash2 hashes?)
Hi ,
On Wed, May 29, 2013 at 11:38 AM, magnum <john.magnum@...hmail.com> wrote:
> BTW that latter line probably needs -r:
>
> sed -ri 's/[[:space::]]+$//' "$FILE"
>
> Another problem with your script is that is will bug out if a file name
> contains spaces. To fix that you need to change IFS. But I (and many much
> more clever guys than me) still recommend to use a hook that just warns,
> not tries to fix anything.
>
As you said, I configured my editor to remove white spaces whenever it
opens a file. Much easier indeed!!
Regards,
Sayantan
Content of type "text/html" skipped
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
