Skip to contentSkip to conversation

package/capnproto: increase gcc version dependency

PATCH New Koen Martens 2 participants 4 messages Aug 21, 2019 – Sep 10, 2019

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/capnproto: increase gcc version dependency

The new version of capnproto requires c++14, therefore the dependency has been increase from at least gcc 4.8 to at least gcc 5.x.

This patch should be applied to a branch that already has the patch that increases the capnproto version to 0.7.0. At the time of writing this, that means the next branch.

---
 package/capnproto/Config.in | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
package/capnproto/Config.in +3 -3
index a979a334ff..a81b7c9d96 100644
@@ -15,10 +15,10 @@ config BR2_PACKAGE_CAPNPROTO
 
 	  https://capnproto.org/index.html
 
-comment "capnproto needs host and target gcc >= 4.8 w/ C++, threads, atomic"
+comment "capnproto needs host and target gcc >= 5 w/ C++, threads, atomic"
 	depends on BR2_USE_MMU
-	depends on !BR2_HOST_GCC_AT_LEAST_4_8 || \
-		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || \
+	depends on !BR2_HOST_GCC_AT_LEAST_5 || \
+		!BR2_TOOLCHAIN_GCC_AT_LEAST_5 || \
 		!BR2_INSTALL_LIBSTDCPP || \
 		!BR2_TOOLCHAIN_HAS_THREADS || \
 		!BR2_TOOLCHAIN_HAS_ATOMIC
Signature · 1 line
2.17.1
#1 lore raw

PATCH 1/1 package/capnproto: increase gcc version dependency

Hi List,

Quoted 8 lines · Koen Martens

The new version of capnproto requires c++14, therefore the dependency has been increase from at least gcc 4.8 to at least gcc 5.x.

This patch should be applied to a branch that already has the patch that increases the capnproto version to 0.7.0. At the time of writing this, that means the next branch.

This patch hasn't been merged yet, but i'm unsubscribing from the list again because the volume is overwhelming my inbox. If anyone wants to discuss this patch, please mail me off-list.

Cheers,

Koen

#2 lore raw

PATCH 1/1 package/capnproto: increase gcc version dependency

Quoted 3 lines · On Mon, 2 Sep 2019 20:06:58 +0200 Koen Martens <[email protected]> wrote:

This patch hasn't been merged yet, but i'm unsubscribing from the list again because the volume is overwhelming my inbox. If anyone wants to discuss this patch, please mail me off-list.

That's quite worrying that people can't stay subscribed due to the traffic, so I'm interested in having a discussion here to understand what can be done about this.

Have you set up a filter to have the mailing list traffic out of your Inbox, and go into a dedicated folder ? The good way of doing that is to filter on the List-Id, so that the e-mails of the mailing list go into a separate folder, but you still receive in your Inbox the e-mails that have both you in To/Cc and the mailing list in To/Cc.

Another option is to remain subscribe, but disable mail delivery, so that you can still post patches, but won't receive any e-mail from the list.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
#3 lore raw

PATCH 1/1 package/capnproto: increase gcc version dependency

In reply to Koen Martens
Quoted 10 lines · On Wed, 21 Aug 2019 15:21:54 +0200 Koen Martens <[email protected]> wrote:

The new version of capnproto requires c++14, therefore the dependency has been increase from at least gcc 4.8 to at least gcc 5.x.

This patch should be applied to a branch that already has the patch that increases the capnproto version to 0.7.0. At the time of writing this, that means the next branch.

Thanks for this contribution. Unfortunately, your patch wasn't correct:
it was only updating the Config.in, but not the dependency of the package itself, so it was in fact doing nothing. Also, c-capnproto had to be fixed as well, as it selects capnproto. In the mean time, Joel Carlson provided correct patches, which I just merged.

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com