All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] protobuf/protobuf-c: bump versions
Date: Fri, 14 Aug 2015 14:37:35 +0200	[thread overview]
Message-ID: <20150814123735.GB4017@free.fr> (raw)
In-Reply-To: <CALoSW66YprLV_WKuWUrPgzT9hL2JzBh69ggg2dNE3rxEkXohXg@mail.gmail.com>

Nimai, All,

On 2015-08-14 08:29 -0400, Nimai Mahajan spake thusly:
> On Fri, Aug 14, 2015 at 8:19 AM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> > On 2015-08-13 21:41 -0400, Nimai Mahajan spake thusly:
[--SNIP--]
> >> diff --git a/package/protobuf/protobuf.mk b/package/protobuf/protobuf.mk
> >> index 0426fce..144e5fb 100644
> >> --- a/package/protobuf/protobuf.mk
> >> +++ b/package/protobuf/protobuf.mk
> >> @@ -4,7 +4,7 @@
> >>  #
> >>  ################################################################################
> >>
> >> -PROTOBUF_VERSION = v2.5.0
> >> +PROTOBUF_VERSION = v2.6.1
> >
> > Last time we tried to bump protobuf to v2.6.0, we got a lot of build
> > failures, so the bump was reverted;
> >     http://git.buildroot.org/buildroot/commit/?id=7b6304af9d69d44ee3040c00b0670f19c02de7d2
> >     http://git.buildroot.org/buildroot/commit/?id=61ae22810a8ca90d3b904d89eb3892c6cf92e87c
> 
> Sorry, I had no idea this was already tried to be pushed up ... I did
> try to search for this but gmane doesn't make that very easy.

No problem! I only knew about the revert because I was investigating the
static build faiulures, and also considered the bump... ;-)

> > Did you test under various conditions, such as:
> >   - static-only  (2.5.0 currently fails in this case, I'm lookimng into it)
> >   - the various supported architectures: arm, i386, mipsel, x86_64
> >   - with various C libraries: uClibc, musl, glibc
> >
> > If you did, then good! :-)
> 
> Bah no I did not :-( I only tested with arm and i386 and glibc as I
> have toolchains for these both. I would love to set up an environment
> where I can test all these combinations more easily before pushing
> something up ... what would be your recommendation for doing that?

You can start with any of those defconfig:

    static based on uClibc:
    http://autobuild.buildroot.org/toolchains/configs/br-arm-full-static.config

    based on musl (but need to manually set static):
    http://autobuild.buildroot.org/toolchains/configs/br-arm-cortex-a9-musl.config

Save them locally, and run:

    make BR2_DEFCONFIG=/path/to/where/you/saved/br-arm-full-static.config defconfig
    make menuconfig
        -- enable protobuf
        -- exit
    make protobuf

(Similarly for the musl build, but you need to enable static-only in the
menuconfig).

> > Otherwise, I'd like this bump to at least be tested in the musl and
> > static-only scenario (to see if that improves the situation...)
> >
> > Also, it seems v2.6.x adds proper support for PowerPC.
> 
> Yes, 2.6.1 also fixes a bunch of bugs (we use protocol buffers in
> production code and can attest to this) so I wanted to get it up there
> but needs a lot more testing with getting built with various
> configurations first.
> 
> I can certainly at least try a musl static build with this.

OK, great! Thanks! :-)

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.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2015-08-14 12:37 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-14  1:41 [Buildroot] [PATCH 1/1] protobuf/protobuf-c: bump versions Nimai Mahajan
2015-08-14 12:19 ` Yann E. MORIN
2015-08-14 12:29   ` Nimai Mahajan
2015-08-14 12:37     ` Yann E. MORIN [this message]
2015-08-14 15:11       ` Nimai Mahajan
2015-08-14 15:42         ` Yann E. MORIN
2015-08-14 15:54           ` Nimai Mahajan
2015-08-14 16:45             ` Yann E. MORIN
2015-08-15  9:28               ` Yann E. MORIN
2015-08-16  0:09                 ` Nimai Mahajan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150814123735.GB4017@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.