Bug#865414: otb: fails to locate OpenCV 3.1 on non-x86

[Mattia Rizzolo]

Control: unblock 841733 from -1

On Wed, Jun 21, 2017 at 11:34:15AM +0200, Bas Couwenberg wrote:
> On 2017-06-21 11:09, Mattia Rizzolo wrote:
> > Trying to rebuild otb with OpenCV 3.1 (as found in experimental) fails
> > on non-x86 due to cmake not locating it.
> 
> I'm aware of this issue, and it has been fixed upstream more generically
> than with my patch.
> 
> Although they still set the PATH_SUFFIXES, see:
> 
> https://git.orfeo-toolbox.org/otb.git/commitdiff/f0541d761ee69f3300d0cdfd3c869140bec29c2d


Right, that also works, even if I don't see why one would want to keep
using PATH_SUFFIXES while using the standard library paths…

> OTB can only be built on i386 & amd64 because ITK4 is only available there,
> otb is not a blocker for the OpenCV transition because it builds everywhere
> where it built before.
> 
> It doesn't look like insighttoolkit4 in experimental has included the
> changes from Ubuntu to support more than amd64 & i386, so we don't have to
> handle other architectures in OTB yet.

Oh, right.
I'm doing the transition in Ubuntu right now, and that's where I spotted
this.  You're right, it's not required for Debian just yet (until
somebody get it to build on non-x86).

> I consider applying your patch as it's a more correct fix for the OpenCV
> issue, but it's not strictly required.

Yeah, feel free to wait for the new upstream containing that, removing
the blocker from the transition bug.

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-grass-devel/attachments/20170621/7000069c/attachment.sig>