From: Peter Korsgaard <peter@korsgaard.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 2/2] flickcurl: new package
Date: Mon, 28 Jul 2014 23:15:54 +0200 [thread overview]
Message-ID: <878und9oc5.fsf@dell.be.48ers.dk> (raw)
In-Reply-To: <c00ebca71aec2bc2434dc90f3b249c2519158323.1406581025.git.yann.morin.1998@free.fr> (Yann E. MORIN's message of "Mon, 28 Jul 2014 22:58:08 +0200")
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
> From: Peter Korsgaard <peter@korsgaard.com>
> We need two patches from upstream git as flickr now requires https access.
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> [yann.morin.1998 at free.fr: fix the kconfig recursive loop when selecting
> a SSL implementation]
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Thanks! One minor issue:
> +++ b/package/flickcurl/flickcurl.mk
> @@ -0,0 +1,23 @@
> +################################################################################
> +#
> +# flickcurl
> +#
> +################################################################################
> +
> +FLICKCURL_VERSION = 1.25
> +FLICKCURL_SITE = http://download.dajobe.org/flickcurl/
> +FLICKCURL_LICENSE = LGPLv2.1+ or GPLv2+ or Apache 2.0+
> +FLICKCURL_LICENSE_FILES = COPYING COPYING.LIB LICENSE-2.0.txt LICENSE.html
> +FLICKCURL_INSTALL_STAGING = YES
> +FLICKCURL_CONFIG_SCRIPTS = flickcurl-config
> +FLICKCURL_CONF_OPT = --without-curl-config --without-xml2-config --without-raptor
> +FLICKCURL_DEPENDENCIES = libcurl libxml2 openssl ca-certificates host-pkgconf
You should no longer unconditionally add openssl to the dependencies. As
the ordering is taken care of by libcurl we can simply remove any of the
ssl libraries here.
Looking at it again, we can actually also remove ca-certificates as it
is only a runtime dependency.
--
Bye, Peter Korsgaard
next prev parent reply other threads:[~2014-07-28 21:15 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-28 20:58 [Buildroot] [PATCH 0/2] flickcurl: new package (branch yem/misc) Yann E. MORIN
2014-07-28 20:58 ` [Buildroot] [PATCH 1/2] package/lftp: inverse the openssl/gnutls selection Yann E. MORIN
2014-07-28 21:17 ` Peter Korsgaard
2014-07-28 20:58 ` [Buildroot] [PATCH 2/2] flickcurl: new package Yann E. MORIN
2014-07-28 21:15 ` Peter Korsgaard [this message]
2014-07-28 21:22 ` Yann E. MORIN
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=878und9oc5.fsf@dell.be.48ers.dk \
--to=peter@korsgaard.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox