[Bug 1369720] Review Request: tpm2-tools - a TPM2.0 testing tool build upon TPM2.0-TSS

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

 



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



--- Comment #8 from yunying.sun@xxxxxxxxx ---
(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
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




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