[Buildroot] [PATCH 2/2] flickcurl: new package
Yann E. MORIN
yann.morin.1998 at free.fr
Mon Jul 28 21:22:41 UTC 2014
Peter, All,
On 2014-07-28 23:15 +0200, Peter Korsgaard spake thusly:
> >>>>> "Yann" == Yann E MORIN <yann.morin.1998 at free.fr> writes:
>
> > From: Peter Korsgaard <peter at korsgaard.com>
> > We need two patches from upstream git as flickr now requires https access.
>
> > Signed-off-by: Peter Korsgaard <peter at 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 at 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.
Doh, I was too hasty with that (I wanted you to see it.)
> Looking at it again, we can actually also remove ca-certificates as it
> is only a runtime dependency.
Yep, will do.
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. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list