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



--- Comment #8 from [email protected] ---
(In reply to Igor Gnatenko from comment #7)
> (In reply to yunying.sun from comment #6)
> > Updated SPEC again, to restrict builds to ix86 & x86_64 only by adding:
> > ExclusiveArch:  %{ix86} x86_64
> > Reason for this change is tool does not support big endian arch, and has
> > been verified only on Intel x86 & x86_64.
> I would recommend to use ExcludeArch and exclude all arches which are big
> endian.

Thanks Igor for the comment. I've updated SPEC to use ExcludeArch instead of
ExclusiveArch.

Besides, there're also some other changes from tpm2-tss comments: updated
Source0 url to match actual upstream package, add a script to rename tarball
when needed, use %autosetup instead of %setup and add "-n" to specify the
uncompressed folder name.

SPEC: https://raw.githubusercontent.com/yunyings/share/master/tpm2-tools.spec
SRPMS:
https://github.com/yunyings/share/raw/master/tpm2-tools-1.1-0.1.beta1.el7.src.rpm
COPR build:
https://copr.fedorainfracloud.org/coprs/yunyings/tpm2-tools/build/451937/

@Igor, could you help to review again? Thanks.

-- 
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
[email protected]
https://lists.fedoraproject.org/admin/lists/[email protected]

Reply via email to