SPECS: qt4.spec - adapter
Radoslaw Zielinski
radek at pld-linux.org
Thu Jun 21 10:48:20 CEST 2007
Rafał Cygnarowski <zswi at pers.pl> [21-06-2007 10:14]:
> Dnia środa, 20 czerwca 2007, glen napisał:
[...]
>> %files -n QtCore-devel -f QtCore-devel.files
>> +%defattr(644,root,root,755)
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> This change is wrong.
Why do you think so?
> Qt*-devel.files already contains this and it's
> respected. If this is done by adapter, then adapter needs to be fixed.
> Remove these lines, please.
All %files sections should begin with %defattr; no exceptions. This
cuts down the time needed to wonder if it's needed here and just missing
or defined somewhere else... Consider it a policy. Benefit: less bugs,
saved time.
--
Radosław Zieliński <radek at pld-linux.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : /mailman/pipermail/pld-devel-en/attachments/20070621/3b192394/attachment.sig
More information about the pld-devel-en
mailing list