Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Robert Yang <liezhi.yang@windriver.com>
To: Jussi Kukkonen <jussi.kukkonen@intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 10/10] glib-2.0: 2.42.1 -> 2.44.0
Date: Wed, 8 Apr 2015 17:32:38 +0800	[thread overview]
Message-ID: <5524F5B6.4000407@windriver.com> (raw)
In-Reply-To: <5524EF4B.8040405@windriver.com>


I've updated the patch in the repo:

   git://git.openembedded.org/openembedded-core-contrib rbt/PU


     glib-2.0: 2.42.1 -> 2.44.0

     Backported a patch to fix build on CentOS' gcc 4.4.7:
     0001-GListModel-roll-back-use-of-type-redefinition.patch

     Signed-off-by: Robert Yang <liezhi.yang@windriver.com>

diff --git 
a/meta/recipes-core/glib-2.0/glib-2.0/0001-GListModel-roll-back-use-of-type-redefinition.patch 
b/meta/recipes-core/glib-2.0/glib-2.0/0001-GListModel-roll-back-use-of-type-redefinit
new file mode 100644
index 0000000..7d95743
--- /dev/null
+++ 
b/meta/recipes-core/glib-2.0/glib-2.0/0001-GListModel-roll-back-use-of-type-redefinition.patch
@@ -0,0 +1,42 @@
+From 4a292721bcf2943bfc05c6a1c859992f28e3efec Mon Sep 17 00:00:00 2001
+From: Ryan Lortie <desrt@desrt.ca>
+Date: Wed, 25 Mar 2015 09:29:49 -0400
+Subject: [PATCH] GListModel: roll back use of type redefinition
+
+We declare the typedefs for GListModel and GListStore in giotypes.h, as
+a matter of convention.  This is not actually required, since the
+typedef is emitted as part of the G_DECLARE_* macros.
+
+The giotypes.h approach is only used to avoid cyclic dependencies
+between headers, which is not a problem in this case.
+
+Type redefinition is a C11 feature, and although it was around in some
+compilers before then, gcc 4.2.1 (from 2007) is apparently still in wide
+use, being the default compiler for OpenBSD.
+
+Eventually, we will probably hit a case where we actually need to
+redefine a type, but since we're not there yet, let's back off a bit.
+
+Upstream-Status: Backport
+
+Signed-off-by: Robert Yang <liezhi.yang@windriver.com>

// RObert

On 04/08/2015 05:05 PM, Robert Yang wrote:
>
>
> On 04/08/2015 03:42 PM, Jussi Kukkonen wrote:
>> On 8 April 2015 at 05:23, Robert Yang <liezhi.yang@windriver.com> wrote:
>>> On 04/08/2015 05:39 AM, Richard Purdie wrote:
>>>> This blew up badly on the autobuilder:
>>>>
>>>> http://errors.yoctoproject.org/Errors/Search/?items=10&query=c8da6d7390ad626c03d5b9b431a97adeaeb60a89
>>>>
>>>
>>> Sorry, the failures are on CentOS 6.6, please ignore this upgrade atm, I
>>> will
>>> find a CentOS 6.x to fix the problem.
>>
>> That's typedef redefinition (C11 feature and supported by gcc for
>> quite a while). Why does CentOS specifically choke on that?
>>
>> Anyway, upstream has decided to roll this back: apparently some gcc
>> versions that are still in use do not support the feature.
>> GLib commit 4a292721bc should help.
>
> Thank you very much, I will backport it to fix the problem.
>
> // Robert
>
>>
>> Jussi
>>
>>


      reply	other threads:[~2015-04-08  9:32 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-07 11:24 [PATCH 00/10] Package Upgrades Robert Yang
2015-04-07 11:24 ` [PATCH 01/10] strace: 4.9 -> 4.10 Robert Yang
2015-04-07 11:24 ` [PATCH 02/10] libuser: 0.60 -> 0.61 Robert Yang
2015-04-07 11:24 ` [PATCH 03/10] patch: 2.7.1 -> 2.7.5 Robert Yang
2015-04-07 11:24 ` [PATCH 04/10] libgcrypt: 1.6.2 -> 1.6.3 Robert Yang
2015-04-07 11:24 ` [PATCH 05/10] gnupg: 2.1.1 -> 2.1.2 Robert Yang
2015-04-07 11:24 ` [PATCH 06/10] socat: 1.7.2.4 -> 1.7.3.0 Robert Yang
2015-04-07 11:24 ` [PATCH 07/10] dhcp: 4.3.1 -> 4.3.2 Robert Yang
2015-04-07 11:24 ` [PATCH 08/10] groff: 1.22.2 -> 1.22.3 Robert Yang
2015-04-08  9:40   ` Richard Purdie
2015-04-07 11:24 ` [PATCH 09/10] man-pages: 3.76 -> 3.82 Robert Yang
2015-04-07 11:24 ` [PATCH 10/10] glib-2.0: 2.42.1 -> 2.44.0 Robert Yang
2015-04-07 21:39   ` Richard Purdie
2015-04-08  2:23     ` Robert Yang
2015-04-08  7:42       ` Jussi Kukkonen
2015-04-08  9:05         ` Robert Yang
2015-04-08  9:32           ` Robert Yang [this message]

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=5524F5B6.4000407@windriver.com \
    --to=liezhi.yang@windriver.com \
    --cc=jussi.kukkonen@intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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