[Bug 1259486] Review Request: libglvnd - The GL Vendor-Neutral Dispatch library

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

 



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

Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zbyszek@xxxxxxxxx



--- Comment #5 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
Why not use a tarball directly from github, and run autogen.sh during %build?
In general it is a good idea to regenerate everything from pristine sources,
and in this case it will avoid the need for manual steps and simplify
maintenance and allow automatic version bumps.

[1] in the %description doesn't point anywhere.

%description should end in a dot.

make %{?_smp_mflags} V=1 → %make_build V=1

find %{buildroot} -name '*.la' -exec rm -f {} ';' → find %{buildroot} -name
'*.la' -delete

(-f is bad, because it means "ignore errors", and if an error happened here,
you actually want the build to fail. Anyway -delete is nicer.)

No %license file?

No %check?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]