* [Buildroot] [PATCH 1/1] ipmiutil: new package
@ 2014-09-18 9:38 David Bachelart
2014-09-21 20:06 ` Thomas Petazzoni
2014-10-05 21:47 ` Thomas Petazzoni
0 siblings, 2 replies; 5+ messages in thread
From: David Bachelart @ 2014-09-18 9:38 UTC (permalink / raw)
To: buildroot
Signed-off-by: David Bachelart <david.bachelart@bbright.com>
---
package/Config.in | 1 +
package/ipmiutil/Config.in | 15 +++++++++++++++
package/ipmiutil/ipmiutil.mk | 15 +++++++++++++++
3 files changed, 31 insertions(+)
create mode 100644 package/ipmiutil/Config.in
create mode 100644 package/ipmiutil/ipmiutil.mk
diff --git a/package/Config.in b/package/Config.in
index 73ba86a..3eafabb 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -322,6 +322,7 @@ endif
source "package/intel-microcode/Config.in"
source "package/iostat/Config.in"
source "package/ipmitool/Config.in"
+ source "package/ipmiutil/Config.in"
source "package/irda-utils/Config.in"
source "package/iucode-tool/Config.in"
source "package/kbd/Config.in"
diff --git a/package/ipmiutil/Config.in b/package/ipmiutil/Config.in
new file mode 100644
index 0000000..29d6e26
--- /dev/null
+++ b/package/ipmiutil/Config.in
@@ -0,0 +1,15 @@
+config BR2_PACKAGE_IPMIUTIL
+ bool "ipmiutil"
+ depends on BR2_TOOLCHAIN_HAS_SSP
+ select BR2_PACKAGE_OPENSSL
+ select BR2_PACKAGE_ZLIB
+ help
+ The ipmiutil package provides easy-to-use utilities to view the SEL,
+ perform an IPMI chassis reset, set up the IPMI LAN and Platform Event Filter
+ entries to allow SNMP alerts, Serial-Over-LAN console, event daemon, and
+ other IPMI tasks.
+
+ http://ipmiutil.sourceforge.net/
+
+comment "ipmiutil needs a toolchain w/ ssp"
+ depends on !BR2_TOOLCHAIN_HAS_SSP
diff --git a/package/ipmiutil/ipmiutil.mk b/package/ipmiutil/ipmiutil.mk
new file mode 100644
index 0000000..9bf7325
--- /dev/null
+++ b/package/ipmiutil/ipmiutil.mk
@@ -0,0 +1,15 @@
+################################################################################
+#
+# ipmiutil
+#
+################################################################################
+
+IPMIUTIL_VERSION = 2.9.4
+IPMIUTIL_SITE = http://sourceforge.net/projects/ipmiutil/files/
+
+IPMIUTIL_DEPENDENCIES = openssl zlib
+
+IPMIUTIL_LICENSE = BSD-2c
+IPMIUTIL_LICENSE_FILES = COPYING
+
+$(eval $(autotools-package))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] ipmiutil: new package
2014-09-18 9:38 [Buildroot] [PATCH 1/1] ipmiutil: new package David Bachelart
@ 2014-09-21 20:06 ` Thomas Petazzoni
2014-10-05 21:47 ` Thomas Petazzoni
1 sibling, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2014-09-21 20:06 UTC (permalink / raw)
To: buildroot
Dear David Bachelart,
On Thu, 18 Sep 2014 11:38:35 +0200, David Bachelart wrote:
> diff --git a/package/ipmiutil/Config.in b/package/ipmiutil/Config.in
> new file mode 100644
> index 0000000..29d6e26
> --- /dev/null
> +++ b/package/ipmiutil/Config.in
> @@ -0,0 +1,15 @@
> +config BR2_PACKAGE_IPMIUTIL
> + bool "ipmiutil"
> + depends on BR2_TOOLCHAIN_HAS_SSP
Did you test if it works with an internal toolchain (i.e a toolchain
built by Buildroot) that has SSP enabled? Apparently, the package
forcefully links against -lssp. But I'm not sure if a libssp library is
built when SSP is enabled in the internal toolchain, because we use the
SSP support from the C library and not from the compiler. A
verification would be good, or even better, an improvement to ipmiutil
to make SSP support optional.
> + select BR2_PACKAGE_OPENSSL
> + select BR2_PACKAGE_ZLIB
This dependency on zlib seems unnecessary: I didn't see anything in
ipmiutil that references zlib directly. Did I miss it?
Also, the code contains call to fork(), so your package should depend
on BR2_USE_MMU. And the code uses threads, so it should depend on
thread support.
Try to make sure that your package builds fine with the following
toolchain configurations:
- http://autobuild.buildroot.org/toolchains/configs/br-arm-basic.config
- http://autobuild.buildroot.org/toolchains/configs/br-arm-full.config
- http://autobuild.buildroot.org/toolchains/configs/bfin-linux-uclibc.config
- http://autobuild.buildroot.org/toolchains/configs/bfin-uclinux.config
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] ipmiutil: new package
2014-09-18 9:38 [Buildroot] [PATCH 1/1] ipmiutil: new package David Bachelart
2014-09-21 20:06 ` Thomas Petazzoni
@ 2014-10-05 21:47 ` Thomas Petazzoni
2014-10-06 8:37 ` David Bachelart
1 sibling, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2014-10-05 21:47 UTC (permalink / raw)
To: buildroot
Dear David Bachelart,
Any news about this patch? I made some comments on September, 21st, but
unless I missed you haven't resent a new version. Do you intend to do
so in the near future?
Thanks a lot for your contribution,
Thomas
On Thu, 18 Sep 2014 11:38:35 +0200, David Bachelart wrote:
>
> Signed-off-by: David Bachelart <david.bachelart@bbright.com>
> ---
> package/Config.in | 1 +
> package/ipmiutil/Config.in | 15 +++++++++++++++
> package/ipmiutil/ipmiutil.mk | 15 +++++++++++++++
> 3 files changed, 31 insertions(+)
> create mode 100644 package/ipmiutil/Config.in
> create mode 100644 package/ipmiutil/ipmiutil.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 73ba86a..3eafabb 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -322,6 +322,7 @@ endif
> source "package/intel-microcode/Config.in"
> source "package/iostat/Config.in"
> source "package/ipmitool/Config.in"
> + source "package/ipmiutil/Config.in"
> source "package/irda-utils/Config.in"
> source "package/iucode-tool/Config.in"
> source "package/kbd/Config.in"
> diff --git a/package/ipmiutil/Config.in b/package/ipmiutil/Config.in
> new file mode 100644
> index 0000000..29d6e26
> --- /dev/null
> +++ b/package/ipmiutil/Config.in
> @@ -0,0 +1,15 @@
> +config BR2_PACKAGE_IPMIUTIL
> + bool "ipmiutil"
> + depends on BR2_TOOLCHAIN_HAS_SSP
> + select BR2_PACKAGE_OPENSSL
> + select BR2_PACKAGE_ZLIB
> + help
> + The ipmiutil package provides easy-to-use utilities to view the SEL,
> + perform an IPMI chassis reset, set up the IPMI LAN and Platform Event Filter
> + entries to allow SNMP alerts, Serial-Over-LAN console, event daemon, and
> + other IPMI tasks.
> +
> + http://ipmiutil.sourceforge.net/
> +
> +comment "ipmiutil needs a toolchain w/ ssp"
> + depends on !BR2_TOOLCHAIN_HAS_SSP
> diff --git a/package/ipmiutil/ipmiutil.mk b/package/ipmiutil/ipmiutil.mk
> new file mode 100644
> index 0000000..9bf7325
> --- /dev/null
> +++ b/package/ipmiutil/ipmiutil.mk
> @@ -0,0 +1,15 @@
> +################################################################################
> +#
> +# ipmiutil
> +#
> +################################################################################
> +
> +IPMIUTIL_VERSION = 2.9.4
> +IPMIUTIL_SITE = http://sourceforge.net/projects/ipmiutil/files/
> +
> +IPMIUTIL_DEPENDENCIES = openssl zlib
> +
> +IPMIUTIL_LICENSE = BSD-2c
> +IPMIUTIL_LICENSE_FILES = COPYING
> +
> +$(eval $(autotools-package))
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] ipmiutil: new package
2014-10-05 21:47 ` Thomas Petazzoni
@ 2014-10-06 8:37 ` David Bachelart
2014-10-06 10:33 ` Thomas Petazzoni
0 siblings, 1 reply; 5+ messages in thread
From: David Bachelart @ 2014-10-06 8:37 UTC (permalink / raw)
To: buildroot
Hi Thomas,
Yes, I do... I'm in a rush, I hope I'll submit a new patch in a few
days/weeks.
Regards,
David
2014-10-05 23:47 GMT+02:00 Thomas Petazzoni <
thomas.petazzoni@free-electrons.com>:
> Dear David Bachelart,
>
> Any news about this patch? I made some comments on September, 21st, but
> unless I missed you haven't resent a new version. Do you intend to do
> so in the near future?
>
> Thanks a lot for your contribution,
>
> Thomas
>
> On Thu, 18 Sep 2014 11:38:35 +0200, David Bachelart wrote:
> >
> > Signed-off-by: David Bachelart <david.bachelart@bbright.com>
> > ---
> > package/Config.in | 1 +
> > package/ipmiutil/Config.in | 15 +++++++++++++++
> > package/ipmiutil/ipmiutil.mk | 15 +++++++++++++++
> > 3 files changed, 31 insertions(+)
> > create mode 100644 package/ipmiutil/Config.in
> > create mode 100644 package/ipmiutil/ipmiutil.mk
> >
> > diff --git a/package/Config.in b/package/Config.in
> > index 73ba86a..3eafabb 100644
> > --- a/package/Config.in
> > +++ b/package/Config.in
> > @@ -322,6 +322,7 @@ endif
> > source "package/intel-microcode/Config.in"
> > source "package/iostat/Config.in"
> > source "package/ipmitool/Config.in"
> > + source "package/ipmiutil/Config.in"
> > source "package/irda-utils/Config.in"
> > source "package/iucode-tool/Config.in"
> > source "package/kbd/Config.in"
> > diff --git a/package/ipmiutil/Config.in b/package/ipmiutil/Config.in
> > new file mode 100644
> > index 0000000..29d6e26
> > --- /dev/null
> > +++ b/package/ipmiutil/Config.in
> > @@ -0,0 +1,15 @@
> > +config BR2_PACKAGE_IPMIUTIL
> > + bool "ipmiutil"
> > + depends on BR2_TOOLCHAIN_HAS_SSP
> > + select BR2_PACKAGE_OPENSSL
> > + select BR2_PACKAGE_ZLIB
> > + help
> > + The ipmiutil package provides easy-to-use utilities to view the
> SEL,
> > + perform an IPMI chassis reset, set up the IPMI LAN and Platform
> Event Filter
> > + entries to allow SNMP alerts, Serial-Over-LAN console, event
> daemon, and
> > + other IPMI tasks.
> > +
> > + http://ipmiutil.sourceforge.net/
> > +
> > +comment "ipmiutil needs a toolchain w/ ssp"
> > + depends on !BR2_TOOLCHAIN_HAS_SSP
> > diff --git a/package/ipmiutil/ipmiutil.mk b/package/ipmiutil/ipmiutil.mk
> > new file mode 100644
> > index 0000000..9bf7325
> > --- /dev/null
> > +++ b/package/ipmiutil/ipmiutil.mk
> > @@ -0,0 +1,15 @@
> >
> +################################################################################
> > +#
> > +# ipmiutil
> > +#
> >
> +################################################################################
> > +
> > +IPMIUTIL_VERSION = 2.9.4
> > +IPMIUTIL_SITE = http://sourceforge.net/projects/ipmiutil/files/
> > +
> > +IPMIUTIL_DEPENDENCIES = openssl zlib
> > +
> > +IPMIUTIL_LICENSE = BSD-2c
> > +IPMIUTIL_LICENSE_FILES = COPYING
> > +
> > +$(eval $(autotools-package))
>
>
>
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141006/8d66a37e/attachment.html>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/1] ipmiutil: new package
2014-10-06 8:37 ` David Bachelart
@ 2014-10-06 10:33 ` Thomas Petazzoni
0 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2014-10-06 10:33 UTC (permalink / raw)
To: buildroot
Dear David Bachelart,
On Mon, 6 Oct 2014 10:37:51 +0200, David Bachelart wrote:
> Yes, I do... I'm in a rush, I hope I'll submit a new patch in a few
> days/weeks.
Sure, I'll mark your patch as "Changes requested" in patch then.
So, please remember to resend a new version, as we will not send
another reminder since the patch will no longer be in our TODO-list of
patches to review.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-10-06 10:33 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-18 9:38 [Buildroot] [PATCH 1/1] ipmiutil: new package David Bachelart
2014-09-21 20:06 ` Thomas Petazzoni
2014-10-05 21:47 ` Thomas Petazzoni
2014-10-06 8:37 ` David Bachelart
2014-10-06 10:33 ` Thomas Petazzoni
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.