* [Buildroot] [PATCH 0/2] flickcurl: new package (branch yem/misc)
@ 2014-07-28 20:58 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 20:58 ` [Buildroot] [PATCH 2/2] flickcurl: new package Yann E. MORIN
0 siblings, 2 replies; 6+ messages in thread
From: Yann E. MORIN @ 2014-07-28 20:58 UTC (permalink / raw)
To: buildroot
Hello All!
Update to the flickcurl package from Peter, with the kconfig recursive
dependency issue fixed byt tweaking the "offender": lftp.
Regards,
Yann E. MORIN.
The following changes since commit abded6bfead49d61c24a45d811a83ca082adf677:
icu: add an option to add a custom data library file (2014-07-28 22:43:04 +0200)
are available in the git repository at:
git://ymorin.is-a-geek.org/buildroot yem/misc
for you to fetch changes up to c00ebca71aec2bc2434dc90f3b249c2519158323:
flickcurl: new package (2014-07-28 22:55:23 +0200)
----------------------------------------------------------------
Peter Korsgaard (1):
flickcurl: new package
Yann E. MORIN (1):
package/lftp: inverse the openssl/gnutls selection
package/Config.in | 1 +
package/flickcurl/Config.in | 19 ++++++++++
...ttps-for-all-service-endpoints.-HTTP-EOL-.patch | 40 ++++++++++++++++++++++
...auth-services-are-on-https-www.flickr.com.patch | 30 ++++++++++++++++
package/flickcurl/flickcurl.mk | 23 +++++++++++++
package/lftp/Config.in | 2 +-
6 files changed, 114 insertions(+), 1 deletion(-)
create mode 100644 package/flickcurl/Config.in
create mode 100644 package/flickcurl/flickcurl-0001-Switch-to-https-for-all-service-endpoints.-HTTP-EOL-.patch
create mode 100644 package/flickcurl/flickcurl-0002-Oauth-services-are-on-https-www.flickr.com.patch
create mode 100644 package/flickcurl/flickcurl.mk
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 6+ messages in thread* [Buildroot] [PATCH 1/2] package/lftp: inverse the openssl/gnutls selection 2014-07-28 20:58 [Buildroot] [PATCH 0/2] flickcurl: new package (branch yem/misc) Yann E. MORIN @ 2014-07-28 20:58 ` 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 1 sibling, 1 reply; 6+ messages in thread From: Yann E. MORIN @ 2014-07-28 20:58 UTC (permalink / raw) To: buildroot openssl is more widely used than gnutls. Other packages prefer openssl over gnutls and would like to do the select in the opposite direction. So switch lftp to use openssl by default, and only revert to using gnutls if explicitly selected by the user. Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> --- package/lftp/Config.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/lftp/Config.in b/package/lftp/Config.in index 6a65edf..fb4ac13 100644 --- a/package/lftp/Config.in +++ b/package/lftp/Config.in @@ -5,7 +5,7 @@ config BR2_PACKAGE_LFTP depends on BR2_USE_MMU # fork() select BR2_PACKAGE_READLINE select BR2_PACKAGE_ZLIB - select BR2_PACKAGE_GNUTLS if !BR2_PACKAGE_OPENSSL + select BR2_PACKAGE_OPENSSL if !BR2_PACKAGE_GNUTLS help LFTP is a sophisticated ftp/http client, and a file transfer program supporting a number of network protocols. Like BASH, it has job -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/2] package/lftp: inverse the openssl/gnutls selection 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 0 siblings, 0 replies; 6+ messages in thread From: Peter Korsgaard @ 2014-07-28 21:17 UTC (permalink / raw) To: buildroot >>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes: > openssl is more widely used than gnutls. Other packages prefer openssl > over gnutls and would like to do the select in the opposite direction. > So switch lftp to use openssl by default, and only revert to using > gnutls if explicitly selected by the user. > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Acked-by: Peter Korsgaard <peter@korsgaard.com> -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 2/2] flickcurl: new package 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 20:58 ` Yann E. MORIN 2014-07-28 21:15 ` Peter Korsgaard 1 sibling, 1 reply; 6+ messages in thread From: Yann E. MORIN @ 2014-07-28 20:58 UTC (permalink / raw) To: buildroot 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> --- package/Config.in | 1 + package/flickcurl/Config.in | 19 ++++++++++ ...ttps-for-all-service-endpoints.-HTTP-EOL-.patch | 40 ++++++++++++++++++++++ ...auth-services-are-on-https-www.flickr.com.patch | 30 ++++++++++++++++ package/flickcurl/flickcurl.mk | 23 +++++++++++++ 5 files changed, 113 insertions(+) create mode 100644 package/flickcurl/Config.in create mode 100644 package/flickcurl/flickcurl-0001-Switch-to-https-for-all-service-endpoints.-HTTP-EOL-.patch create mode 100644 package/flickcurl/flickcurl-0002-Oauth-services-are-on-https-www.flickr.com.patch create mode 100644 package/flickcurl/flickcurl.mk diff --git a/package/Config.in b/package/Config.in index 559b895..11087da 100644 --- a/package/Config.in +++ b/package/Config.in @@ -753,6 +753,7 @@ menu "Networking" source "package/cppzmq/Config.in" source "package/czmq/Config.in" source "package/filemq/Config.in" + source "package/flickcurl/Config.in" source "package/fmlib/Config.in" source "package/geoip/Config.in" source "package/glib-networking/Config.in" diff --git a/package/flickcurl/Config.in b/package/flickcurl/Config.in new file mode 100644 index 0000000..7b91710 --- /dev/null +++ b/package/flickcurl/Config.in @@ -0,0 +1,19 @@ +config BR2_PACKAGE_FLICKCURL + bool "flickcurl" + select BR2_PACKAGE_LIBCURL + select BR2_PACKAGE_LIBXML2 + select BR2_PACKAGE_OPENSSL if !(BR2_PACKAGE_GNUTLS || BR2_PACKAGE_LIBNSS) + select BR2_PACKAGE_CA_CERTIFICATES # runtime + help + Flickcurl is a C library for the Flickr API, handling + creating the requests, signing, token management, calling + the API, marshalling request parameters and decoding + responses. + + http://librdf.org/flickcurl/ + +config BR2_PACKAGE_FLICKCURL_UTILS + bool "flickcurl utilities" + depends on BR2_PACKAGE_FLICKCURL + help + Install flickcurl utilites (flickcurl + flickrdf) to target. diff --git a/package/flickcurl/flickcurl-0001-Switch-to-https-for-all-service-endpoints.-HTTP-EOL-.patch b/package/flickcurl/flickcurl-0001-Switch-to-https-for-all-service-endpoints.-HTTP-EOL-.patch new file mode 100644 index 0000000..576b611 --- /dev/null +++ b/package/flickcurl/flickcurl-0001-Switch-to-https-for-all-service-endpoints.-HTTP-EOL-.patch @@ -0,0 +1,40 @@ +From 1f5e208a4709a3030258024ea63834aab41bca3c Mon Sep 17 00:00:00 2001 +From: Dave Beckett <dave@dajobe.org> +Date: Sat, 31 May 2014 12:10:35 -0700 +Subject: [PATCH] Switch to https for all service endpoints. HTTP EOL + 2014-06-27 + +2014-04-30 Flickr announced that it would be HTTPS only from 2014-06-27 +http://code.flickr.net/2014/04/30/flickr-api-going-ssl-only-on-june-27th-2014/ + +Signed-off-by: Peter Korsgaard <peter@korsgaard.com> +--- + src/common.c | 12 ++++++------ + 1 file changed, 6 insertions(+), 6 deletions(-) + +diff --git a/src/common.c b/src/common.c +index ef05e08..245db18 100644 +--- a/src/common.c ++++ b/src/common.c +@@ -73,12 +73,12 @@ const char* const flickcurl_home_url_string = "http://librdf.org/flickcurl/"; + const char* const flickcurl_version_string = VERSION; + + +-const char* const flickcurl_flickr_service_uri = "http://www.flickr.com/services/rest/"; +-const char* const flickcurl_flickr_upload_service_uri = "http://api.flickr.com/services/upload/"; +-const char* const flickcurl_flickr_replace_service_uri = "http://api.flickr.com/services/replace/"; +-const char* const flickcurl_flickr_oauth_request_token_uri = "http://www.flickr.com/services/oauth/request_token"; +-const char* const flickcurl_flickr_oauth_authorize_uri = "http://www.flickr.com/services/oauth/authorize"; +-const char* const flickcurl_flickr_oauth_access_token_uri = "http://www.flickr.com/services/oauth/access_token"; ++const char* const flickcurl_flickr_service_uri = "https://api.flickr.com/services/rest/"; ++const char* const flickcurl_flickr_upload_service_uri = "https://up.flickr.com/services/upload/"; ++const char* const flickcurl_flickr_replace_service_uri = "https://up.flickr.com/services/replace/"; ++const char* const flickcurl_flickr_oauth_request_token_uri = "https://api.flickr.com/services/oauth/request_token"; ++const char* const flickcurl_flickr_oauth_authorize_uri = "https://api.flickr.com/services/oauth/authorize"; ++const char* const flickcurl_flickr_oauth_access_token_uri = "https://api.flickr.com/services/oauth/access_token"; + + + static void +-- +2.0.0 + diff --git a/package/flickcurl/flickcurl-0002-Oauth-services-are-on-https-www.flickr.com.patch b/package/flickcurl/flickcurl-0002-Oauth-services-are-on-https-www.flickr.com.patch new file mode 100644 index 0000000..1ed9af2 --- /dev/null +++ b/package/flickcurl/flickcurl-0002-Oauth-services-are-on-https-www.flickr.com.patch @@ -0,0 +1,30 @@ +From 3e65bbfc1c556ac8bd1d30db66d6e63957066b75 Mon Sep 17 00:00:00 2001 +From: Dave Beckett <dave@dajobe.org> +Date: Sat, 31 May 2014 12:56:22 -0700 +Subject: [PATCH] Oauth services are on https www.flickr.com + +Signed-off-by: Peter Korsgaard <peter@korsgaard.com> +--- + src/common.c | 6 +++--- + 1 file changed, 3 insertions(+), 3 deletions(-) + +diff --git a/src/common.c b/src/common.c +index 245db18..c697a7d 100644 +--- a/src/common.c ++++ b/src/common.c +@@ -76,9 +76,9 @@ const char* const flickcurl_version_string = VERSION; + const char* const flickcurl_flickr_service_uri = "https://api.flickr.com/services/rest/"; + const char* const flickcurl_flickr_upload_service_uri = "https://up.flickr.com/services/upload/"; + const char* const flickcurl_flickr_replace_service_uri = "https://up.flickr.com/services/replace/"; +-const char* const flickcurl_flickr_oauth_request_token_uri = "https://api.flickr.com/services/oauth/request_token"; +-const char* const flickcurl_flickr_oauth_authorize_uri = "https://api.flickr.com/services/oauth/authorize"; +-const char* const flickcurl_flickr_oauth_access_token_uri = "https://api.flickr.com/services/oauth/access_token"; ++const char* const flickcurl_flickr_oauth_request_token_uri = "https://www.flickr.com/services/oauth/request_token"; ++const char* const flickcurl_flickr_oauth_authorize_uri = "https://www.flickr.com/services/oauth/authorize"; ++const char* const flickcurl_flickr_oauth_access_token_uri = "https://www.flickr.com/services/oauth/access_token"; + + + static void +-- +2.0.0 + diff --git a/package/flickcurl/flickcurl.mk b/package/flickcurl/flickcurl.mk new file mode 100644 index 0000000..7fb212f --- /dev/null +++ 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 + +ifeq ($(BR2_PACKAGE_FLICKCURL_UTILS),) +define FLICKCURL_REMOVE_UTILS + rm -f $(TARGET_DIR)/usr/bin/flickcurl $(TARGET_DIR)/usr/bin/flickrdf +endef +FLICKCURL_POST_INSTALL_TARGET_HOOKS += FLICKCURL_REMOVE_UTILS +endif + +$(eval $(autotools-package)) -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 2/2] flickcurl: new package 2014-07-28 20:58 ` [Buildroot] [PATCH 2/2] flickcurl: new package Yann E. MORIN @ 2014-07-28 21:15 ` Peter Korsgaard 2014-07-28 21:22 ` Yann E. MORIN 0 siblings, 1 reply; 6+ messages in thread From: Peter Korsgaard @ 2014-07-28 21:15 UTC (permalink / raw) To: buildroot >>>>> "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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 2/2] flickcurl: new package 2014-07-28 21:15 ` Peter Korsgaard @ 2014-07-28 21:22 ` Yann E. MORIN 0 siblings, 0 replies; 6+ messages in thread From: Yann E. MORIN @ 2014-07-28 21:22 UTC (permalink / raw) To: buildroot Peter, All, On 2014-07-28 23:15 +0200, Peter Korsgaard spake thusly: > >>>>> "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. 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. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-28 21:22 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 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 2014-07-28 21:22 ` Yann E. MORIN
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.