[Bug 705587] Review Request: android-tools - Android platform tools (adb, fastboot, etc)

[bugzilla at redhat.com]

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=705587

--- Comment #8 from Volker Fröhlich <volker27 at gmx.at> 2011-08-26 16:27:18 EDT ---
If you aim for EPEL5 and older, define a buildroot. If not, delete the rm in
the install section. Defattr is not necessary anymore as well.

The install macro is not used anymore; just use the install command.

I'm under the impression, the description only describes one part of the
package. Is this correct? The description could probably be more concise in the
end.

"etc" and "etc.." should be "etc."

There are a few files licensed under GPL:

debuggerd/arm/pr-support.c: GPL 
debuggerd/arm/unwind.c: GPL 
sh/arith.c: GPL (with incorrect FSF address)

-- 
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.