[Buildroot] [PATCH v2 1/2] trousers: new package

[Buildroot] [PATCH v2 1/2] trousers: new package

[Noé Rubinstein]

Signed-off-by: Al West <al.west@v-nova.com>
Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>

---

Previously submitted by Al West <al.west@v-nova.com>

Changes since v1:
* Removed upstreamed patch
* Remove configuration option "--disable-usercheck"
* Use upstream hashes
* More verbose package description
* Cosmetic changes
---
 package/Config.in              |  1 +
 package/trousers/Config.in     | 11 +++++++++++
 package/trousers/trousers.hash |  3 +++
 package/trousers/trousers.mk   | 17 +++++++++++++++++
 4 files changed, 32 insertions(+)
 create mode 100644 package/trousers/Config.in
 create mode 100644 package/trousers/trousers.hash
 create mode 100644 package/trousers/trousers.mk

diff --git a/package/Config.in b/package/Config.in
index f0ddf0c..850cb83 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -765,6 +765,7 @@ menu "Crypto"
 	source "package/nettle/Config.in"
 	source "package/openssl/Config.in"
 	source "package/polarssl/Config.in"
+	source "package/trousers/Config.in"
 endmenu
 
 menu "Database"
diff --git a/package/trousers/Config.in b/package/trousers/Config.in
new file mode 100644
index 0000000..a740c00
--- /dev/null
+++ b/package/trousers/Config.in
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_TROUSERS
+	bool "trousers"
+	depends on BR2_PACKAGE_OPENSSL
+	help
+	  The open-source TCG Software Stack (TSS).
+
+	  This library enables the use of a Trusted Platform Module that
+	  complies with the TPM specification, version 1.2. It implements the
+	  TSS specification, version 1.2.
+
+	  http://trousers.sourceforge.net/
diff --git a/package/trousers/trousers.hash b/package/trousers/trousers.hash
new file mode 100644
index 0000000..997cfa9
--- /dev/null
+++ b/package/trousers/trousers.hash
@@ -0,0 +1,3 @@
+# http://sourceforge.net/projects/trousers/files/trousers/0.3.13/
+sha1 d23f1a3df4febffc4174f5cca7d1c54230477bb2 trousers-0.3.13.tar.gz
+md5  ad508f97b406f6e48cd90e85d78e7ca8         trousers-0.3.13.tar.gz
diff --git a/package/trousers/trousers.mk b/package/trousers/trousers.mk
new file mode 100644
index 0000000..86366e9
--- /dev/null
+++ b/package/trousers/trousers.mk
@@ -0,0 +1,17 @@
+#############################################################
+#
+# trousers
+#
+##############################################################
+
+TROUSERS_VERSION = 0.3.13
+TROUSERS_SOURCE = trousers-$(TROUSERS_VERSION).tar.gz
+TROUSERS_SITE = http://downloads.sourceforge.net/project/trousers/trousers/$(TROUSERS_VERSION)
+TROUSERS_INSTALL_STAGING = YES
+TROUSERS_DEPENDENCIES = openssl
+
+# The TrouSerS build system attempts to create the tss user and group on the
+# host system. Disable the user checking feature as a workaround.
+TROUSERS_CONF_OPTS += --disable-usercheck
+
+$(eval $(autotools-package))
-- 
2.1.4

[Buildroot] [PATCH v2 2/2] tpm-tools: new package

[Noé Rubinstein]

Signed-off-by: Al West <al.west@v-nova.com>
Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>

---

Previously submitted by Al West <al.west@v-nova.com>

Changes since v1:
* Removed upstreamed patch
* Remove configuration option "--disable-usercheck"
* Use upstream hashes
* More verbose package description
* Cosmetic changes
---
 package/Config.in                |  1 +
 package/tpm-tools/Config.in      |  8 ++++++++
 package/tpm-tools/tpm-tools.hash |  3 +++
 package/tpm-tools/tpm-tools.mk   | 13 +++++++++++++
 4 files changed, 25 insertions(+)
 create mode 100644 package/tpm-tools/Config.in
 create mode 100644 package/tpm-tools/tpm-tools.hash
 create mode 100644 package/tpm-tools/tpm-tools.mk

diff --git a/package/Config.in b/package/Config.in
index 850cb83..ad77244 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -765,6 +765,7 @@ menu "Crypto"
 	source "package/nettle/Config.in"
 	source "package/openssl/Config.in"
 	source "package/polarssl/Config.in"
+	source "package/tpm-tools/Config.in"
 	source "package/trousers/Config.in"
 endmenu
 
diff --git a/package/tpm-tools/Config.in b/package/tpm-tools/Config.in
new file mode 100644
index 0000000..0c25c76
--- /dev/null
+++ b/package/tpm-tools/Config.in
@@ -0,0 +1,8 @@
+config BR2_PACKAGE_TPM_TOOLS
+	bool "tpm-tools"
+	select BR2_PACKAGE_TROUSERS
+	depends on BR2_PACKAGE_OPENSSL
+	help
+	  Tools to manage and diagnose a TPM
+
+	  http://trousers.sourceforge.net/
diff --git a/package/tpm-tools/tpm-tools.hash b/package/tpm-tools/tpm-tools.hash
new file mode 100644
index 0000000..8c1ff89
--- /dev/null
+++ b/package/tpm-tools/tpm-tools.hash
@@ -0,0 +1,3 @@
+# http://sourceforge.net/projects/trousers/files/tpm-tools/1.3.8/
+sha1 f2c799ac8dc78f3537e1dbe4ae7c6e93f37e868c tpm-tools-1.3.8.tar.gz
+md5  85a978c4e03fefd4b73cbeadde7c4d0b         tpm-tools-1.3.8.tar.gz
diff --git a/package/tpm-tools/tpm-tools.mk b/package/tpm-tools/tpm-tools.mk
new file mode 100644
index 0000000..d6fea7b
--- /dev/null
+++ b/package/tpm-tools/tpm-tools.mk
@@ -0,0 +1,13 @@
+#############################################################
+#
+# tpm-tools
+#
+##############################################################
+
+TPM_TOOLS_VERSION = 1.3.8
+TPM_TOOLS_SOURCE = tpm-tools-$(TPM_TOOLS_VERSION).tar.gz
+TPM_TOOLS_SITE = http://downloads.sourceforge.net/project/trousers/tpm-tools/$(TPM_TOOLS_VERSION)
+TPM_TOOLS_STRIP_COMPONENTS = 2
+TPM_TOOLS_DEPENDENCIES = trousers
+
+$(eval $(autotools-package))
-- 
2.1.4

[Buildroot] [PATCH v2 1/2] trousers: new package

[Noé Rubinstein]

> * Remove configuration option "--disable-usercheck"

Oops; as the diff shows, I changed my mind on this one, and added a
comment instead.

--
No? Rubinstein

[Buildroot] [PATCH v2 1/2] trousers: new package

[Thomas Petazzoni]

Dear No? Rubinstein,

On Wed,  4 Nov 2015 18:47:04 +0100, No? Rubinstein wrote:
> Signed-off-by: Al West <al.west@v-nova.com>
> Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>
> 
> ---

I've applied, but there were a number of remaining issues that had to
be fixed:

    [Thomas:
     - use a 'select' instead of a 'depends on' for openssl
     - select libiconv when locale support is not available, otherwise the
       build fails due to missing iconv.h
     - disable on BR2_STATIC_LIBS configurations as it doesn't build. It
       could be made to build by patching the trousers Makefile.
     - add missing dependency on thread support in the toolchain
     - add Config.in comment about the thread and dynamic library
       dependencies.
     - add missing <pkg>_LICENSE and <pkg>_LICENSE_FILES variables.]

Please monitor autobuild.buildroot.org in the coming days to see if
there are any failures related to this new package.

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2 2/2] tpm-tools: new package

[Thomas Petazzoni]

Dear No? Rubinstein,

On Wed,  4 Nov 2015 18:47:05 +0100, No? Rubinstein wrote:
> Signed-off-by: Al West <al.west@v-nova.com>
> Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>
> 
> ---

Applied, after doing quite a few changes:

    [Thomas:
     - move from Libraries/Crypto to System tools in menuconfig
     - add patch to fix build with uClibc
     - use 'select' for openssl rather than a 'depends on'
     - propagate trousers dependencies
     - add <pkg>_LICENSE and <pkg>_LICENSEFILES
     - link against libiconv when needed, fixing !locale builds
     - explicitly disable pkcs11 support, since it is not handled by
       tpm-tools.mk currently.]

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2 1/2] trousers: new package

[Noé RUBINSTEIN]

Good morning Thomas,

Thanks a lot for all these fixes and for merging.

2015-12-24 8:42 GMT+01:00 Thomas Petazzoni <
thomas.petazzoni@free-electrons.com>:

>      - select libiconv when locale support is not available, otherwise the
>        build fails due to missing iconv.h
>      - disable on BR2_STATIC_LIBS configurations as it doesn't build. It
>        could be made to build by patching the trousers Makefile.
>      - add missing dependency on thread support in the toolchain
>

Are you running an automatic test set on your host machine to detect these
problems?


> Please monitor autobuild.buildroot.org in the coming days to see if
> there are any failures related to this new package.
>

I will do this.

Thanks again,
No?.

-- 




*This email and any attachment thereto are confidential and intended solely 
for the use of the individual or entity to whom they are addressed.If you 
are not the intended recipient, please be advised that disclosing, copying, 
distributing or taking any action in reliance on the contents of this email 
is strictly prohibited. In such case, please immediately advise the sender, 
and delete all copies and attachment from your system.This email shall not 
be construed and is not tantamount to an offer, an acceptance of offer, or 
an agreement by Aldebaran Robotics on any discussion or contractual 
document whatsoever. No employee or agent is authorized to represent or 
bind Aldebaran Robotics to third parties by email, or act on behalf of 
Aldebaran Robotics by email, without express written confirmation by 
Aldebaran Robotics? duly authorized representatives.*
------------------------------

*Ce message ?lectronique et ?ventuelles pi?ces jointes sont confidentiels, 
et exclusivement destin?s ? la personne ou l'entit? ? qui ils sont 
adress?s.*

*Si vous n'?tes pas le destinataire vis?, vous ?tes pri? de ne pas 
divulguer, copier, distribuer ou prendre toute d?cision sur la foi de ce 
message ?lectronique. Merci d'en aviser imm?diatement l'exp?diteur et de 
supprimer toutes les copies et ?ventuelles pi?ces jointes de votre syst?me.*

*Ce message ?lectronique n'?quivaut pas ? une offre, ? une acceptation 
d?offre, ou ? un accord d'Aldebaran Robotics sur toute discussion ou 
document contractuel quel qu?il soit, et ne peut ?tre interpr?t? comme tel. 
Aucun employ? ou agent d?Aldebaran Robotics n'est autoris? ? repr?senter ou 
? engager la soci?t? par email, ou ? agir au nom et pour le compte de la 
soci?t? par email, sans qu?une confirmation ?crite soit donn?e par le 
repr?sentant l?gal d?Aldebaran Robotics ou par toute autre personne ayant 
re?u d?l?gation de pouvoir appropri?e.*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20151224/59abe049/attachment.html>

[Buildroot] [PATCH v2 1/2] trousers: new package

[Thomas Petazzoni]

Hello,

On Thu, 24 Dec 2015 10:21:00 +0100, No? RUBINSTEIN wrote:

> >      - select libiconv when locale support is not available, otherwise the
> >        build fails due to missing iconv.h
> >      - disable on BR2_STATIC_LIBS configurations as it doesn't build. It
> >        could be made to build by patching the trousers Makefile.
> >      - add missing dependency on thread support in the toolchain
> >
> 
> Are you running an automatic test set on your host machine to detect these
> problems?

No, I am testing with a bunch of toolchains used by autobuild.b.o.
Since your package had no toolchain dependencies, I tested with
http://autobuild.buildroot.org/toolchains/configs/br-arm-basic.config.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2 2/2] tpm-tools: new package

[Thomas Petazzoni]

Dear No? Rubinstein,

On Wed,  4 Nov 2015 18:47:05 +0100, No? Rubinstein wrote:
> Signed-off-by: Al West <al.west@v-nova.com>
> Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>
> 
> ---

This package is causing a number of build issues. Can you look into
them? See:

  http://autobuild.buildroot.org/?reason=tpm-tools-1.3.8

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com