https://bugzilla.redhat.com/show_bug.cgi?id=1611829
--- Comment #23 from Stefan Berger <stefanb(a)us.ibm.com> ---
(In reply to Marc-Andre Lureau from comment #17)
- remove the duplicated "License: BSD" lines, the top-level
one is enough
Actually added a license to the -devel package.
- use regular section order, %files come just before %changelog (thanks!)
Fixed.
- since the package builds with python3, why not BR python3 and remove BR
python2-twisted?
- %post -> %post selinux ?
Dropped selinux package.
Why do you make it a seperate (required) -selinux package, and not include
the policy in swtpm package directly?
What is the status of selinux policy review ?
Haven't heared back from Lukas.
- I suggest to remove the top-level comment: # --- swtpm rpm-spec ---
Done.
- dist'ing libswtpm_libtpms.so in -devel isn't useful if you don't provide a
header & stable API to use it.
- it is a good idea to add a link to this BZ in the changelog
Done. And pushed the update.
--
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