Message ID | 1437086706-20437-1-git-send-email-joerg.krause@embedded.rocks |
---|---|
State | Accepted |
Headers | show |
Dear Jörg Krause, On Fri, 17 Jul 2015 00:45:06 +0200, Jörg Krause wrote: > Those toolchains use gcc <= 4.5, which is too old as they are missing to > construct an empty std::shared_ptr with a nullptr (0) as an argument. This > is only introduced with gcc-4.6. > > Fixes: > http://autobuild.buildroot.net/results/faf/faf1b071c0a5bd8395d79abf2e6a32169ad7f136 > http://autobuild.buildroot.net/results/247/247b3f1ac3d3a9712e9410eef298fce8d885f92c > http://autobuild.buildroot.net/results/de6/de601c4d4dd7922895892d78d3a37902fb98c7be > http://autobuild.buildroot.net/results/a4d/a4d7bf9863c134f59d285616b9c1f78201bd1a3e > > and many more. > > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> > --- > package/upmpdcli/Config.in | 4 ++++ > 1 file changed, 4 insertions(+) I've applied since it's the way we have been handling this so far. However, during the hackaton, we discussed this a little bit, and we believe we probably need to add some hidden Config.in options like "BR2_PACKAGE_GCC_AT_LEAST_4_5", "BR2_PACKAGE_GCC_AT_LEAST_4_8", so that packages can depend on these options more easily. Thanks! Thomas
Dear Thomas Petazzoni, On Sa, 2015-07-18 at 11:16 +0200, Thomas Petazzoni wrote: > Dear Jörg Krause, > > On Fri, 17 Jul 2015 00:45:06 +0200, Jörg Krause wrote: > > Those toolchains use gcc <= 4.5, which is too old as they are > > missing to > > construct an empty std::shared_ptr with a nullptr (0) as an > > argument. This > > is only introduced with gcc-4.6. > > > > Fixes: > > http://autobuild.buildroot.net/results/faf/faf1b071c0a5bd8395d79abf > > 2e6a32169ad7f136 > > http://autobuild.buildroot.net/results/247/247b3f1ac3d3a9712e9410ee > > f298fce8d885f92c > > http://autobuild.buildroot.net/results/de6/de601c4d4dd7922895892d78 > > d3a37902fb98c7be > > http://autobuild.buildroot.net/results/a4d/a4d7bf9863c134f59d285616 > > b9c1f78201bd1a3e > > > > and many more. > > > > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> > > --- > > package/upmpdcli/Config.in | 4 ++++ > > 1 file changed, 4 insertions(+) > > I've applied since it's the way we have been handling this so far. > However, during the hackaton, we discussed this a little bit, and we > believe we probably need to add some hidden Config.in options like > "BR2_PACKAGE_GCC_AT_LEAST_4_5", "BR2_PACKAGE_GCC_AT_LEAST_4_8", so > that > packages can depend on these options more easily. I see! This would be indeed helpful. Upstream submitted a patch regarding this build error which removes the necessity for a gcc >= 4.6 toolchain. Shall I submit this upstream patch to allow using the excluded toolchains again, or do we just wait until a new version of upmpdcli will be released and we include them again in that case? Best regards Jörg Krause
Hello, On Sun, 19 Jul 2015 09:07:44 +0200, Jörg Krause wrote: > Upstream submitted a patch regarding this build error which removes the > necessity for a gcc >= 4.6 toolchain. Shall I submit this upstream > patch to allow using the excluded toolchains again, or do we just wait > until a new version of upmpdcli will be released and we include them > again in that case? I think you can wait for the new upstream release, and at this point remove the toolchain exclusions. Thanks! Thomas
diff --git a/package/upmpdcli/Config.in b/package/upmpdcli/Config.in index 6464d06..0fe3d10 100644 --- a/package/upmpdcli/Config.in +++ b/package/upmpdcli/Config.in @@ -7,6 +7,8 @@ config BR2_PACKAGE_UPMPDCLI depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2014R1 # Too old gcc depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2013R1 # Ditto depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2 # Ditto + depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201103 # Ditto + depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201009 # Ditto # libupnpp triggers the _gp link issue depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_NIOSII201305 depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_NIOSII201405 @@ -21,6 +23,8 @@ comment "upmpdcli needs a toolchain w/ C++, threads" depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2014R1 # Too old gcc depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2013R1 # Ditto depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2 # Ditto + depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201103 # Ditto + depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_POWERPC201009 # Ditto depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_NIOSII201305 depends on !BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_NIOSII201405 depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS
Those toolchains use gcc <= 4.5, which is too old as they are missing to construct an empty std::shared_ptr with a nullptr (0) as an argument. This is only introduced with gcc-4.6. Fixes: http://autobuild.buildroot.net/results/faf/faf1b071c0a5bd8395d79abf2e6a32169ad7f136 http://autobuild.buildroot.net/results/247/247b3f1ac3d3a9712e9410eef298fce8d885f92c http://autobuild.buildroot.net/results/de6/de601c4d4dd7922895892d78d3a37902fb98c7be http://autobuild.buildroot.net/results/a4d/a4d7bf9863c134f59d285616b9c1f78201bd1a3e and many more. Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> --- package/upmpdcli/Config.in | 4 ++++ 1 file changed, 4 insertions(+)