Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=570424

--- Comment #11 from Satya Komaragiri <[email protected]> 2010-04-07 
07:05:10 EDT ---
> Satya, the mentioned guideline just allows you to omit the BuildRoot-tag at 
> the
> beginning of the file. This is not the same as the %{buildroot} macro or the
> $RPM_BUILD_ROOT variable. rpmlint still claims, that the 'rm -rf %{buildroot}'
> command at the beginning of the %install section is missing. It wouldn't if it
> really was obsolete. Please add it before requesting CVS access. :)
> 
Corrected
> 
> Thanks very much for correcting the mentioned isssues and for your patience.
> I'm sorry I was a bit stressed the last weeks and had lot of things to do at
> work, which caused my delayed answer to this issue. Except the mentioned issue
> with the %install section, your package looks good now and is APPROVED.
> Remember to add the 'rm -rf %{buildroot}' to the %install section before
> requesting the CVS access.    

No problem :) Thanks a lot for reviewing!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to