Re: [oe] [PATCH v4 1/4] colord: initial add 1.4.4

2019-10-31 Thread Andreas Müller
On Tue, Oct 29, 2019 at 10:52 PM Khem Raj  wrote:
>
> see
>
> gcc  
> -LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib
>
> -LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/lib
> -Wl,--enable-new-dtags
> -Wl,-rpath-link,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib
>
> -Wl,-rpath-link,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/lib
>
> -Wl,-rpath,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib
>
> -Wl,-rpath,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/lib
> -Wl,-O1 -Wl,--allow-shlib-undefined
> -Wl,--dynamic-linker=TOPDIR/build/tmp/sysroots-uninative/x86_64-linux/lib/ld-linux-x86-64.so.2
> -LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib/pkgconfig/../../../usr/lib
> -lgio-2.0 -lgobject-2.0 -lglib-2.0
> -LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib/pkgconfig/../../../usr/lib
> -llcms2 -lm -LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/build
> -lcolord 
> TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/build/cd_idt8.o
> -o TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/build/cd_idt8
>
> it does show that we are linking -lgobject-2.0 but order seems a bit
> iffy to me. Since it should appeat after .o files.
>
> On Tue, Oct 29, 2019 at 10:37 PM Khem Raj  wrote:
> >
> > it seems its not linking with gobject-2.0 properly, pango had similar
> > problem which was fixed with
> > https://gitlab.gnome.org/GNOME/pango/commit/d0cb6be7431d1a3c711bd45bcf05b34601604037
> >
> > On Tue, Oct 29, 2019 at 4:43 PM Andreas Müller  
> > wrote:
> > >
> > > On Tue, Oct 29, 2019 at 3:15 PM Khem Raj  wrote:
> > > >
> > > > Fails again
> > > >
> > > > https://errors.yoctoproject.org/Errors/Details/274899/
> > > >
> > > Continuing like this does not make sense any more. Need to set up an
> > > environment where I can reproduce this error - may last.
> > >
Good news: Our company's stone-age autobuilder shows the same error.
That opens me the door to a more systematic approach :)

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


Re: [oe] [PATCH v4 1/4] colord: initial add 1.4.4

2019-10-29 Thread Khem Raj
see

gcc  
-LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib

-LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/lib
-Wl,--enable-new-dtags
-Wl,-rpath-link,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib

-Wl,-rpath-link,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/lib

-Wl,-rpath,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib

-Wl,-rpath,TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/lib
-Wl,-O1 -Wl,--allow-shlib-undefined
-Wl,--dynamic-linker=TOPDIR/build/tmp/sysroots-uninative/x86_64-linux/lib/ld-linux-x86-64.so.2
-LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib/pkgconfig/../../../usr/lib
-lgio-2.0 -lgobject-2.0 -lglib-2.0
-LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/recipe-sysroot-native/usr/lib/pkgconfig/../../../usr/lib
-llcms2 -lm -LTOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/build
-lcolord 
TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/build/cd_idt8.o
-o TOPDIR/build/tmp/work/x86_64-linux/colord-native/1.4.4-r0/build/cd_idt8

it does show that we are linking -lgobject-2.0 but order seems a bit
iffy to me. Since it should appeat after .o files.

On Tue, Oct 29, 2019 at 10:37 PM Khem Raj  wrote:
>
> it seems its not linking with gobject-2.0 properly, pango had similar
> problem which was fixed with
> https://gitlab.gnome.org/GNOME/pango/commit/d0cb6be7431d1a3c711bd45bcf05b34601604037
>
> On Tue, Oct 29, 2019 at 4:43 PM Andreas Müller  
> wrote:
> >
> > On Tue, Oct 29, 2019 at 3:15 PM Khem Raj  wrote:
> > >
> > > Fails again
> > >
> > > https://errors.yoctoproject.org/Errors/Details/274899/
> > >
> > Continuing like this does not make sense any more. Need to set up an
> > environment where I can reproduce this error - may last.
> >
> > Andreas
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [PATCH v4 1/4] colord: initial add 1.4.4

2019-10-29 Thread Khem Raj
it seems its not linking with gobject-2.0 properly, pango had similar
problem which was fixed with
https://gitlab.gnome.org/GNOME/pango/commit/d0cb6be7431d1a3c711bd45bcf05b34601604037

On Tue, Oct 29, 2019 at 4:43 PM Andreas Müller  wrote:
>
> On Tue, Oct 29, 2019 at 3:15 PM Khem Raj  wrote:
> >
> > Fails again
> >
> > https://errors.yoctoproject.org/Errors/Details/274899/
> >
> Continuing like this does not make sense any more. Need to set up an
> environment where I can reproduce this error - may last.
>
> Andreas
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [PATCH v4 1/4] colord: initial add 1.4.4

2019-10-29 Thread Andreas Müller
On Tue, Oct 29, 2019 at 3:15 PM Khem Raj  wrote:
>
> Fails again
>
> https://errors.yoctoproject.org/Errors/Details/274899/
>
Continuing like this does not make sense any more. Need to set up an
environment where I can reproduce this error - may last.

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


Re: [oe] [PATCH v4 1/4] colord: initial add 1.4.4

2019-10-29 Thread Khem Raj
Fails again

https://errors.yoctoproject.org/Errors/Details/274899/

On Tue, Oct 29, 2019 at 2:03 AM Andreas Müller  wrote:
>
> * It is a dependency for upcoming gnome-settings-daemon
> * It is used by GNOME and KDE [1] so  move it to meta-oe
> * Getting it to build was a bit tricky. Udev (systemd) is a mandatory
>   dependency and for native we do not support build. To get around native 
> build
>   was stripped down to what's necessary only. Native build is done with the
>   help of configuration in recipe and (old-school) Makefile. This solution was
>   chosen to avoid massive patching of meson build which tends to be a
>   maintenance burden (as experienced at KDE recipes in meta-qt5-extra).
>
> [1] https://www.freedesktop.org/software/colord/intro.html
>
> Signed-off-by: Andreas Müller 
> ---
>
> v1 -> v2: native replace pkg-config by pkg-config-native in Makefile to avoid
> v2 -> v3: fix typo in PACKAGECONFIG[systemd]
> v3 -> v4: add LDFLAGS to native libcolord.so build
>
>  .../recipes-support/colord/colord-native.bb   |  64 +++
>  ...n-cd_icc_create_from_edid-to-avoid-u.patch | 166 ++
>  .../colord/colord-native/Makefile |  48 +
>  meta-oe/recipes-support/colord/colord.bb  |  45 +
>  meta-oe/recipes-support/colord/colord.inc |  13 ++
>  ...Run-native-cd_idt8-cd_create_profile.patch |  59 +++
>  6 files changed, 395 insertions(+)
>  create mode 100644 meta-oe/recipes-support/colord/colord-native.bb
>  create mode 100644 
> meta-oe/recipes-support/colord/colord-native/0001-Move-the-function-cd_icc_create_from_edid-to-avoid-u.patch
>  create mode 100644 meta-oe/recipes-support/colord/colord-native/Makefile
>  create mode 100644 meta-oe/recipes-support/colord/colord.bb
>  create mode 100644 meta-oe/recipes-support/colord/colord.inc
>  create mode 100644 
> meta-oe/recipes-support/colord/colord/0001-Run-native-cd_idt8-cd_create_profile.patch
>
> diff --git a/meta-oe/recipes-support/colord/colord-native.bb 
> b/meta-oe/recipes-support/colord/colord-native.bb
> new file mode 100644
> index 0..845041cc4
> --- /dev/null
> +++ b/meta-oe/recipes-support/colord/colord-native.bb
> @@ -0,0 +1,64 @@
> +require ${BPN}.inc
> +
> +FILESEXTRAPATHS_prepend := "${THISDIR}/${BPN}-native:"
> +
> +inherit meson native
> +
> +DEPENDS += " \
> +meson-native \
> +glib-2.0-native \
> +lcms-native \
> +"
> +
> +SRC_URI += " \
> +file://0001-Move-the-function-cd_icc_create_from_edid-to-avoid-u.patch \
> +file://Makefile;subdir=${BPN}-${PV} \
> +"
> +
> +do_configure() {
> +# we expect meson to fail - but before it extracts project's version in 
> log file
> +meson ${MESONOPTS} "${MESON_SOURCEPATH}" "${B}" ${MESON_CROSS_FILE} 
> ${EXTRA_OEMESON} > /dev/null 2>&1 || true
> +
> +# extract and split version
> +version=`grep 'Project version:' ${B}/meson-logs/meson-log.txt | sed 
> 's|Project version: ||'`
> +major=`echo $version | cut -d. -f1`
> +minor=`echo $version | cut -d. -f2`
> +micro=`echo $version | cut -d. -f3`
> +echo "Project version: $major.$minor.$micro"
> +
> +# extract project name
> +proj_name=`grep 'Project name:' ${B}/meson-logs/meson-log.txt | sed 
> 's|Project name: ||'`
> +
> +# create cd-version.h
> +mkdir -p ${B}/colord
> +   sed ${S}/lib/colord/cd-version.h.in \
> +   -e 's:@CD_MAJOR_VERSION_PRIVATE@:1:g' \
> +   -e 's:@CD_MINOR_VERSION_PRIVATE@:4:g' \
> +   -e 's:@CD_MICRO_VERSION_PRIVATE@:4:g' \
> +   > ${B}/colord/cd-version.h
> +
> +# create config.h based on target build and add what's necessary only
> +localedir=`echo ${datadir}/locale | sed 's:${prefix}/::g'`
> +echo "#define LOCALEDIR \"$localedir\"" >> ${B}/config.h
> +echo "#define GETTEXT_PACKAGE \"colord\"" >> ${B}/config.h
> +echo "#define PACKAGE_NAME \"$proj_name\"" >> ${B}/config.h
> +echo "#define PACKAGE_VERSION \"$version\"" >> ${B}/config.h
> +}
> +
> +do_compile() {
> +oe_runmake -C${S} DESTDIR=${B}
> +}
> +
> +do_install() {
> +version=`grep 'Project version:' ${B}/meson-logs/meson-log.txt | sed 
> 's|Project version: ||'`
> +major=`echo $version | cut -d. -f1`
> +
> +install -d ${D}${libdir}
> +install -m 755 ${B}/libcolord.so ${D}${libdir}/libcolord.so.$version
> +ln -s libcolord.so.$version "${D}/${libdir}/libcolord.so"
> +ln -s libcolord.so.$version "${D}/${libdir}/libcolord.so.$major"
> +
> +install -d ${D}${bindir}
> +install -m 755 ${B}/cd_create_profile ${D}${bindir}/
> +install -m 755 ${B}/cd_idt8 ${D}${bindir}/
> +}
> diff --git 
> a/meta-oe/recipes-support/colord/colord-native/0001-Move-the-function-cd_icc_create_from_edid-to-avoid-u.patch
>  
> b/meta-oe/recipes-support/colord/colord-native/0001-Move-the-function-cd_icc_create_from_edid-to-avoid-u.patch
> new file mode 100644
> index 0..278b90e07
> --- /dev/null
> +++ 
>