* [Buildroot] [PATCH] keyutils: new package
@ 2012-10-24 20:58 Jarkko Sakkinen
2012-10-24 21:06 ` Yann E. MORIN
0 siblings, 1 reply; 4+ messages in thread
From: Jarkko Sakkinen @ 2012-10-24 20:58 UTC (permalink / raw)
To: buildroot
Linux key management utilities
Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@iki.fi>
---
package/Config.in | 1 +
package/keyutils/Config.in | 7 +++++++
package/keyutils/keyutils.mk | 28 ++++++++++++++++++++++++++++
3 files changed, 36 insertions(+)
create mode 100644 package/keyutils/Config.in
create mode 100644 package/keyutils/keyutils.mk
diff --git a/package/Config.in b/package/Config.in
index 1650c71..00d242d 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -715,6 +715,7 @@ source "package/supervisor/Config.in"
source "package/systemd/Config.in"
source "package/util-linux/Config.in"
source "package/dsp-tools/Config.in"
+source "package/keyutils/Config.in"
endmenu
menu "Text editors and viewers"
diff --git a/package/keyutils/Config.in b/package/keyutils/Config.in
new file mode 100644
index 0000000..2296e5b
--- /dev/null
+++ b/package/keyutils/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_KEYUTILS
+ bool "keyutils"
+ help
+ Linux key management utilities
+
+ http://people.redhat.com/dhowells/keyutils/
+
diff --git a/package/keyutils/keyutils.mk b/package/keyutils/keyutils.mk
new file mode 100644
index 0000000..82b1f58
--- /dev/null
+++ b/package/keyutils/keyutils.mk
@@ -0,0 +1,28 @@
+#############################################################
+#
+# keyutils
+#
+#############################################################
+
+KEYUTILS_VERSION = 1.5.5
+KEYUTILS_SITE = http://people.redhat.com/dhowells/keyutils/
+KEYUTILS_SOURCE = keyutils-$(KEYUTILS_VERSION).tar.bz2
+
+define KEYUTILS_BUILD_CMDS
+ $(MAKE) CFLAGS="$(TARGET_CFLAGS)" LDFLAGS="$(TARGET_LDFLAGS)" \
+ CC="$(TARGET_CC)" -C $(@D)
+endef
+
+define KEYUTILS_INSTALL_TARGET_CMDS
+ $(INSTALL) -m 0755 -D $(@D)/keyctl $(TARGET_DIR)/bin/keyctl
+endef
+
+define KEYUTILS_UNINSTALL_TARGET_CMDS
+ rm -f $(TARGET_DIR)/bin/keyctl
+endef
+
+define KEYUTILS_CLEAN_CMDS
+ -$(MAKE) -C $(@D) clean
+endef
+
+$(eval $(generic-package))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] keyutils: new package
2012-10-24 20:58 [Buildroot] [PATCH] keyutils: new package Jarkko Sakkinen
@ 2012-10-24 21:06 ` Yann E. MORIN
2012-10-24 22:27 ` Yann E. MORIN
2012-10-25 5:23 ` Jarkko Sakkinen
0 siblings, 2 replies; 4+ messages in thread
From: Yann E. MORIN @ 2012-10-24 21:06 UTC (permalink / raw)
To: buildroot
Jarkko, All,
On Wednesday 24 October 2012 Jarkko Sakkinen wrote:
> Linux key management utilities
There are two memleaks known in keyutils.
I'm about to post a patch series that includes keyutils
with patches to fix the memleaks.
[--SNIP--]
> diff --git a/package/keyutils/keyutils.mk b/package/keyutils/keyutils.mk
> new file mode 100644
> index 0000000..82b1f58
> --- /dev/null
> +++ b/package/keyutils/keyutils.mk
> @@ -0,0 +1,28 @@
> +#############################################################
> +#
> +# keyutils
> +#
> +#############################################################
> +
> +KEYUTILS_VERSION = 1.5.5
> +KEYUTILS_SITE = http://people.redhat.com/dhowells/keyutils/
> +KEYUTILS_SOURCE = keyutils-$(KEYUTILS_VERSION).tar.bz2
> +
> +define KEYUTILS_BUILD_CMDS
> + $(MAKE) CFLAGS="$(TARGET_CFLAGS)" LDFLAGS="$(TARGET_LDFLAGS)" \
> + CC="$(TARGET_CC)" -C $(@D)
> +endef
I have:
define KEYUTILS_BUILD_CMDS
$(KEYUTILS_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D)
endef
Which includes all our CFLAGS, while doing the way you did overrides the
CFLAGS in keytuils' Makefile, and breaks (at least for me on one arch).
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread* [Buildroot] [PATCH] keyutils: new package
2012-10-24 21:06 ` Yann E. MORIN
@ 2012-10-24 22:27 ` Yann E. MORIN
2012-10-25 5:23 ` Jarkko Sakkinen
1 sibling, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2012-10-24 22:27 UTC (permalink / raw)
To: buildroot
Jarkko, All,
On Wednesday 24 October 2012 Yann E. MORIN wrote:
> On Wednesday 24 October 2012 Jarkko Sakkinen wrote:
> > Linux key management utilities
>
> There are two memleaks known in keyutils.
>
> I'm about to post a patch series that includes keyutils
> with patches to fix the memleaks.
Also, the install rule needs to be fixed. See:
http://lists.busybox.net/pipermail/buildroot/2012-October/060377.html
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] keyutils: new package
2012-10-24 21:06 ` Yann E. MORIN
2012-10-24 22:27 ` Yann E. MORIN
@ 2012-10-25 5:23 ` Jarkko Sakkinen
1 sibling, 0 replies; 4+ messages in thread
From: Jarkko Sakkinen @ 2012-10-25 5:23 UTC (permalink / raw)
To: buildroot
On Thu, Oct 25, 2012 at 12:06 AM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> Jarkko, All,
>
> On Wednesday 24 October 2012 Jarkko Sakkinen wrote:
>> Linux key management utilities
>
> There are two memleaks known in keyutils.
>
> I'm about to post a patch series that includes keyutils
> with patches to fix the memleaks.
Ok, great. I'll look forward to that.
>
> [--SNIP--]
>> diff --git a/package/keyutils/keyutils.mk b/package/keyutils/keyutils.mk
>> new file mode 100644
>> index 0000000..82b1f58
>> --- /dev/null
>> +++ b/package/keyutils/keyutils.mk
>> @@ -0,0 +1,28 @@
>> +#############################################################
>> +#
>> +# keyutils
>> +#
>> +#############################################################
>> +
>> +KEYUTILS_VERSION = 1.5.5
>> +KEYUTILS_SITE = http://people.redhat.com/dhowells/keyutils/
>> +KEYUTILS_SOURCE = keyutils-$(KEYUTILS_VERSION).tar.bz2
>> +
>> +define KEYUTILS_BUILD_CMDS
>> + $(MAKE) CFLAGS="$(TARGET_CFLAGS)" LDFLAGS="$(TARGET_LDFLAGS)" \
>> + CC="$(TARGET_CC)" -C $(@D)
>> +endef
>
> I have:
> define KEYUTILS_BUILD_CMDS
> $(KEYUTILS_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D)
> endef
>
> Which includes all our CFLAGS, while doing the way you did overrides the
> CFLAGS in keytuils' Makefile, and breaks (at least for me on one arch).
>
> Regards,
> Yann E. MORIN.
>
> --
> .-----------------.--------------------.------------------.--------------------.
> | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
> | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
> '------------------------------^-------^------------------^--------------------'
/Jarkko
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-10-25 5:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-24 20:58 [Buildroot] [PATCH] keyutils: new package Jarkko Sakkinen
2012-10-24 21:06 ` Yann E. MORIN
2012-10-24 22:27 ` Yann E. MORIN
2012-10-25 5:23 ` Jarkko Sakkinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox