[Bug 815951] Review Request: weston - Reference compositor for Wayland

[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=815951

--- Comment #3 from Richard Hughes <rhughes at redhat.com> 2012-04-25 08:03:22 EDT ---
(In reply to comment #2)
> Few notes:
> 
> * You may drop %clean section entirely. I believe this package isn't intended
> to run on onl EL boxes.

Correct, fixed.

> * Unowned directories
> ** %{_libdir}/weston/
> ** %{_libdir}/weston/
> Either specifically mark them as %dir in the %files section or change %files
> section to that

Fixed, thanks.

> * I don't like this line
> autoreconf -v --install || exit 1
> So if autoreconf were fail for whatever reason what would we expect then?
> Successful building?
> Can you simplify this to "autoreconv -ivf" (notice -f switch)?

Yup, done.

> * License field is wron. Must be "BSD and CC-BY-SA". The latter is for content.

Agreed, fixed.

> So, please, address/explain my notes and I'll finish it.

New files: 
http://people.freedesktop.org/~hughsient/temp/weston.spec
http://people.freedesktop.org/~hughsient/temp/weston-0.89-0.3.fc17.src.rpm

Thanks for the super-quick turnaround.

Richard.

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