[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: hello 2.1.91 pretest
From: |
Mark D. Baushke |
Subject: |
Re: hello 2.1.91 pretest |
Date: |
Mon, 21 Aug 2006 02:08:17 -0700 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Ralf Wildenhues <address@hidden> writes:
> Hello Mark,
>
> * Mark D. Baushke wrote on Mon, Aug 21, 2006 at 10:37:06AM CEST:
> > Ralf Wildenhues <address@hidden> writes:
> > >
> > > Suggested patch below. OK?
> >
> > I like your change as I think it makes it easier to understand.
>
> Thanks. I'll apply in a day or two if noone objects.
Thanks!
> > I suggest you add quotes around $module (i.e., "$module") to be
> > consistent with all of the other uses (see for example the case in
> > func_verify_nontests_module()).
>
> But the word after `case' is not split by the shell, so the quoting is
> not necessary here (just like the right hand side of an assignment).
> There are people who feel that quoting should be added anyway, for
> consistency, and so they would not have to remember this "exception".
> But I would say: it strikes _me_ as inconsistent, and I try to never
> quote the word after `case'. I've once been bitten by a
> case $foo
>
> where I silently had _expected_ word-splitting to take place (more
> precisely, I expected a TAB in `$foo' to be turned into a space), but
> which in fact doesn't. Since then I really frown upon quoting there:
> you don't need to, and not doing so consistently helps make me stay
> aware of this.
I have no problems with it either way. If you want to help it be
consistent, then you might want to go through the rest of the
gnulib-tool and change "$module" to $module where it makes sense.
Enjoy!
-- Mark
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.4 (FreeBSD)
iD8DBQFE6XgBCg7APGsDnFERAgsmAJ9ykeSQzNxD8nyoV290XSKruyJB3wCfSfoH
rgoj3q0idomWbtHgYxs37F8=
=tJS8
-----END PGP SIGNATURE-----
Re: hello 2.1.91 pretest, Karl Berry, 2006/08/21
Re: hello 2.1.91 pretest, Ralf Wildenhues, 2006/08/21
hello-2.1.91 warning, Bruno Haible, 2006/08/22
hello-2.1.91 compilation failure on MacOS X, Bruno Haible, 2006/08/22
hello-2.1.91 modernization, Bruno Haible, 2006/08/22