[pacman-dev] [PATCHES] Refreshed PKGBUILDs
Travis Willard
travis at archlinux.org
Sat May 17 16:22:01 EDT 2008
On Sat, May 17, 2008 at 3:54 PM, Xavier <shiningxc at gmail.com> wrote:
> Aaron Griffin wrote:
>> On Sat, May 17, 2008 at 10:28 AM, Geoffroy Carrier
>> <geoffroy.carrier at koon.fr> wrote:
>>>>> - Still, lightweight means smater
>>>>> => Don't use
>>>>> "$_svntrunk" ${something}
>>>>> instead of
>>>>> $_svntrunk $something
>>>>>
>>>> Where exactly is this change?
>>> This was an example of the rule I wanted to follow... ie. i did not
>>> introduced quotes of $_svntrunk which obviously won't include spaces.
>>> (or it's REAAALLY dumb).
>>
>> Just to be a tad more technical. I always try to use the ${} syntax
>> when I can. It IS more to type, but prevents errors much more and is a
>> good habit to follow. It's a minor thing, but I prefer that syntax. I
>> don't think we should really enforce it either way
>>
>
> That's interesting. We don't really enforce anything with the prototypes
> but it's good to show the preferred syntax with them.
>
> I think $pkgname and $pkgver should be safe though, right? They
> shouldn't contain spaces or other weird things so don't require
> additional quotings. But pkgdir, srcdir and startdir do.
>
> So maybe instead of :
> cd "$srcdir/$pkgname-$pkgver"
> we could use :
> cd ${srcdir}/$pkgname-$pkgver
Well, ${srcdir} still needs the quotes.
"${srcdir}/$pkgname-$pkgver"
More information about the pacman-dev
mailing list