Bugzilla Bug 171561

The version of app-crypt/tpm-emulator is very outdated. 
The new version has major improvements.
Ebuild for installing current version 0.5 attached, product is installed and
tested on my machine without errors 

Reproducible: Always

Created an attachment (id=113873) [details]
ebuild and helper files

Thank you for your report.

Next time, please:
1. Attach individual files to bug, not archives.
2. If an existing file is modify please attach only diff.
3. Please explain any architecture changes.

Issues:
1. Why you removed the kernel modules from ebuild? Should we add a new ebuild
for this?
2. The checkconfig device handling should be done using udev rule and not
manually.
3. What is the /dev/tpm0 story?

Thanks!

Hi Alon,

1. Why you removed the kernel modules from ebuild? Should we add a new ebuild
for this?
The Makefile of the original package takes care of the module, so I left the
Makefile unchanged. 

2. The checkconfig device handling should be done using udev rule and not
manually.

I copied this over from the trousers script, I am not that familiar with udev,
sorry...if you give me a pointer I will try.

3. What is the /dev/tpm0 story?
Several apps use /dev/tpm0 and do not allow this to be configured, while the
emulator creates /dev/tpm. I use this workaround for a long time now and it
does the job. 

Oliver

Created an attachment (id=115069) [details]
tpm-emulator-0.5.ebuild

Created an attachment (id=115070) [details]
files/tpm-emulator.initd

Created an attachment (id=115071) [details]
files/tpm-emulator.udev

This was an effort...
Please check this.

Issues for upstram:

1. Allow specifying LDFLAGS, current fix is:
sed -i 's/LDFLAGS :=/override LDFLAGS +=/g' tpmd/Makefile

2. Put runtime state in /var/run/tpm and not /var/tpm, or allow specifying it
externally. Current fix is:
sed -i 's#/var/tpm#/var/run/tpm#g' tpmd/tpmd.c tddl/tddl.c
tpm_dev/linux_module.c

3. Ask upstream to separate kernel module build and userspace.

4. "make install" should allow to use DESTDIR to install into temperary
location, current solution is to install manually.

5. "make install" does not use "install" command, so file attributes are not
set correctly, current solution is to install manually.

6. Upstream should add udev rules to distribution.

https://developer.berlios.de/feature/index.php?func=detailfeature&feature_id=3304&group_id=2491

Upstream does not seems to be responsive at all.