Re: Are brackets around macros required?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am 15.08.2011 19:12, schrieb Rex Dieter:
> On 08/15/2011 12:02 PM, Göran Uddeborg wrote:
>> In a package review recently I was asked to put brackets around all
>> macro usages.  That is, writing
>>    %{name}.desktop
>> rather than
>>    %name.desktop
> 
> imho, using brackets is near-universally safer, but I doubt we need to 
> mandate it.  If anyone's non-use of brackets ever does break anything, 
> that's their responsibility to fix I suppose.
> 
I'm the package reviewer who has critizized the missing brackets [1].

First, it is usual in almost all packages to use them. However, that's
not reason enough to make them mandatory. Actually, I wrote about the
readability. I'm using Gedit for writing and viewing the spec files,
which uses gtksourceview for syntax highlighting. Gedit doesn't
recognize any macros without brackets and doesn't highlight them. That's
my experience regarding the readability, and I would strongly recommend
to use the brackets. Either mandatory, or as a fully-or-nowhere choice,
means, either always in a certain spec file, or nowhere.

Best Regards,
Mario

[1] https://bugzilla.redhat.com/show_bug.cgi?id=730426
--
packaging mailing list
packaging@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/packaging



[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite Forum]     [KDE Users]

  Powered by Linux