Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 95764 - new version of trousers: 0.1.11
Summary: new version of trousers: 0.1.11
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High enhancement (vote)
Assignee: Crypto team [DISABLED]
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 96516
  Show dependency tree
 
Reported: 2005-06-11 04:22 UTC by Erik Ekman
Modified: 2005-06-22 07:27 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
0.1.11-makefile-statedir.patch (0.1.11-makefile-statedir.patch,3.53 KB, patch)
2005-06-11 09:45 UTC, Marcelo Goes (RETIRED)
Details | Diff
New patched ebuild (trousers-0.1.11.ebuild,1.32 KB, text/plain)
2005-06-12 01:36 UTC, Erik Ekman
Details
trousers-0.1.11.ebuild (alternative) (trousers-0.1.11.ebuild,2.38 KB, text/plain)
2005-06-19 04:28 UTC, TGL
Details
files/tcsd.initd (tcsd.initd,841 bytes, text/plain)
2005-06-19 04:28 UTC, TGL
Details
files/tcsd.confd (tcsd.confd,372 bytes, text/plain)
2005-06-19 04:29 UTC, TGL
Details

Note You need to log in before you can comment on or make changes to this bug.
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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 2005-06-22 07:27:17 UTC
note to self/others - need udev permissions file as described in  
documentation.