Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=907213

--- Comment #3 from Miro HronĨok <[email protected]> ---
(In reply to comment #2)
> 1. You can't package the demo binaries into /usr/share/doc/*, because all
> files below /usr/share must be arch-independent.
That was not my intention. They are present by mistake.

> I'd recommend to do one of the following:
> a) To only ship the sources (*.c) under /usr/share/doc/*
This is what i wanted. I deleted the apps from /usr/bin, but I didn't noticed
they are in demos too.

> 2. Consider appending --disable-static to %configure.
> This would avoid much of the "rm"-sequence you are using to remove the
> static libs and would avoid compiling the libraries twice.
Good point

> 3. The %check section is superflous.
> %check is supposed to run a testsuite, but this package doesn't ship one.
OK.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ZiR26BHPvE&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to