[Buildroot] [PATCH] package/opencv3: fix build for x86_64 target

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Fri May 12 10:27:23 UTC 2017


Hello,

On Fri, 12 May 2017 11:20:54 +0200, Thomas Petazzoni wrote:

> On Thu,  4 May 2017 23:53:33 +0200, Samuel Martin wrote:
> > This change backports a fix from upstream fixing missing symbols due to
> > wrong compiler optimization flags.
> > 
> > On Buildroot side, also remove deprecated options passed on the
> > configure command.
> > 
> > Fixes:
> >   http://autobuild.buildroot.net/results/af9/af950c6172a578c4eeeb9440b6990ceb48263fa6/build-end.log
> >   http://autobuild.buildroot.net/results/e88/e8832302cd4f30cec076f67965f6aacbaca5b955/build-end.log
> > 
> > Cc: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
> > Signed-off-by: Samuel Martin <s.martin49 at gmail.com>
> > ---
> >  .../0001-cmake-support-multiple-CPU-targets.patch  | 2501 ++++++++++++++++++++
> >  package/opencv3/opencv3.mk                         |   16 +-
> >  2 files changed, 2502 insertions(+), 15 deletions(-)
> >  create mode 100644 package/opencv3/0001-cmake-support-multiple-CPU-targets.patch  
> 
> This patch does more harm than good: it breaks a lot of configurations.
> See http://autobuild.buildroot.net/?reason=opencv3-3.2.0.
> 
> I'll revert later today unless a better fix is proposed.

I have found and fixed the problem: your patch contains a file rename,
and patch < 2.7 don't support file renames. So I've re-generated the
patch without file renames (git format-patch --no-renames). I verified
that it indeed fixes the problem on build machines with patch < 2.7.

Best regards,

Thomas Petazzoni
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com



More information about the buildroot mailing list