From mboxrd@z Thu Jan 1 00:00:00 1970 From: Romain Naour Date: Fri, 15 May 2015 21:22:56 +0200 Subject: [Buildroot] [RFC] package/boost: fix powerpc uClibc build issue In-Reply-To: <20150515204256.7028a9d1@free-electrons.com> References: <1431709207-25679-1-git-send-email-romain.naour@openwide.fr> <20150515204256.7028a9d1@free-electrons.com> Message-ID: <55564790.3020109@openwide.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hi Thomas, Le 15/05/2015 20:42, Thomas Petazzoni a ?crit : > Dear Romain Naour, > > On Fri, 15 May 2015 19:00:07 +0200, Romain Naour wrote: >> With powerpc target and uClibc toolchain the spirit library doesn't >> build properly due to an invalid field access. >> >> The "bits" field is intended only for internal use in >> include/boost/math/special_functions/sign.hpp not from >> include/boost/spirit/home/support/detail/sign.hpp. >> >> Simply call (boost::math::changesign)(x) from >> include/boost/spirit/home/support/detail/sign.hpp to fixes this. >> >> Fixes: >> http://autobuild.buildroot.net/results/be1/be1069e8528d299f487f431f7e2d793413ccbab8/ >> >> And many more. >> >> Signed-off-by: Romain Naour >> --- >> It's not clear why this issue is only trigged by powerpc >> with uClibc toolchain. >> Also, it's a pity that the spirit library doesn't use directly >> math library instead of using a wrapper. > > My opinion is that this patch is too tricky to be committed without > being checked by upstream Boost developers, especially since there is > no explanation as to why the problem occurs only on PowerPC. > > I would prefer to disable boost-log on PowerPC/uClibc, as proposed by > the patch I submitted shortly after this one. I'm agree with your proposal and I'll wait for upstream comment on this patch: http://sourceforge.net/p/spirit/mailman/message/34116604/ Best regards, Romain > Best regards, > > Thomas >