Skip to contentSkip to conversation

package/upmpdcli: fix missing dependency of libcurl

PATCH New Jörg Krause 3 participants 4 messages Oct 12, 2014 – Oct 14, 2014

lore
Share & tools
lore

Jörg Krause

author
#0 lore raw

PATCH 1/1 package/upmpdcli: fix missing dependency of libcurl

Signed-off-by J?rg Krause <[email protected]>
---
 package/upmpdcli/Config.in   | 1 +
 package/upmpdcli/upmpdcli.mk | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
package/upmpdcli/Config.in +1 -0
index dbbc9b5..f28c775 100644
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_UPMPDCLI
 	bool "upmpdcli"
 	select BR2_PACKAGE_EXPAT
+	select BR2_PACKAGE_LIBCURL
 	select BR2_PACKAGE_LIBUPNP
 	select BR2_PACKAGE_LIBMPDCLIENT
 	depends on BR2_INSTALL_LIBSTDCPP
package/upmpdcli/upmpdcli.mk +1 -1
index 88a5a16..547752a 100644
@@ -8,7 +8,7 @@ UPMPDCLI_VERSION = 0.8.3
 UPMPDCLI_SITE = http://www.lesbonscomptes.com/upmpdcli/downloads
 UPMPDCLI_LICENSE = GPLv2+
 UPMPDCLI_LICENSE_FILES = COPYING
-UPMPDCLI_DEPENDENCIES = expat libupnp libmpdclient
+UPMPDCLI_DEPENDENCIES = expat libcurl libupnp libmpdclient
 
 # Upmpdcli only runs if user upmpdcli exists
 define UPMPDCLI_USERS
Signature · 1 line
2.1.2
#1 lore raw

PATCH 1/1 package/upmpdcli: fix missing dependency of libcurl

Dear J?rg Krause,

If this is fixing an autobuilder issues, please include the autobuilder result URL in the commit log.

Thanks,

Thomas

Quoted 31 lines · J?rg Krause
Signed-off-by J?rg Krause <[email protected]>
---
 package/upmpdcli/Config.in   | 1 +
 package/upmpdcli/upmpdcli.mk | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
package/upmpdcli/Config.in +1 -0
index dbbc9b5..f28c775 100644
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_UPMPDCLI
 	bool "upmpdcli"
 	select BR2_PACKAGE_EXPAT
+	select BR2_PACKAGE_LIBCURL
 	select BR2_PACKAGE_LIBUPNP
 	select BR2_PACKAGE_LIBMPDCLIENT
 	depends on BR2_INSTALL_LIBSTDCPP
package/upmpdcli/upmpdcli.mk +1 -1
index 88a5a16..547752a 100644
@@ -8,7 +8,7 @@ UPMPDCLI_VERSION = 0.8.3
 UPMPDCLI_SITE = http://www.lesbonscomptes.com/upmpdcli/downloads
 UPMPDCLI_LICENSE = GPLv2+
 UPMPDCLI_LICENSE_FILES = COPYING
-UPMPDCLI_DEPENDENCIES = expat libupnp libmpdclient
+UPMPDCLI_DEPENDENCIES = expat libcurl libupnp libmpdclient
 
 # Upmpdcli only runs if user upmpdcli exists
 define UPMPDCLI_USERS
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
author
#2 lore raw

PATCH 1/1 package/upmpdcli: fix missing dependency of libcurl

Dear Thomas Petazzoni,

Quoted 4 lines · Thomas Petazzoni

Dear J?rg Krause,

If this is fixing an autobuilder issues, please include the autobuilder result URL in the commit log.

Sorry, I forgot. Patch is already committed now. Should I leave it as it
is or?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141013/9dd470fd/attachment.html>

#3 lore raw

PATCH 1/1 package/upmpdcli: fix missing dependency of libcurl

Quoted 5 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"J?rg" == J?rg Krause <[email protected]> writes:

Dear Thomas Petazzoni,
On 10/12/2014 09:37 PM, Thomas Petazzoni wrote:

Quoted 7 lines

Dear J?rg Krause,

If this is fixing an autobuilder issues, please include the autobuilder result URL in the commit log.

Sorry, I forgot. Patch is already committed now. Should I leave it as it is or?

Yes, now it is too late (I forgot and didn't notice Thomas' mail). Just something to keep in mind for future patches.

Signature · 1 line
Bye, Peter Korsgaard