[Bug 323441] Review Request: trousers - The open-source TCG software stack
bugzilla at redhat.com
bugzilla at redhat.com
Thu Dec 13 16:38:28 UTC 2007
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: trousers - The open-source TCG software stack
https://bugzilla.redhat.com/show_bug.cgi?id=323441
------- Additional Comments From yoder1 at us.ibm.com 2007-12-13 11:38 EST -------
Hi Mamoru,
(In reply to comment #28)
> Well, for 0.3.1-2:
>
> ? License related issue
> - build.log says that
> * the two binaries ps_convert and ps_inspect are created, but
> they are not installed.
> -------------------------------------------------
> 891 creating ps_convert
> 892 creating ps_inspect
> 893 make[1]: Leaving directory `/builddir/build/BUILD/trousers-0.3.1/tools'
> 894 make[1]: Entering directory `/builddir/build/BUILD/trousers-0.3.1'
> 895 make[1]: Nothing to be done for `all-am'.
> 896 make[1]: Leaving directory `/builddir/build/BUILD/trousers-0.3.1'
> 897 + exit 0
> -------------------------------------------------
> Is this okay?
> ! If these two binaries are not needed, it is okay. If needed,
> it is legally problematic.
Yes, this is ok. Those binaries are for converting the trousers key stores
from one previous version to another. They are only needed in the case a user
has been running trousers since version 0.2.1 or so.
> ! Patch name
> - Please rename the name of the patch for us to understand
> what the patch is for easily.
Oops, that was unintentional. I'll upload a new version today.
> * Timestamps
> - Please use
> --------------------------------------------------
> make install DESTDIR=${RPM_BUILD_ROOT} INSTALL="install -p"
> --------------------------------------------------
> to keep timestamps on installed files. While sometimes this does not
> work, this ususally works for most recent Makefiles.
No problem.
> * Again %clean section
> - Please remove "[ "${RPM_BUILD_ROOT}" != "/" ] && [ -d ${RPM_BUILD_ROOT} ] && "
> again.
> i.e. Leave only "rm -rf ${RPM_BUILD_ROOT}"
Ok.
> * scriptlets
> - What is the below part in %post for?
> ---------------------------------------------------
> if [ $? == 0 ]; then
> /sbin/chkconfig --level 35 tcsd on
> fi
> ---------------------------------------------------
> On %post $1 is not be 0 (check the section "Syntax" in
> http://fedoraproject.org/wiki/Packaging/ScriptletSnippets )
> And actually this part is not needed as service should not
> be activated by default.
Ok, I left in the --add section and ldconfig call only.
> * Documents
> - I think the following files can be added to %doc.
> -------------------------------------------------
> ChangeLog
> --------------------------------------------------
Added.
> (In reply to comment #27)
> > I don't see any blocking issues, but I wonder if there is a use for the
> > trousers-static rpm that gets built? I know static libraries need to be
> > packaged separately, but I was wondering if there were any users of these static
> > libraries or if they could be omitted?
> - Generally static archives should be removed.
> If you don't think this is not needed, please remove static archives
> completely.
For completeness of the TSS itself, I'd like to at least have this available
to developers. The TDDL is an interface documented by the TCG specs for use by
software. Its possible that others may want to develop code against it.
Thanks,
Kent
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
More information about the Fedora-package-review
mailing list