Re: [oe] [meta-oe][PATCH] opencv: Reorder PACKAGES variable

2016-06-02 Thread Martin Jansa 
Please resend both.

On Thu, Jun 2, 2016 at 1:07 PM, Otavio Salvador <
otavio.salva...@ossystems.com.br> wrote:

> On Thu, Jun 2, 2016 at 7:08 AM, Ricardo Ribalda Delgado
>  wrote:
> > Shall i send a v2 with the empty line?
>
> Martin?
>
> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.brhttp://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
> --
> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-oe][PATCH] opencv: Reorder PACKAGES variable

2016-06-02 Thread Otavio Salvador 
On Thu, Jun 2, 2016 at 7:08 AM, Ricardo Ribalda Delgado
 wrote:
> Shall i send a v2 with the empty line?

Martin?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-oe][PATCH] opencv: Reorder PACKAGES variable

2016-06-02 Thread Ricardo Ribalda Delgado 
Shall i send a v2 with the empty line?


Thanks!

On Wed, Jun 1, 2016 at 3:44 PM, Otavio Salvador
 wrote:
> On Wed, Jun 1, 2016 at 10:13 AM, Ricardo Ribalda Delgado
>  wrote:
>> PACKAGES content order is important, because it determines the
>> destination of the files, when the folders overlaps.
>>
>> Without this patch:
>>
>> NOTE: Executing RunQueue Tasks
>> ERROR: QA Issue: non debug package contains .debug directory:
>> opencv-apps path
>> packages-split/opencv-apps/usr/share/OpenCV/java/.debug/libopencv_java310.so
>> [debug-files]
>> WARNING: QA Issue: opencv-apps: found library in wrong location:
>> /usr/share/OpenCV/java/libopencv_java310.so
>> opencv-apps: found library in wrong location:
>> /usr/share/OpenCV/java/.debug/libopencv_java310.so [libdir]
>> ERROR: QA run found fatal errors. Please consider fixing them.
>> ERROR: Function failed: do_package_qa
>>
>> Fixes: 3132af1d1d15 ("opencv: Fix metapkg dependencies for opencv-java ... ")
>
> Empty line here, please.
>
>> Signed-off-by: Ricardo Ribalda Delgado 
>
>
> Acked-by: Otavio Salvador 
>
>
> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.brhttp://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
> --
> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel



-- 
Ricardo Ribalda
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-oe][PATCH] opencv: Reorder PACKAGES variable

2016-06-01 Thread Otavio Salvador 
On Wed, Jun 1, 2016 at 10:13 AM, Ricardo Ribalda Delgado
 wrote:
> PACKAGES content order is important, because it determines the
> destination of the files, when the folders overlaps.
>
> Without this patch:
>
> NOTE: Executing RunQueue Tasks
> ERROR: QA Issue: non debug package contains .debug directory:
> opencv-apps path
> packages-split/opencv-apps/usr/share/OpenCV/java/.debug/libopencv_java310.so
> [debug-files]
> WARNING: QA Issue: opencv-apps: found library in wrong location:
> /usr/share/OpenCV/java/libopencv_java310.so
> opencv-apps: found library in wrong location:
> /usr/share/OpenCV/java/.debug/libopencv_java310.so [libdir]
> ERROR: QA run found fatal errors. Please consider fixing them.
> ERROR: Function failed: do_package_qa
>
> Fixes: 3132af1d1d15 ("opencv: Fix metapkg dependencies for opencv-java ... ")

Empty line here, please.

> Signed-off-by: Ricardo Ribalda Delgado 


Acked-by: Otavio Salvador 


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

[oe] [meta-oe][PATCH] opencv: Reorder PACKAGES variable

2016-06-01 Thread Ricardo Ribalda Delgado 
PACKAGES content order is important, because it determines the
destination of the files, when the folders overlaps.

Without this patch:

NOTE: Executing RunQueue Tasks
ERROR: QA Issue: non debug package contains .debug directory:
opencv-apps path
packages-split/opencv-apps/usr/share/OpenCV/java/.debug/libopencv_java310.so
[debug-files]
WARNING: QA Issue: opencv-apps: found library in wrong location:
/usr/share/OpenCV/java/libopencv_java310.so
opencv-apps: found library in wrong location:
/usr/share/OpenCV/java/.debug/libopencv_java310.so [libdir]
ERROR: QA run found fatal errors. Please consider fixing them.
ERROR: Function failed: do_package_qa

Fixes: 3132af1d1d15 ("opencv: Fix metapkg dependencies for opencv-java ... ")
Signed-off-by: Ricardo Ribalda Delgado 
---
 meta-oe/recipes-support/opencv/opencv_3.1.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-support/opencv/opencv_3.1.bb 
b/meta-oe/recipes-support/opencv/opencv_3.1.bb
index 512f1b3dea9b..83a300e44801 100644
--- a/meta-oe/recipes-support/opencv/opencv_3.1.bb
+++ b/meta-oe/recipes-support/opencv/opencv_3.1.bb
@@ -80,8 +80,8 @@ export ANT_DIR="${STAGING_DIR_NATIVE}/usr/share/ant/"
 
 TARGET_CC_ARCH += "-I${S}/include "
 
-PACKAGES += "${PN}-samples-dbg ${PN}-samples ${PN}-apps python-opencv \
- ${@bb.utils.contains('PACKAGECONFIG', 'oracle-java', 
'${PN}-java-dbg ${PN}-java', '', d)}"
+PACKAGES += "${@bb.utils.contains('PACKAGECONFIG', 'oracle-java', 
'${PN}-java-dbg ${PN}-java', '', d)} \
+ ${PN}-samples-dbg ${PN}-samples ${PN}-apps python-opencv"
 
 python populate_packages_prepend () {
 cv_libdir = d.expand('${libdir}')
-- 
2.8.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel