package/poco: bump to version 1.6.1
PATCH 1/2 package/poco: bump to version 1.6.1
Also:
* switch from sourceforge to github
* remove hash (github-helper)
* remove upstream applied patches
* re-number remaining patches
* add patch to fix build issue with --bundled config option
--- package/poco/0001-aarch64.patch | 39 ---------- ...-poco-add-the-staging-path-to-search-path.patch | 38 ++++++++++ package/poco/0002-add-cross-build-config.patch | 82 ---------------------- .../0002-poco-don-t-build-debug-libraries.patch | 33 +++++++++ ...nflicting-declaration-with-unbundled-pcre.patch | 56 +++++++++++++++ package/poco/0003-add-staging-search-path.patch | 23 ------ package/poco/0004-dont-build-debug-libs.patch | 19 ----- package/poco/poco.hash | 4 +- package/poco/poco.mk | 10 ++- 9 files changed, 133 insertions(+), 171 deletions(-) delete mode 100644 package/poco/0001-aarch64.patch create mode 100644 package/poco/0001-poco-add-the-staging-path-to-search-path.patch delete mode 100644 package/poco/0002-add-cross-build-config.patch create mode 100644 package/poco/0002-poco-don-t-build-debug-libraries.patch create mode 100644 package/poco/0003-Fix-conflicting-declaration-with-unbundled-pcre.patch delete mode 100644 package/poco/0003-add-staging-search-path.patch delete mode 100644 package/poco/0004-dont-build-debug-libs.patch
package/poco/0001-aarch64.patch deleted +0 -39
@@ -1,39 +0,0 @@ -From 37899eda9563d9134cb5864751b6432cabfaac3e Mon Sep 17 00:00:00 2001 -From: Will Newton <[email protected]> -Date: Tue, 21 May 2013 22:11:05 +0100 -Subject: [PATCH] Foundation/include/Poco/Platform.h: Add support for AArch64. - -Add support for big and little endian AArch64 preprocessor defines. ---- - Foundation/include/Poco/Platform.h | 8 +++++++- - 1 file changed, 7 insertions(+), 1 deletion(-) - -diff --git a/Foundation/include/Poco/Platform.h b/Foundation/include/Poco/Platform.h -index f34f277..8ddb4cb 100644 ---- a/Foundation/include/Poco/Platform.h -+++ b/Foundation/include/Poco/Platform.h -@@ -145,6 +145,7 @@ - #define POCO_ARCH_S390 0x0c - #define POCO_ARCH_SH 0x0d - #define POCO_ARCH_NIOS2 0x0e -+#define POCO_ARCH_AARCH64 0x0f - - - #if defined(__ALPHA) || defined(__alpha) || defined(__alpha__) || defined(_M_ALPHA) -@@ -207,7 +208,12 @@ - #else - #define POCO_ARCH_BIG_ENDIAN 1 - #endif -- -+#elif defined(__AARCH64EL__) -+ #define POCO_ARCH POCO_ARCH_AARCH64 -+ #define POCO_ARCH_LITTLE_ENDIAN 1 -+#elif defined(__AARCH64EB__) -+ #define POCO_ARCH POCO_ARCH_AARCH64 -+ #define POCO_ARCH_BIG_ENDIAN 1 - #endif - - --- -1.8.1.4 -
package/poco/0001-poco-add-the-staging-path-to-search-path.patch new file +38 -0
@@ -0,0 +1,38 @@ +From b8417607ec8840e6a1e27cf03b6958c794a33e49 Mon Sep 17 00:00:00 2001 +From: Baruch Siach <[email protected]> +Date: Tue, 4 Aug 2015 10:14:00 +0200 +Subject: [PATCH 1/2] poco: add the staging path to search path +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Add the mysql headers and client libraries to the search path of the +preprocessor and the linker. The $MYSQL_LIBDIR / $MYSQL_INCIDR variables +must be set from the make command line. + +[Peter: Remove host dirs, add MYSQL_INCDIR] +Signed-off-by: Baruch Siach <[email protected]> +[J?rg: Update to version 1.6.1 from github] +Signed-off-by: J?rg Krause <[email protected]> +--- + Data/MySQL/Makefile | 4 ++-- + 1 file changed, 2 insertions(+), 2 deletions(-) + +diff --git a/Data/MySQL/Makefile b/Data/MySQL/Makefile +index 0b18ca5..094e807 100644 +--- a/Data/MySQL/Makefile ++++ b/Data/MySQL/Makefile +@@ -8,8 +8,8 @@ + + include $(POCO_BASE)/build/rules/global + +-SYSLIBS += -L/usr/local/lib$(LIB64SUFFIX)/mysql -L/usr/lib$(LIB64SUFFIX)/mysql -L/usr/mysql/lib$(LIB64SUFFIX) -L/usr/mysql/lib$(LIB64SUFFIX)/mysql -L/usr/local/mysql/lib$(LIB64SUFFIX) -lmysqlclient +-INCLUDE += -I/usr/local/include/mysql/ -I/usr/include/mysql/ -I/usr/mysql/include/mysql -I/usr/local/mysql/include ++SYSLIBS += -L$(MYSQL_LIBDIR) -lmysqlclient ++INCLUDE += -I$(MYSQL_INCDIR) + SYSFLAGS += -DTHREADSAFE -DNO_TCL + + objects = Binder Extractor SessionImpl Connector \ +-- +2.5.0 +
package/poco/0002-add-cross-build-config.patch deleted +0 -82
82 lines not on this page · raw message
package/poco/0002-poco-don-t-build-debug-libraries.patch new file +33 -0
@@ -0,0 +1,33 @@ +From e84c85cfa6742becbd4f82047ce92290bb3ec19a Mon Sep 17 00:00:00 2001 +From: Baruch Siach <[email protected]> +Date: Tue, 4 Aug 2015 10:15:17 +0200 +Subject: [PATCH 2/2] poco: don't build debug libraries +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Remove the debug libraries build dependency from the default target. + +Signed-off-by: Baruch Siach <[email protected]> +[J?rg: Update to version 1.6.1 from github] +Signed-off-by: J?rg Krause <[email protected]> +--- + build/rules/compile | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/build/rules/compile b/build/rules/compile +index 1945e3c..e0d7fbc 100644 +--- a/build/rules/compile ++++ b/build/rules/compile +@@ -14,7 +14,7 @@ + + all: $(DEFAULT_TARGET) + all_static: static_debug static_release +-all_shared: shared_debug shared_release ++all_shared: shared_release + all_debug: static_debug shared_debug + all_release: static_release shared_release + +-- +2.5.0 +
package/poco/0003-Fix-conflicting-declaration-with-unbundled-pcre.patch new file +56 -0
56 lines not on this page · raw message
package/poco/0003-add-staging-search-path.patch deleted +0 -23
@@ -1,23 +0,0 @@ -From: Baruch Siach <[email protected]> -Subject: [PATCH] poco: add the staging path to search path - -Add the mysql headers and client libraries to the search path of the -preprocessor and the linker. The $MYSQL_LIBDIR / $MYSQL_INCIDR variables -must be set from the make command line. - -[Peter: Remove host dirs, add MYSQL_INCDIR] -Signed-off-by: Baruch Siach <[email protected]> ---- ---- poco-1.4.4-all-dist/Data/MySQL/Makefile 2012-09-20 01:30:42.000000000 +0300 -+++ poco-1.4.4-all/Data/MySQL/Makefile 2012-10-11 14:36:10.343153420 +0200 -@@ -8,8 +8,8 @@ - - include $(POCO_BASE)/build/rules/global - --SYSLIBS += -L/usr/local/lib$(LIB64SUFFIX)/mysql -L/usr/lib$(LIB64SUFFIX)/mysql -L/usr/mysql/lib$(LIB64SUFFIX)/mysql -L/usr/local/mysql/lib$(LIB64SUFFIX) -lmysqlclient --INCLUDE += -I/usr/local/include/mysql/ -I/usr/include/mysql -I/usr/mysql/include/mysql -I/usr/local/mysql/include -+SYSLIBS += -L$(MYSQL_LIBDIR) -lmysqlclient -+INCLUDE += -I$(MYSQL_INCDIR) - SYSFLAGS += -DTHREADSAFE -DNO_TCL - - objects = Binder Extractor SessionImpl Connector \
package/poco/0004-dont-build-debug-libs.patch deleted +0 -19
@@ -1,19 +0,0 @@ -From: Baruch Siach <[email protected]> -Subject: [PATCH] poco: don't build debug libraries - -Remove the debug libraries build dependency from the default target. - -Signed-off-by: Baruch Siach <[email protected]> ---- - ---- poco-1.4.1p1-dist/build/rules/compile 2011-02-09 11:12:24.000000000 +0200 -+++ poco-1.4.1p1/build/rules/compile 2011-08-03 07:51:42.946044176 +0300 -@@ -14,7 +14,7 @@ - - all: $(DEFAULT_TARGET) - all_static: static_debug static_release --all_shared: shared_debug shared_release -+all_shared: shared_release - all_debug: static_debug shared_debug - all_release: static_release shared_release -
package/poco/poco.hash +2 -2
@@ -1,2 +1,2 @@ -# Locally computed: -sha256 6791c35e790b055ae9c55abe0bccc453e9616f0183d85a1eeea69ad084e994fe poco-1.4.6p1-all.tar.gz +# No hash for poco, comes from the github-helper: +none xxx poco-poco-1.6.1-release.tar.gz
package/poco/poco.mk +4 -6
@@ -4,10 +4,8 @@ # ################################################################################ -POCO_VERSION_MAJOR = 1.4.6 -POCO_VERSION = $(POCO_VERSION_MAJOR)p1 -POCO_SOURCE = poco-$(POCO_VERSION)-all.tar.gz -POCO_SITE = http://downloads.sourceforge.net/project/poco/sources/poco-$(POCO_VERSION_MAJOR) +POCO_VERSION = poco-1.6.1-release +POCO_SITE = $(call github,pocoproject,poco,$(POCO_VERSION)) POCO_LICENSE = Boost-v1.0 POCO_LICENSE_FILES = LICENSE POCO_INSTALL_STAGING = YES
@@ -41,7 +39,7 @@ endif define POCO_CONFIGURE_CMDS (cd $(@D); ./configure \ - --config=Linux-CrossEnv \ + --config=Linux \ --prefix=/usr \ --omit="$(POCO_OMIT)" \ $(POCO_CONF_OPTS) \
@@ -51,7 +49,7 @@ define POCO_CONFIGURE_CMDS endef define POCO_BUILD_CMDS - $(MAKE1) POCO_TARGET_OSARCH=$(ARCH) CROSSENV=$(TARGET_CROSS) \ + $(MAKE1) POCO_TARGET_OSARCH=$(ARCH) CROSS_COMPILE=$(TARGET_CROSS) \ MYSQL_LIBDIR=$(STAGING_DIR)/usr/lib/mysql \ MYSQL_INCDIR=$(STAGING_DIR)/usr/include/mysql -C $(@D) endef
Signature · 1 line
2.5.0
PATCH 2/2 package/poco: disable for static build
poco requires dlopen()
Fixes:
http://autobuild.buildroot.net/results/952/952f05efd245ba59991f3c5be02a0572e8b9e544/
--- package/poco/Config.in | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
package/poco/Config.in +3 -2
@@ -3,6 +3,7 @@ config BR2_PACKAGE_POCO depends on BR2_INSTALL_LIBSTDCPP depends on BR2_USE_WCHAR depends on BR2_TOOLCHAIN_HAS_THREADS + depends on !BR2_STATIC_LIBS # dlopen() depends on !(BR2_arc || BR2_bfin || BR2_microblaze || BR2_mipsel \ || BR2_xtensa) select BR2_PACKAGE_ZLIB
@@ -60,8 +61,8 @@ config BR2_PACKAGE_POCO_DATA_MYSQL endif # BR2_PACKAGE_POCO -comment "poco needs a toolchain w/ wchar, threads, C++" +comment "poco needs a toolchain w/ wchar, threads, C++, dynamic library" depends on !BR2_USE_WCHAR || !BR2_INSTALL_LIBSTDCPP \ - || !BR2_TOOLCHAIN_HAS_THREADS + || !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS depends on !(BR2_arc || BR2_bfin || BR2_microblaze || BR2_mipsel \ || BR2_xtensa)
Signature · 1 line
2.5.0
PATCH 2/2 package/poco: disable for static build
Dear J?rg Krause,
Quoted 5 lines · J?rg Krause
poco requires
dlopen()Fixes:
http://autobuild.buildroot.net/results/952/952f05efd245ba59991f3c5be02a0572e8b9e544/Signed-off-by J?rg Krause <[email protected]>
There should be an empty line before the SoB line.
Also, there is a --no-sharedlibs option in poco's configure script that is supposed to support statically linked environment. However, it does not work.
So I've applied your patch after amending the commit log.
Thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
PATCH 1/2 package/poco: bump to version 1.6.1
In reply to Jörg KrauseDear J?rg Krause,
Quoted 3 lines · J?rg Krause
Also:
* switch from sourceforge to github
* remove hash (github-helper)
You should have actually removed the hash file entirely, so I've done that and applied to the next branch.
Thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
PATCH 1/2 package/poco: bump to version 1.6.1
Dear Thomas Petazzoni,
Quoted 9 lines · Thomas Petazzoni
Dear J?rg Krause,
Quoted 3 lines · J?rg Krause
Also:
* switch from sourceforge to github
* remove hash (github-helper)You should have actually removed the hash file entirely, so I've done that and applied to the next branch.
In that case I do not understand the help text from nightly.buildroot.org, 17.4. The .hash file:
"The none hash type is reserved to those archives downloaded from a repository, like a git clone, a subversion checkout? or archives downloaded with the github helper Section 17.19.2, ?How to add a
^^^^^^^^^^^^^
package from GitHub?."
And from the example:
"# No hash for 1234, comes from the github-helper:
none xxx
libfoo-1234.tar.gz"
Best regards
J?rg Krause
PATCH 1/2 package/poco: bump to version 1.6.1
Dear J?rg Krause,
Quoted 6 lines · J?rg Krause
In that case I do not understand the help text from nightly.buildroot.org, 17.4. The .hash file:
"The none hash type is reserved to those archives downloaded from a repository, like a git clone, a subversion checkout? or archives downloaded with the github helper Section 17.19.2, ?How to add a
Quoted 7 lines
^^^^^^^^^^^^^package from GitHub?."
And from the example:
"# No hash for 1234, comes from the github-helper:
none xxx
libfoo-1234.tar.gz"
I think the case where we really need the "none" type is for packages that sometimes get downloaded as a tarball, and sometimes from Github. Look for example at binutils.hash:
# From ftp://gcc.gnu.org/pub/binutils/releases/sha512.sum
sha512 ffe8ef263ef99183e8cc823fe8487ff7d0f7bf9a8efd2853b5f4636aca0023850d13de4eac7d77a5f69413d8a50e6f95bb14569be53df86c0bce38034525ab74 binutils-2.22.tar.bz2 sha512 dec753bbba008f1526b89cf1bd85feba78f362f5333ffdf93953fd131eb755976dec82a0a4ba38c43d2434da007137780cfe674de5414be5cf7ce7fbc6af6d16 binutils-2.23.2.tar.bz2 sha512 5ec95ad47d49b12c4558a8db0ca2109d3ee1955e3776057f3330c4506f8f4d1cf5e505fbf8a16b98403a0fcdeaaf986fe0a22be6456247dbdace63ce1f776b12 binutils-2.24.tar.bz2 sha512 0b36dda0e6d32cd25613c0e64b56b28312515c54d6a159efd3db9a86717f114ab0a0a1f69d08975084d55713ebaeab64e4085c9b3d1c3fa86712869f80eb954d binutils-2.25.1.tar.bz2
# No hash for the ARC variant, comes from the github-helper:
none xxx binutils-arc-2015.06.tar.gz
We want hashes for the tarballs, so we added them. But then, the logic of the hash verification is that if a .hash file exists, then *all* files downloaded by the package should have hashes. If not, then the build is aborted. That's why we added this "none" type.
Now, we could also decided to add just a single "none" entry in the hash file of packages downloaded from Github just to make it clear that there is voluntarily no hash being provided, and therefore clarify between packages that don't have a hash file because it hasn't been added yet from packages that don't have a hash file because they are downloaded from Github. But it means adding quite a bunch of pretty useless .hash files to the ~220+ packages we download from github.
In any case, we should clarify that in the documentation, indeed.
Yann, thoughts?
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
PATCH 1/2 package/poco: bump to version 1.6.1
Thomas, All,
On 2015-08-10 20:54 +0200, Thomas Petazzoni spake thusly:
Quoted 10 lines · J?rg Krause
Quoted 9 lines
In that case I do not understand the help text from nightly.buildroot.org, 17.4. The .hash file:
"The none hash type is reserved to those archives downloaded from a repository, like a git clone, a subversion checkout? or archives downloaded with the github helper Section 17.19.2, ?How to add a
^^^^^^^^^^^^^package from GitHub?."
Quoted 7 lines
Quoted 4 lines
And from the example:
"# No hash for 1234, comes from the github-helper:
none xxx
libfoo-1234.tar.gz"I think the case where we really need the "none" type is for packages that sometimes get downloaded as a tarball, and sometimes from Github.
Yes, that's one of the reasons the 'none' type was added.
Quoted 23 lines
Look for example at binutils.hash:
# From ftp://gcc.gnu.org/pub/binutils/releases/sha512.sum
sha512 ffe8ef263ef99183e8cc823fe8487ff7d0f7bf9a8efd2853b5f4636aca0023850d13de4eac7d77a5f69413d8a50e6f95bb14569be53df86c0bce38034525ab74 binutils-2.22.tar.bz2 sha512 dec753bbba008f1526b89cf1bd85feba78f362f5333ffdf93953fd131eb755976dec82a0a4ba38c43d2434da007137780cfe674de5414be5cf7ce7fbc6af6d16 binutils-2.23.2.tar.bz2 sha512 5ec95ad47d49b12c4558a8db0ca2109d3ee1955e3776057f3330c4506f8f4d1cf5e505fbf8a16b98403a0fcdeaaf986fe0a22be6456247dbdace63ce1f776b12 binutils-2.24.tar.bz2 sha512 0b36dda0e6d32cd25613c0e64b56b28312515c54d6a159efd3db9a86717f114ab0a0a1f69d08975084d55713ebaeab64e4085c9b3d1c3fa86712869f80eb954d binutils-2.25.1.tar.bz2# No hash for the ARC variant, comes from the github-helper:
none xxx binutils-arc-2015.06.tar.gzWe want hashes for the tarballs, so we added them. But then, the logic of the hash verification is that if a .hash file exists, then *all* files downloaded by the package should have hashes. If not, then the build is aborted. That's why we added this "none" type.
Now, we could also decided to add just a single "none" entry in the hash file of packages downloaded from Github just to make it clear that there is voluntarily no hash being provided, and therefore clarify between packages that don't have a hash file because it hasn't been added yet from packages that don't have a hash file because they are downloaded from Github. But it means adding quite a bunch of pretty useless .hash files to the ~220+ packages we download from github.
Quoted 3 lines
In any case, we should clarify that in the documentation, indeed.
Yann, thoughts?
I think the overall goal is to have .hash files for all packages. Having a .hash files for all packages would allow us to make hashes truly mandatory, and make a missing .hash file an error, rather than the warning it is today.
Which in turn would ensure we have hashes for all packages for which it makes sense.
Of course, that would mean adding a 'none' hash for those packages for which we can't have ahash (like the github helper, or any git/svn/... checkouts.
So, what J?rg did was in my opinion correct, even if we did not explicitly document that! ;-)
Regards,
Yann E. MORIN.
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------. | 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. | '------------------------------^-------^------------------^--------------------'
[Buildroot] .hash files for all packages?
Dear Yann E. MORIN,
Quoted 8 lines · Yann E. MORIN
I think the overall goal is to have .hash files for all packages. Having a .hash files for all packages would allow us to make hashes truly mandatory, and make a missing .hash file an error, rather than the warning it is today.
Which in turn would ensure we have hashes for all packages for which it makes sense.
Quoted 6 lines
Of course, that would mean adding a 'none' hash for those packages for which we can't have ahash (like the github helper, or any git/svn/... checkouts.
So, what J?rg did was in my opinion correct, even if we did not explicitly document that! ;-)
Ok. I'm a bit worried about having 220 useless hash files, so I'd like to hear the opinion of others on that. And once we made our decision, document it more clearly.
Thanks,
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
[Buildroot] .hash files for all packages?
Thomas, All,
On 2015-08-11 10:13 +0200, Thomas Petazzoni spake thusly:
Quoted 9 lines · Yann E. MORIN
Quoted 8 lines
I think the overall goal is to have .hash files for all packages. Having a .hash files for all packages would allow us to make hashes truly mandatory, and make a missing .hash file an error, rather than the warning it is today.
Which in turn would ensure we have hashes for all packages for which it makes sense.
Quoted 8 lines
Quoted 6 lines
Of course, that would mean adding a 'none' hash for those packages for which we can't have ahash (like the github helper, or any git/svn/... checkouts.
So, what J?rg did was in my opinion correct, even if we did not explicitly document that! ;-)
Ok. I'm a bit worried about having 220 useless hash files,
The problem we have when there is no .hash file, is that we can't differentiate those two cases;
- there is no hash for this package, because it does not make sense (github helper, git/svn/foo checkout...)
- we forgot to add a hash for that package.
I can understand that having lots of similarly looking files that jsut say "don't do hash checks" might seem a bit worrying at first.
But I do think we really do need a way to explicitly say so.
Adding Gustavo in Cc because we already talked about that with him, and last we discussed this, I think he agreed (well, I have IRC logs to back that claim up! ;-] )
Quoted 3 lines
so I'd like
to hear the opinion of others on that. And once we made our decision, document it more clearly.
Yes, it should be throughly documented.
But note: this will have an impact on BR2_EXTERNAL trees; they will also have to provide .hash files for their packages.
Regards,
Yann E. MORIN.
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------. | 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. | '------------------------------^-------^------------------^--------------------'