Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] Make libmicrohttpd version configurable
@ 2012-03-06 17:18 rolandu at gmail.com
  2012-03-06 17:20 ` Thomas Petazzoni
  0 siblings, 1 reply; 4+ messages in thread
From: rolandu at gmail.com @ 2012-03-06 17:18 UTC (permalink / raw)
  To: buildroot

From: Roland Uuesoo <roland@martem.eu>

Signed-off-by: Roland Uuesoo <roland@martem.eu>
---
 package/libmicrohttpd/Config.in        |    7 +++++++
 package/libmicrohttpd/libmicrohttpd.mk |    2 +-
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/package/libmicrohttpd/Config.in b/package/libmicrohttpd/Config.in
index cfedc8f..96cd408 100644
--- a/package/libmicrohttpd/Config.in
+++ b/package/libmicrohttpd/Config.in
@@ -6,6 +6,13 @@ config BR2_PACKAGE_LIBMICROHTTPD
 
 	  http://www.gnu.org/software/libmicrohttpd/
 
+config BR2_PACKAGE_LIBMICROHTTPD_VERSION
+	string "libmicrohttpd version"
+	depends on BR2_PACKAGE_LIBMICROHTTPD
+	default 0.4.6
+	help
+	  Specify libmicrohttpd version
+
 config BR2_PACKAGE_LIBMICROHTTPD_SSL
        bool "https support"
        depends on BR2_PACKAGE_LIBMICROHTTPD
diff --git a/package/libmicrohttpd/libmicrohttpd.mk b/package/libmicrohttpd/libmicrohttpd.mk
index ff2b55d..739d779 100644
--- a/package/libmicrohttpd/libmicrohttpd.mk
+++ b/package/libmicrohttpd/libmicrohttpd.mk
@@ -3,7 +3,7 @@
 # libmicrohttpd
 #
 #############################################################
-LIBMICROHTTPD_VERSION = 0.4.6
+LIBMICROHTTPD_VERSION = $(BR2_PACKAGE_LIBMICROHTTPD_VERSION)
 LIBMICROHTTPD_SOURCE = libmicrohttpd-$(LIBMICROHTTPD_VERSION).tar.gz
 LIBMICROHTTPD_SITE = $(BR2_GNU_MIRROR)/libmicrohttpd
 LIBMICROHTTPD_INSTALL_STAGING = YES
-- 
1.7.5.4

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

* [Buildroot] [PATCH] Make libmicrohttpd version configurable
  2012-03-06 17:18 [Buildroot] [PATCH] Make libmicrohttpd version configurable rolandu at gmail.com
@ 2012-03-06 17:20 ` Thomas Petazzoni
       [not found]   ` <CAG0mjpva6VP1u7EK9oOXUqeYYypCiC2yuE7ag4UtBqCkgc52UA@mail.gmail.com>
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2012-03-06 17:20 UTC (permalink / raw)
  To: buildroot

Le Tue,  6 Mar 2012 19:18:48 +0200,
rolandu at gmail.com a ?crit :

> From: Roland Uuesoo <roland@martem.eu>

Except for some particular packages such as U-Boot or the kernel, which
are highly hardware-dependent, we generally don't like having multiple
version support for userspace libraries/applications.

Could you give us details as to why this would be needed specifically
for libmicrohttpd?

Thanks,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

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

* [Buildroot] [PATCH] Make libmicrohttpd version configurable
       [not found]   ` <CAG0mjpva6VP1u7EK9oOXUqeYYypCiC2yuE7ag4UtBqCkgc52UA@mail.gmail.com>
@ 2012-03-06 17:49     ` Thomas Petazzoni
  2012-03-06 18:14       ` Roland
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2012-03-06 17:49 UTC (permalink / raw)
  To: buildroot

Hello Roland,

Please don't reply just to me: keep the list in Cc.

Le Tue, 6 Mar 2012 19:38:39 +0200,
Roland Uuesoo <rolandu@gmail.com> a ?crit :

> Version 0.4.6 is 2 years old (13 march 2010).
> 
> A lot of bugs/security problems have been fixed.
> 
> And there are new features added.

Ok.

> Made it configurable because someone might need the old version.

Then just bump the version to whatever more recent version is
available. We don't support multiple versions per package in Buildroot,
because it would be a maintenance nightmare. For example: each version
will need different patches, different configuration options, etc.

So we have decided to keep things simple and support only one version
at a time, except for U-Boot, Barebox, kernel, and critical things like
that. People doing a project with Buildroot are anyway forking a
particular version of Buildroot to ensure that they are fixed on a
given version of all the components they are using in their system.

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

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

* [Buildroot] [PATCH] Make libmicrohttpd version configurable
  2012-03-06 17:49     ` Thomas Petazzoni
@ 2012-03-06 18:14       ` Roland
  0 siblings, 0 replies; 4+ messages in thread
From: Roland @ 2012-03-06 18:14 UTC (permalink / raw)
  To: buildroot

Ok, didn't know about the 'one version only policy'.

Le Tue, 6 Mar 2012 19:38:39 +0200, Roland Uuesoo <rolandu@gmail.com> a 
?crit :
>> Version 0.4.6 is 2 years old (13 march 2010).
>>
>> A lot of bugs/security problems have been fixed.
>>
>> And there are new features added.
> Ok.
>
>> Made it configurable because someone might need the old version.
> Then just bump the version to whatever more recent version is
> available. We don't support multiple versions per package in Buildroot,
> because it would be a maintenance nightmare. For example: each version
> will need different patches, different configuration options, etc.
>
> So we have decided to keep things simple and support only one version
> at a time, except for U-Boot, Barebox, kernel, and critical things like
> that. People doing a project with Buildroot are anyway forking a
> particular version of Buildroot to ensure that they are fixed on a
> given version of all the components they are using in their system.
>
> Best regards,
>
> Thomas

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

end of thread, other threads:[~2012-03-06 18:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-06 17:18 [Buildroot] [PATCH] Make libmicrohttpd version configurable rolandu at gmail.com
2012-03-06 17:20 ` Thomas Petazzoni
     [not found]   ` <CAG0mjpva6VP1u7EK9oOXUqeYYypCiC2yuE7ag4UtBqCkgc52UA@mail.gmail.com>
2012-03-06 17:49     ` Thomas Petazzoni
2012-03-06 18:14       ` Roland

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox