From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 4/5] gtest: Upgrade to GitHub HEAD
Date: Thu, 18 Feb 2016 17:18:14 +0100 [thread overview]
Message-ID: <20160218171814.7709b731@free-electrons.com> (raw)
In-Reply-To: <1397733218.10703.1455812066287.JavaMail.zimbra@datacom.ind.br>
Hello,
On Thu, 18 Feb 2016 14:14:26 -0200 (BRST), Carlos Santos wrote:
> 1. Submit a patchset that replaces the sites from googlecode.com to github, maintaining the same package versions (including gtest and gmock, that would be left at version 1.7.0).
>
> This change is easy to apply and could even be brought to the impending Buildroot 2016.02, IMO, since it does not change the package versions, only reduces the risk of build failures due to problems in googlecode.com (I consider this a bug fix). The patchset would update the following patches:
>
> http://patchwork.ozlabs.org/patch/530387/
> http://patchwork.ozlabs.org/patch/530388/
> http://patchwork.ozlabs.org/patch/530389/
> http://patchwork.ozlabs.org/patch/530390/
> http://patchwork.ozlabs.org/patch/530391/
>
>
> 2. Submit a patchset that replaces packages gtest and gmock by a "googletest" package, as suggested by Thomas. This change is quite disruptive for us at DATACOM because it may require changing the recipes that we maintain in $(BR2_EXTERNAL), so I humbly ask you to postpone the change.
>
> 3. Submit a patchset that upgrades googletest to a newer release, hopefully v1.8.0, or to the current HEAD, as decided by Peter Korsgaard and Alan. I will do this after ensuring that the change does not break anything.
Sounds good. Note that (1), (2) and (3) could also be part of the same
patch series if you wish. But it's fine to have (1) first, wait to get
it merge, and only then continue with (2) and (3).
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-02-18 16:18 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-14 21:51 [Buildroot] [PATCH 0/5] Move download links away from Google Code Alan Ott
2015-10-14 21:51 ` [Buildroot] [PATCH 1/5] dsp-tools: Move download link " Alan Ott
2015-10-14 21:51 ` [Buildroot] [PATCH 2/5] gst-dsp: " Alan Ott
2015-10-14 21:51 ` [Buildroot] [PATCH 3/5] gst-omapfb: " Alan Ott
2015-10-14 21:51 ` [Buildroot] [PATCH 4/5] gtest: Upgrade to GitHub HEAD Alan Ott
2015-10-14 21:55 ` Thomas Petazzoni
2015-10-14 22:00 ` Alan Ott
2015-10-15 7:10 ` Thomas Petazzoni
2015-10-19 19:29 ` Carlos Santos
2016-02-17 18:17 ` Carlos Santos
2016-02-17 20:50 ` Thomas Petazzoni
2016-02-18 0:13 ` Alan Ott
2016-02-18 16:14 ` Carlos Santos
2016-02-18 16:18 ` Thomas Petazzoni [this message]
2016-02-18 19:26 ` Carlos Santos
2015-10-14 21:51 ` [Buildroot] [PATCH 5/5] gmock: " Alan Ott
2015-10-14 21:56 ` [Buildroot] [PATCH 0/5] Move download links away from Google Code Thomas Petazzoni
2015-10-14 22:02 ` Alan Ott
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=20160218171814.7709b731@free-electrons.com \
--to=thomas.petazzoni@free-electrons.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox