Bug 95764 – new version of trousers: 0.1.11

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description Erik Ekman 2005-06-11 04:22:53 UTC]

A new version of trousers is available:
http://sourceforge.net/project/showfiles.php?group_id=126012



Reproducible: Always
Steps to Reproduce:
1.
2.
3.

[Comment 1 Erik Ekman 2005-06-11 04:24:26 UTC]

the old 0.1.7 ebuild works if renamed, after removing the 0.1.7-makefile-
statedir patch.

[Comment 2 Marcelo Goes (RETIRED) 2005-06-11 09:45:51 UTC]

Created attachment 61060 [details, diff]
0.1.11-makefile-statedir.patch

I ported the patch to 0.1.11, but it fails for me.

configure.in:8: version mismatch.  This is Automake 1.7.9,
configure.in:8: but the definition used by this AM_INIT_AUTOMAKE
configure.in:8: comes from Automake 1.8.5.  You should recreate
configure.in:8: aclocal.m4 with aclocal and run automake again.
configure.in: installing `./mkinstalldirs'

[Comment 3 Erik Ekman 2005-06-12 01:36:07 UTC]

Created attachment 61098 [details]
New patched ebuild

[Comment 4 Erik Ekman 2005-06-12 01:36:33 UTC]

Changed automake DEPEND to
>=sys-devel/automake-1.8.5

and in src_compile:
WANT_AUTOMAKE=1.8

attached ebuild

[Comment 5 Erik Ekman 2005-06-12 01:50:07 UTC]

it installs fine here without the makefile-statdir patch. 

this error appears in make install when patch applied:

make  install-exec-hook
make[3]: Entering directory `/var/tmp/portage/trousers-0.1.11/work/trousers-0.1.
11/dist'
/usr/sbin/groupadd tss || true
groupadd: group tss exists
/usr/sbin/useradd -r tss -g tss || true
/usr/sbin/useradd: invalid option -- r
Usage: useradd [-u uid [-o]] [-g group] [-G group,...] 
               [-d home] [-s shell] [-c comment] [-m [-k template]]
               [-f inactive] [-e expire]
               [-p passwd] name
       useradd -D [-g group] [-b base] [-s shell]
               [-f inactive] [-e expire]
/bin/chown tss:tss /var/tmp/portage/trousers-0.1.11/image///usr/sbin/tcsd
/bin/sh -c 'if [ ! -e /var/tmp/portage/trousers-0.1.11/image///var/state/lib/tpm
 ];then mkdir -p /var/tmp/portage/trousers-0.1.11/image///var/lib/tpm; fi'
/bin/chown tss:tss /var/tmp/portage/trousers-0.1.11/image///var/state/lib/tpm
/bin/chown: cannot access `/var/tmp/portage/trousers-0.1.11/image///var/state/
lib/tpm': No such file or directory
make[3]: *** [install-exec-hook] Error 1
make[3]: Leaving directory `/var/tmp/portage/trousers-0.1.11/work/trousers-0.1.
11/dist'
make[2]: *** [install-exec-am] Error 2
make[2]: Leaving directory `/var/tmp/portage/trousers-0.1.11/work/trousers-0.1.
11/dist'
make[1]: *** [install-am] Error 2
make[1]: Leaving directory `/var/tmp/portage/trousers-0.1.11/work/trousers-0.1.
11/dist'
make: *** [install-recursive] Error 1

!!! ERROR: app-crypt/trousers-0.1.11 failed.
!!! Function src_install, Line 50, Exitcode 2
!!! (no error message)
!!! If you need support, post the topmost build error, NOT this status message.

[Comment 6 TGL 2005-06-19 04:27:36 UTC]

I've worked too on a 0.1.11 ebuild before finding (eh, ok, before searching
for...) this report. I will attach my ebuild, since it differs a bit anyway:

 * 0.1.11 now uses /var/lib/tpm for storing data, instead of /var/tpm like in
0.1.7, and I think that's right from a LFHS point-of-view. So I've not patched
it to use /var/state. That simplifies compilation a bit (no need to autotool
it). But that also invalidates existing "tss" user's homedir (see note in the
ebuild about a possible wa to fix that).

 * I've added a simple init.d script that can load the tpm module if needed and
then start tcsd.

 * I've dropped the dependency on app-crypt/tpm-module since it's no in 2.6.12
kernel, and added some pkg_setup() checks and einfo instead to deal with the
different possible situations. Sure, it would be cleaner to RDEPEND on "a
>=2.6.12 kernel or the tpm-module", but I'm not sure how to do that:
 - IIRC, it's not right to depend on a versioned virtual, thus we can't
RDEPEND="|| ( >=virtual/linux-sources-2.6.12  app-crypt/tpm )". But i'm not
really sure, so that should be verified with people who know.
 - we could also think of a "virtual/tpm-driver" provided by both
app-crypt/tpm-module and >=2.6.12 kernels, like for alsa drivers, but i guess
that's would be "bloat".

I've also written an ebuild for tpm-tools-1.0.0, but i will attach it to a new
report.

[Comment 7 TGL 2005-06-19 04:28:23 UTC]

Created attachment 61482 [details]
trousers-0.1.11.ebuild (alternative)

[Comment 8 TGL 2005-06-19 04:28:43 UTC]

Created attachment 61483 [details]
files/tcsd.initd

[Comment 9 TGL 2005-06-19 04:29:00 UTC]

Created attachment 61484 [details]
files/tcsd.confd

[Comment 10 Daniel Black (RETIRED) 2005-06-19 05:52:27 UTC]

Thankyou TGL. 
 
Thankyou Erik. 
 
Much appreciated. 0.1.11 added. 
 
The only odd think I noted is: 
drwxrwxrwt   2 tss      tss          4096 Jun 19 22:45 ./var/lib/tpm 
 
Are these permissions correct? 

[Comment 11 Daniel Black (RETIRED) 2005-06-22 07:27:17 UTC]

note to self/others - need udev permissions file as described in  
documentation.