All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2 0/2] make host-cryptopp selectable
@ 2018-12-20 23:22 Carlos Santos
  2018-12-20 23:22 ` [Buildroot] [PATCH v2 1/2] package/cryptopp: add a blind Config.in.host Carlos Santos
  2018-12-20 23:22 ` [Buildroot] [PATCH v2 2/2] package/tegrarcm: select host-cryptopp Carlos Santos
  0 siblings, 2 replies; 3+ messages in thread
From: Carlos Santos @ 2018-12-20 23:22 UTC (permalink / raw)
  To: buildroot

Quoting Thomas Petazzoni, the current situation is:

 * Most host packages don't have any Config.in.host options (blind or
   visible), because they are merely build dependencies of other
   packages.

 * A few host packages do have a visible Config.in.host because they
   are useful by themselves, and not just as a build dependency of
   something else: image generation tools, flashing tools, qemu, etc.

The idea we have for the future is:

 * All host packages have a Config.in.host option.

 * The host packages that are only build dependencies of other packages
   have a blind Config.in.host option

 * The host packages that are useful by themselves continue to have a
   visible Config.in.host option.

Carlos Santos (2):
  package/cryptopp: add a blind Config.in.host
  package/tegrarcm: select host-cryptopp

 package/Config.in.host          | 1 +
 package/cryptopp/Config.in.host | 6 ++++++
 package/tegrarcm/Config.in.host | 1 +
 3 files changed, 8 insertions(+)
 create mode 100644 package/cryptopp/Config.in.host

-- 
2.19.2

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH v2 1/2] package/cryptopp: add a blind Config.in.host
  2018-12-20 23:22 [Buildroot] [PATCH v2 0/2] make host-cryptopp selectable Carlos Santos
@ 2018-12-20 23:22 ` Carlos Santos
  2018-12-20 23:22 ` [Buildroot] [PATCH v2 2/2] package/tegrarcm: select host-cryptopp Carlos Santos
  1 sibling, 0 replies; 3+ messages in thread
From: Carlos Santos @ 2018-12-20 23:22 UTC (permalink / raw)
  To: buildroot

Quoting Thomas Petazzoni, the idea we have for the future is:

 * All host packages have a Config.in.host option.

 * The host packages that are only build dependencies of other packages
   have a blind Config.in.host option

 * The host packages that are useful by themselves continue to have a
   visible Config.in.host option.

host-cryptopp gets a blind Config.in.host, since only exists to build
tegrarcm. A help text is included to document the package, only, since
it is not shown in the configuration menu.

Signed-off-by: Carlos Santos <casantos@datacom.com.br>
---
Changes v1->v2:
  - Explain the motivation in the commit message
  - Make the Config.in.host blind
---
 package/Config.in.host          | 1 +
 package/cryptopp/Config.in.host | 6 ++++++
 2 files changed, 7 insertions(+)
 create mode 100644 package/cryptopp/Config.in.host

diff --git a/package/Config.in.host b/package/Config.in.host
index 16b474fc9d..04537fc2e2 100644
--- a/package/Config.in.host
+++ b/package/Config.in.host
@@ -9,6 +9,7 @@ menu "Host utilities"
 	source "package/checksec/Config.in.host"
 	source "package/cmake/Config.in.host"
 	source "package/cramfs/Config.in.host"
+	source "package/cryptopp/Config.in.host"
 	source "package/cryptsetup/Config.in.host"
 	source "package/dfu-util/Config.in.host"
 	source "package/dos2unix/Config.in.host"
diff --git a/package/cryptopp/Config.in.host b/package/cryptopp/Config.in.host
new file mode 100644
index 0000000000..e8159b0a7e
--- /dev/null
+++ b/package/cryptopp/Config.in.host
@@ -0,0 +1,6 @@
+config BR2_PACKAGE_HOST_CRYPTOPP
+	bool
+	help
+	  A free C++ class library of cryptographic schemes
+
+	  https://www.cryptopp.com/
-- 
2.19.2

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH v2 2/2] package/tegrarcm: select host-cryptopp
  2018-12-20 23:22 [Buildroot] [PATCH v2 0/2] make host-cryptopp selectable Carlos Santos
  2018-12-20 23:22 ` [Buildroot] [PATCH v2 1/2] package/cryptopp: add a blind Config.in.host Carlos Santos
@ 2018-12-20 23:22 ` Carlos Santos
  1 sibling, 0 replies; 3+ messages in thread
From: Carlos Santos @ 2018-12-20 23:22 UTC (permalink / raw)
  To: buildroot

Since cryptopp now has a Config.in.host, make the depending packages
(currently only tegrarcm) select it.

Signed-off-by: Carlos Santos <casantos@datacom.com.br>
---
Changes v1->v2:
  - Explain the motivation in the commit message
  - Restore the BR2_arm dependency
---
 package/tegrarcm/Config.in.host | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/tegrarcm/Config.in.host b/package/tegrarcm/Config.in.host
index d8cbb947d6..aeff8f158c 100644
--- a/package/tegrarcm/Config.in.host
+++ b/package/tegrarcm/Config.in.host
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_HOST_TEGRARCM
 	bool "host tegrarcm"
 	depends on BR2_arm || BR2_armeb
+	select BR2_PACKAGE_HOST_CRYPTOPP
 	help
 	  This program is used to send code to a Tegra device in
 	  recovery mode.
-- 
2.19.2

^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-12-20 23:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-20 23:22 [Buildroot] [PATCH v2 0/2] make host-cryptopp selectable Carlos Santos
2018-12-20 23:22 ` [Buildroot] [PATCH v2 1/2] package/cryptopp: add a blind Config.in.host Carlos Santos
2018-12-20 23:22 ` [Buildroot] [PATCH v2 2/2] package/tegrarcm: select host-cryptopp Carlos Santos

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.