All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
@ 2015-05-27 17:35 Zoltan Kuscsik
  0 siblings, 0 replies; 8+ messages in thread
From: Zoltan Kuscsik @ 2015-05-27 17:35 UTC (permalink / raw)
  To: openembedded-devel

Updated recipe to fetch Chromium from git.
LASTCHANGE file is now created using the proper
gclient hook.

Change-Id: Ie1790abec002a0dd39c8d0aae3d03f5ebe6f46aa
Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
---
 .../cef3/01_get_svn_version_from_LASTCHANGE.patch  | 32 ------------
 recipes-browser/chromium/cef3_280796.bb            | 57
++++++++++++++++------
 2 files changed, 43 insertions(+), 46 deletions(-)
 delete mode 100644
recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch

diff --git
a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
b/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
deleted file mode 100644
index 9b14429..0000000
--- a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-Fetch the revision from LASTCHANGE file programmatically
-
-Upstream-Status: Pending
-
-Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
-
-diff --git a/cef/tools/make_version_header.py
b/cef/tools/make_version_header.py
-index 84d49f5..395c2cf 100644
---- a/cef/tools/make_version_header.py
-+++ b/cef/tools/make_version_header.py
-@@ -64,6 +64,8 @@ def write_svn_header(header, chrome_version,
cef_version, cpp_header_dir):
-       revision = svn.get_revision()
-     elif git.is_checkout('.'):
-       revision = git.get_svn_revision()
-+    elif os.path.isfile("../build/util/LASTCHANGE"):
-+      revision =
open("../build/util/LASTCHANGE").read().split("=")[1].strip()
-     else:
-       raise Exception('Not a valid checkout')
-
-diff --git a/cef/tools/revision.py b/cef/tools/revision.py
-index 1d94602..eb89e3b 100644
---- a/cef/tools/revision.py
-+++ b/cef/tools/revision.py
-@@ -16,6 +16,8 @@ if os.path.exists(os.path.join('.', '.svn')):
-   sys.stdout.write(svn.get_revision())
- elif os.path.exists(os.path.join('.', '.git')):
-   sys.stdout.write(git.get_svn_revision())
-+elif os.path.isfile("../build/util/LASTCHANGE"):
-+  revision = open("../build/util/LASTCHANGE").read().split("=")[1].strip()
- else:
-   raise Exception('Not a valid checkout')
diff --git a/recipes-browser/chromium/cef3_280796.bb
b/recipes-browser/chromium/cef3_280796.bb
index 9a9efe5..806a4ac 100644
--- a/recipes-browser/chromium/cef3_280796.bb
+++ b/recipes-browser/chromium/cef3_280796.bb
@@ -7,21 +7,22 @@ RDEPENDS_${PN} += "pango cairo fontconfig pciutils
pulseaudio freetype fontconfi
 SRCREV_tools = "99bcb0e676eb396bcf8e1af3903aa4b578aeeee0"
 SRCREV_cef = "bbad53dfca9f98dddcb31a590410fece0a4f0234"
 SRCREV_egl = "a5b81b7617ba6757802b9b5f8c950034d5f961ec"
-SRCREV_FORMAT = "cef_egl_tools"
+SRCREV_FORMAT = "cef_egl"

-SRC_URI = "
http://people.linaro.org/~zoltan.kuscsik/chromium-browser/chromium_rev_${PV}.tar.xz
\
-           git://
github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=src/cef;branch=aura;name=cef
\
-           git://
github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=src/ui/ozone/platform/egl;branch=master;name=egl
\
-           git://
chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
\
-           file://01_get_svn_version_from_LASTCHANGE.patch \
-          file://cef-simple \
-         "
-SRC_URI[md5sum] = "9efbb50283b731042e62b9bd5e312b2f"
-SRC_URI[sha256sum] =
"f608e97dadf6ea4d885b24fd876896d46840fa39bf743ea2025075aee9fb348d"
+SRC_URI = "git://
chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
\
+           git://
github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=${S}/src/cef;branch=aura;name=cef
<http://github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=$%7BS%7D/src/cef;branch=aura;name=cef>
\
+           git://
github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=${S}/src/ui/ozone/platform/egl;branch=master;name=egl\
<http://github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=$%7BS%7D/src/ui/ozone/platform/egl;branch=master;name=egl%5C>
+           file://cef-simple\
+  "

-S = "${WORKDIR}/chromium_rev_${PV}"
+export CHROMIUM_SRC_GIT_URL="https://chromium.googlesource.com/chromium/src
"
+export CHROMIUM_SRC_GIT_COMMIT="e800fe7470fa87dc1ca5b148a7c2c41f603fdcbd"

-do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl SRCREV_tools"
+S = "${WORKDIR}/chromium_rev_${PV}/"
+
+B = "${S}/src/"
+
+do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl"

 GYP_ARCH_DEFINES_armv7a = " target_arch=arm arm_float_abi=hard"
 GYP_ARCH_DEFINES_i586 = " target_arch=ia32"
@@ -30,14 +31,41 @@ export GYP_GENERATORS="ninja"
 export BUILD_TARGET_ARCH="${TARGET_ARCH}"
 export GYP_DEFINES="${GYP_ARCH_DEFINES}
release_extra_cflags='-Wno-error=unused-local-typedefs' sysroot=''"

+do_fetch_chromium_source_from_git() {
+  cd ${S}
+  export PATH=${WORKDIR}/depot_tools:"$PATH"
+
+  # Note: --deps-file needs to set to .DEPS.git for syncing to old
+  # chromium versions
+  gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
+
+  # Runhooks is disabled. Running the hooks will trigger the
+  # gyp_chromium command replaced with cef_create_projects.sh by
+  # cef3.
+
+  gclient sync -f --nohooks --revision src@${CHROMIUM_SRC_GIT_COMMIT}
+
+  # We need the lastchange hook to generate the LASTCHANGE revision info
+  # file.
+
+  python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
+}
+
+addtask fetch_chromium_source_from_git after do_patch before do_configure
+
 do_configure_append() {
        export PATH=${WORKDIR}/depot_tools:"$PATH"
        # End of LD Workaround
        #-----------------------
        # Configure cef
        #------------------------
-       cd cef
-       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
../
+       cd ${S}/src/cef
+
+  # Fatal linker warnings need to be disalbed to avoid linker issu
+  # with GCC 4.9:
+  #
http://lists.openembedded.org/pipermail/openembedded-core/2014-August/095947.html
+
+       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
../ -Ddisable_fatal_linker_warnings=
        cd -
 }

@@ -47,5 +75,6 @@ do_qa_configure() {
 }

 do_compile() {
+ cd ${S}/src
        ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
 }
--
1.9.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
@ 2015-05-27 17:37 Zoltan Kuscsik
  2015-05-28  0:56 ` Khem Raj
  0 siblings, 1 reply; 8+ messages in thread
From: Zoltan Kuscsik @ 2015-05-27 17:37 UTC (permalink / raw)
  To: openembedded-devel

Updated recipe to fetch Chromium from git.
LASTCHANGE file is now created using the proper
gclient hook.

Change-Id: Ie1790abec002a0dd39c8d0aae3d03f5ebe6f46aa
Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
---
 .../cef3/01_get_svn_version_from_LASTCHANGE.patch  | 32 ------------
 recipes-browser/chromium/cef3_280796.bb            | 57
++++++++++++++++------
 2 files changed, 43 insertions(+), 46 deletions(-)
 delete mode 100644
recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch

diff --git
a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
b/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
deleted file mode 100644
index 9b14429..0000000
--- a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-Fetch the revision from LASTCHANGE file programmatically
-
-Upstream-Status: Pending
-
-Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
-
-diff --git a/cef/tools/make_version_header.py
b/cef/tools/make_version_header.py
-index 84d49f5..395c2cf 100644
---- a/cef/tools/make_version_header.py
-+++ b/cef/tools/make_version_header.py
-@@ -64,6 +64,8 @@ def write_svn_header(header, chrome_version,
cef_version, cpp_header_dir):
-       revision = svn.get_revision()
-     elif git.is_checkout('.'):
-       revision = git.get_svn_revision()
-+    elif os.path.isfile("../build/util/LASTCHANGE"):
-+      revision =
open("../build/util/LASTCHANGE").read().split("=")[1].strip()
-     else:
-       raise Exception('Not a valid checkout')
-
-diff --git a/cef/tools/revision.py b/cef/tools/revision.py
-index 1d94602..eb89e3b 100644
---- a/cef/tools/revision.py
-+++ b/cef/tools/revision.py
-@@ -16,6 +16,8 @@ if os.path.exists(os.path.join('.', '.svn')):
-   sys.stdout.write(svn.get_revision())
- elif os.path.exists(os.path.join('.', '.git')):
-   sys.stdout.write(git.get_svn_revision())
-+elif os.path.isfile("../build/util/LASTCHANGE"):
-+  revision = open("../build/util/LASTCHANGE").read().split("=")[1].strip()
- else:
-   raise Exception('Not a valid checkout')
diff --git a/recipes-browser/chromium/cef3_280796.bb
b/recipes-browser/chromium/cef3_280796.bb
index 9a9efe5..806a4ac 100644
--- a/recipes-browser/chromium/cef3_280796.bb
+++ b/recipes-browser/chromium/cef3_280796.bb
@@ -7,21 +7,22 @@ RDEPENDS_${PN} += "pango cairo fontconfig pciutils
pulseaudio freetype fontconfi
 SRCREV_tools = "99bcb0e676eb396bcf8e1af3903aa4b578aeeee0"
 SRCREV_cef = "bbad53dfca9f98dddcb31a590410fece0a4f0234"
 SRCREV_egl = "a5b81b7617ba6757802b9b5f8c950034d5f961ec"
-SRCREV_FORMAT = "cef_egl_tools"
+SRCREV_FORMAT = "cef_egl"

-SRC_URI = "
http://people.linaro.org/~zoltan.kuscsik/chromium-browser/chromium_rev_${PV}.tar.xz
\
-           git://
github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=src/cef;branch=aura;name=cef
\
-           git://
github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=src/ui/ozone/platform/egl;branch=master;name=egl
\
-           git://
chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
\
-           file://01_get_svn_version_from_LASTCHANGE.patch \
-          file://cef-simple \
-         "
-SRC_URI[md5sum] = "9efbb50283b731042e62b9bd5e312b2f"
-SRC_URI[sha256sum] =
"f608e97dadf6ea4d885b24fd876896d46840fa39bf743ea2025075aee9fb348d"
+SRC_URI = "git://
chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
\
+           git://
github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=${S}/src/cef;branch=aura;name=cef
\
+           git://
github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=${S}/src/ui/ozone/platform/egl;branch=master;name=egl\
+           file://cef-simple\
+  "

-S = "${WORKDIR}/chromium_rev_${PV}"
+export CHROMIUM_SRC_GIT_URL="https://chromium.googlesource.com/chromium/src
"
+export CHROMIUM_SRC_GIT_COMMIT="e800fe7470fa87dc1ca5b148a7c2c41f603fdcbd"

-do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl SRCREV_tools"
+S = "${WORKDIR}/chromium_rev_${PV}/"
+
+B = "${S}/src/"
+
+do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl"

 GYP_ARCH_DEFINES_armv7a = " target_arch=arm arm_float_abi=hard"
 GYP_ARCH_DEFINES_i586 = " target_arch=ia32"
@@ -30,14 +31,41 @@ export GYP_GENERATORS="ninja"
 export BUILD_TARGET_ARCH="${TARGET_ARCH}"
 export GYP_DEFINES="${GYP_ARCH_DEFINES}
release_extra_cflags='-Wno-error=unused-local-typedefs' sysroot=''"

+do_fetch_chromium_source_from_git() {
+  cd ${S}
+  export PATH=${WORKDIR}/depot_tools:"$PATH"
+
+  # Note: --deps-file needs to set to .DEPS.git for syncing to old
+  # chromium versions
+  gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
+
+  # Runhooks is disabled. Running the hooks will trigger the
+  # gyp_chromium command replaced with cef_create_projects.sh by
+  # cef3.
+
+  gclient sync -f --nohooks --revision src@${CHROMIUM_SRC_GIT_COMMIT}
+
+  # We need the lastchange hook to generate the LASTCHANGE revision info
+  # file.
+
+  python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
+}
+
+addtask fetch_chromium_source_from_git after do_patch before do_configure
+
 do_configure_append() {
        export PATH=${WORKDIR}/depot_tools:"$PATH"
        # End of LD Workaround
        #-----------------------
        # Configure cef
        #------------------------
-       cd cef
-       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
../
+       cd ${S}/src/cef
+
+  # Fatal linker warnings need to be disalbed to avoid linker issu
+  # with GCC 4.9:
+  #
http://lists.openembedded.org/pipermail/openembedded-core/2014-August/095947.html
+
+       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
../ -Ddisable_fatal_linker_warnings=
        cd -
 }

@@ -47,5 +75,6 @@ do_qa_configure() {
 }

 do_compile() {
+ cd ${S}/src
        ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
 }
--
1.9.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
  2015-05-27 17:37 [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git Zoltan Kuscsik
@ 2015-05-28  0:56 ` Khem Raj
  2015-05-28  7:09   ` Zoltan Kuscsik
  0 siblings, 1 reply; 8+ messages in thread
From: Khem Raj @ 2015-05-28  0:56 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 7003 bytes --]


> On May 27, 2015, at 10:37 AM, Zoltan Kuscsik <zoltan.kuscsik@linaro.org> wrote:
> 
> Updated recipe to fetch Chromium from git.
> LASTCHANGE file is now created using the proper
> gclient hook.
> 
> Change-Id: Ie1790abec002a0dd39c8d0aae3d03f5ebe6f46aa
> Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
> ---
> .../cef3/01_get_svn_version_from_LASTCHANGE.patch  | 32 ------------
> recipes-browser/chromium/cef3_280796.bb            | 57
> ++++++++++++++++------
> 2 files changed, 43 insertions(+), 46 deletions(-)
> delete mode 100644
> recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> 
> diff --git
> a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> b/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> deleted file mode 100644
> index 9b14429..0000000
> --- a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> +++ /dev/null
> @@ -1,32 +0,0 @@
> -Fetch the revision from LASTCHANGE file programmatically
> -
> -Upstream-Status: Pending
> -
> -Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> -
> -diff --git a/cef/tools/make_version_header.py
> b/cef/tools/make_version_header.py
> -index 84d49f5..395c2cf 100644
> ---- a/cef/tools/make_version_header.py
> -+++ b/cef/tools/make_version_header.py
> -@@ -64,6 +64,8 @@ def write_svn_header(header, chrome_version,
> cef_version, cpp_header_dir):
> -       revision = svn.get_revision()
> -     elif git.is_checkout('.'):
> -       revision = git.get_svn_revision()
> -+    elif os.path.isfile("../build/util/LASTCHANGE"):
> -+      revision =
> open("../build/util/LASTCHANGE").read().split("=")[1].strip()
> -     else:
> -       raise Exception('Not a valid checkout')
> -
> -diff --git a/cef/tools/revision.py b/cef/tools/revision.py
> -index 1d94602..eb89e3b 100644
> ---- a/cef/tools/revision.py
> -+++ b/cef/tools/revision.py
> -@@ -16,6 +16,8 @@ if os.path.exists(os.path.join('.', '.svn')):
> -   sys.stdout.write(svn.get_revision())
> - elif os.path.exists(os.path.join('.', '.git')):
> -   sys.stdout.write(git.get_svn_revision())
> -+elif os.path.isfile("../build/util/LASTCHANGE"):
> -+  revision = open("../build/util/LASTCHANGE").read().split("=")[1].strip()
> - else:
> -   raise Exception('Not a valid checkout')
> diff --git a/recipes-browser/chromium/cef3_280796.bb
> b/recipes-browser/chromium/cef3_280796.bb
> index 9a9efe5..806a4ac 100644
> --- a/recipes-browser/chromium/cef3_280796.bb
> +++ b/recipes-browser/chromium/cef3_280796.bb
> @@ -7,21 +7,22 @@ RDEPENDS_${PN} += "pango cairo fontconfig pciutils
> pulseaudio freetype fontconfi
> SRCREV_tools = "99bcb0e676eb396bcf8e1af3903aa4b578aeeee0"
> SRCREV_cef = "bbad53dfca9f98dddcb31a590410fece0a4f0234"
> SRCREV_egl = "a5b81b7617ba6757802b9b5f8c950034d5f961ec"
> -SRCREV_FORMAT = "cef_egl_tools"
> +SRCREV_FORMAT = "cef_egl"
> 
> -SRC_URI = "
> http://people.linaro.org/~zoltan.kuscsik/chromium-browser/chromium_rev_${PV}.tar.xz
> \
> -           git://
> github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=src/cef;branch=aura;name=cef
> \
> -           git://
> github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=src/ui/ozone/platform/egl;branch=master;name=egl
> \
> -           git://
> chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
> \
> -           file://01_get_svn_version_from_LASTCHANGE.patch \
> -          file://cef-simple \
> -         "
> -SRC_URI[md5sum] = "9efbb50283b731042e62b9bd5e312b2f"
> -SRC_URI[sha256sum] =
> "f608e97dadf6ea4d885b24fd876896d46840fa39bf743ea2025075aee9fb348d"
> +SRC_URI = "git://
> chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
> \
> +           git://
> github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=${S}/src/cef;branch=aura;name=cef
> \
> +           git://
> github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=${S}/src/ui/ozone/platform/egl;branch=master;name=egl\
> +           file://cef-simple\
> +  "
> 
> -S = "${WORKDIR}/chromium_rev_${PV}"
> +export CHROMIUM_SRC_GIT_URL="https://chromium.googlesource.com/chromium/src
> "
> +export CHROMIUM_SRC_GIT_COMMIT="e800fe7470fa87dc1ca5b148a7c2c41f603fdcbd"
> 
> -do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl SRCREV_tools"
> +S = "${WORKDIR}/chromium_rev_${PV}/"
> +
> +B = "${S}/src/"
> +
> +do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl"
> 
> GYP_ARCH_DEFINES_armv7a = " target_arch=arm arm_float_abi=hard"
> GYP_ARCH_DEFINES_i586 = " target_arch=ia32"
> @@ -30,14 +31,41 @@ export GYP_GENERATORS="ninja"
> export BUILD_TARGET_ARCH="${TARGET_ARCH}"
> export GYP_DEFINES="${GYP_ARCH_DEFINES}
> release_extra_cflags='-Wno-error=unused-local-typedefs' sysroot=''"
> 
> +do_fetch_chromium_source_from_git() {
> +  cd ${S}
> +  export PATH=${WORKDIR}/depot_tools:"$PATH"
> +
> +  # Note: --deps-file needs to set to .DEPS.git for syncing to old
> +  # chromium versions
> +  gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
> +
> +  # Runhooks is disabled. Running the hooks will trigger the
> +  # gyp_chromium command replaced with cef_create_projects.sh by
> +  # cef3.
> +
> +  gclient sync -f --nohooks --revision src@${CHROMIUM_SRC_GIT_COMMIT}
> +
> +  # We need the lastchange hook to generate the LASTCHANGE revision info
> +  # file.
> +
> +  python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
> +}


small nit. formatting is not correct for this shell function.
> +
> +addtask fetch_chromium_source_from_git after do_patch before do_configure
> +


so we will never patch chromium sources ? if we have to how will we do it ?


> do_configure_append() {
>        export PATH=${WORKDIR}/depot_tools:"$PATH"
>        # End of LD Workaround
>        #-----------------------
>        # Configure cef
>        #------------------------
> -       cd cef
> -       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
> ../
> +       cd ${S}/src/cef
> +
> +  # Fatal linker warnings need to be disalbed to avoid linker issu
> +  # with GCC 4.9:
> +  #
> http://lists.openembedded.org/pipermail/openembedded-core/2014-August/095947.html
> +
> +       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
> ../ -Ddisable_fatal_linker_warnings=
>        cd -
> }
> 
> @@ -47,5 +75,6 @@ do_qa_configure() {
> }
> 
> do_compile() {
> + cd ${S}/src

formatting is inconsistent

>        ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
> }
> --
> 1.9.1
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
  2015-05-28  0:56 ` Khem Raj
@ 2015-05-28  7:09   ` Zoltan Kuscsik
  2015-05-28  7:14     ` Martin Jansa
  2015-06-03 19:47     ` Khem Raj
  0 siblings, 2 replies; 8+ messages in thread
From: Zoltan Kuscsik @ 2015-05-28  7:09 UTC (permalink / raw)
  To: openembedded-devel

On 28 May 2015 at 02:56, Khem Raj <raj.khem@gmail.com> wrote:

>
> > On May 27, 2015, at 10:37 AM, Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
> wrote:
> >
> > Updated recipe to fetch Chromium from git.
> > LASTCHANGE file is now created using the proper
> > gclient hook.
> >
> > Change-Id: Ie1790abec002a0dd39c8d0aae3d03f5ebe6f46aa
> > Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
> > ---
> > .../cef3/01_get_svn_version_from_LASTCHANGE.patch  | 32 ------------
> > recipes-browser/chromium/cef3_280796.bb            | 57
> > ++++++++++++++++------
> > 2 files changed, 43 insertions(+), 46 deletions(-)
> > delete mode 100644
> > recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> >
> > diff --git
> > a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> > b/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> > deleted file mode 100644
> > index 9b14429..0000000
> > ---
> a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> > +++ /dev/null
> > @@ -1,32 +0,0 @@
> > -Fetch the revision from LASTCHANGE file programmatically
> > -
> > -Upstream-Status: Pending
> > -
> > -Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
> > -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > -
> > -diff --git a/cef/tools/make_version_header.py
> > b/cef/tools/make_version_header.py
> > -index 84d49f5..395c2cf 100644
> > ---- a/cef/tools/make_version_header.py
> > -+++ b/cef/tools/make_version_header.py
> > -@@ -64,6 +64,8 @@ def write_svn_header(header, chrome_version,
> > cef_version, cpp_header_dir):
> > -       revision = svn.get_revision()
> > -     elif git.is_checkout('.'):
> > -       revision = git.get_svn_revision()
> > -+    elif os.path.isfile("../build/util/LASTCHANGE"):
> > -+      revision =
> > open("../build/util/LASTCHANGE").read().split("=")[1].strip()
> > -     else:
> > -       raise Exception('Not a valid checkout')
> > -
> > -diff --git a/cef/tools/revision.py b/cef/tools/revision.py
> > -index 1d94602..eb89e3b 100644
> > ---- a/cef/tools/revision.py
> > -+++ b/cef/tools/revision.py
> > -@@ -16,6 +16,8 @@ if os.path.exists(os.path.join('.', '.svn')):
> > -   sys.stdout.write(svn.get_revision())
> > - elif os.path.exists(os.path.join('.', '.git')):
> > -   sys.stdout.write(git.get_svn_revision())
> > -+elif os.path.isfile("../build/util/LASTCHANGE"):
> > -+  revision =
> open("../build/util/LASTCHANGE").read().split("=")[1].strip()
> > - else:
> > -   raise Exception('Not a valid checkout')
> > diff --git a/recipes-browser/chromium/cef3_280796.bb
> > b/recipes-browser/chromium/cef3_280796.bb
> > index 9a9efe5..806a4ac 100644
> > --- a/recipes-browser/chromium/cef3_280796.bb
> > +++ b/recipes-browser/chromium/cef3_280796.bb
> > @@ -7,21 +7,22 @@ RDEPENDS_${PN} += "pango cairo fontconfig pciutils
> > pulseaudio freetype fontconfi
> > SRCREV_tools = "99bcb0e676eb396bcf8e1af3903aa4b578aeeee0"
> > SRCREV_cef = "bbad53dfca9f98dddcb31a590410fece0a4f0234"
> > SRCREV_egl = "a5b81b7617ba6757802b9b5f8c950034d5f961ec"
> > -SRCREV_FORMAT = "cef_egl_tools"
> > +SRCREV_FORMAT = "cef_egl"
> >
> > -SRC_URI = "
> >
> http://people.linaro.org/~zoltan.kuscsik/chromium-browser/chromium_rev_${PV}.tar.xz
> > \
> > -           git://
> >
> github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=src/cef;branch=aura;name=cef
> > \
> > -           git://
> >
> github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=src/ui/ozone/platform/egl;branch=master;name=egl
> > \
> > -           git://
> >
> chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
> > \
> > -           file://01_get_svn_version_from_LASTCHANGE.patch \
> > -          file://cef-simple \
> > -         "
> > -SRC_URI[md5sum] = "9efbb50283b731042e62b9bd5e312b2f"
> > -SRC_URI[sha256sum] =
> > "f608e97dadf6ea4d885b24fd876896d46840fa39bf743ea2025075aee9fb348d"
> > +SRC_URI = "git://
> >
> chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
> > \
> > +           git://
> >
> github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=${S}/src/cef;branch=aura;name=cef
> <http://github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=$%7BS%7D/src/cef;branch=aura;name=cef>
> > \
> > +           git://
> >
> github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=${S}/src/ui/ozone/platform/egl;branch=master;name=egl\
> <http://github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=$%7BS%7D/src/ui/ozone/platform/egl;branch=master;name=egl%5C>
> > +           file://cef-simple\
> > +  "
> >
> > -S = "${WORKDIR}/chromium_rev_${PV}"
> > +export CHROMIUM_SRC_GIT_URL="
> https://chromium.googlesource.com/chromium/src
> > "
> > +export
> CHROMIUM_SRC_GIT_COMMIT="e800fe7470fa87dc1ca5b148a7c2c41f603fdcbd"
> >
> > -do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl SRCREV_tools"
> > +S = "${WORKDIR}/chromium_rev_${PV}/"
> > +
> > +B = "${S}/src/"
> > +
> > +do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl"
> >
> > GYP_ARCH_DEFINES_armv7a = " target_arch=arm arm_float_abi=hard"
> > GYP_ARCH_DEFINES_i586 = " target_arch=ia32"
> > @@ -30,14 +31,41 @@ export GYP_GENERATORS="ninja"
> > export BUILD_TARGET_ARCH="${TARGET_ARCH}"
> > export GYP_DEFINES="${GYP_ARCH_DEFINES}
> > release_extra_cflags='-Wno-error=unused-local-typedefs' sysroot=''"
> >
> > +do_fetch_chromium_source_from_git() {
> > +  cd ${S}
> > +  export PATH=${WORKDIR}/depot_tools:"$PATH"
> > +
> > +  # Note: --deps-file needs to set to .DEPS.git for syncing to old
> > +  # chromium versions
> > +  gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
> > +
> > +  # Runhooks is disabled. Running the hooks will trigger the
> > +  # gyp_chromium command replaced with cef_create_projects.sh by
> > +  # cef3.
> > +
> > +  gclient sync -f --nohooks --revision src@${CHROMIUM_SRC_GIT_COMMIT}
> > +
> > +  # We need the lastchange hook to generate the LASTCHANGE revision info
> > +  # file.
> > +
> > +  python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
> > +}
>
>
> small nit. formatting is not correct for this shell function.
>

Z: Could you explain this in more details?


> > +
> > +addtask fetch_chromium_source_from_git after do_patch before
> do_configure
> > +
>
>
> so we will never patch chromium sources ? if we have to how will we do it ?
>
> Z: CEF is always built on clean chromium checkouts. CEF patches on
Chromium are applied using cef's internal patch script. Here is an exampla:

https://github.com/kuscsik/chromiumembedded/commit/da9609deab1345fc480667ce04ef670f4c94ef62


>
> > do_configure_append() {
> >        export PATH=${WORKDIR}/depot_tools:"$PATH"
> >        # End of LD Workaround
> >        #-----------------------
> >        # Configure cef
> >        #------------------------
> > -       cd cef
> > -       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi
> --depth
> > ../
> > +       cd ${S}/src/cef
> > +
> > +  # Fatal linker warnings need to be disalbed to avoid linker issu
> > +  # with GCC 4.9:
> > +  #
> >
> http://lists.openembedded.org/pipermail/openembedded-core/2014-August/095947.html
> > +
> > +       ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi
> --depth
> > ../ -Ddisable_fatal_linker_warnings=
> >        cd -
> > }
> >
> > @@ -47,5 +75,6 @@ do_qa_configure() {
> > }
> >
> > do_compile() {
> > + cd ${S}/src
>
> formatting is inconsistent
>
> Z: Will fix

> >        ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
> > }
> > --
> > 1.9.1
> > --
> > _______________________________________________
> > 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
>
>


-- 
Zoltan Kuscsik

Senior Software Developer, Linaro Digital Home Group
email: zoltan.kuscsik@linaro.org
IRC: zoli_ on irc.linaro.org


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
  2015-05-28  7:09   ` Zoltan Kuscsik
@ 2015-05-28  7:14     ` Martin Jansa
  2015-06-03 19:26       ` Zoltan Kuscsik
  2015-06-03 19:47     ` Khem Raj
  1 sibling, 1 reply; 8+ messages in thread
From: Martin Jansa @ 2015-05-28  7:14 UTC (permalink / raw)
  To: openembedded-devel

On Thu, May 28, 2015 at 09:09:24AM +0200, Zoltan Kuscsik wrote:
> On 28 May 2015 at 02:56, Khem Raj <raj.khem@gmail.com> wrote:
> > >
> > > +do_fetch_chromium_source_from_git() {
> > > +  cd ${S}
> > > +  export PATH=${WORKDIR}/depot_tools:"$PATH"
> > > +
> > > +  # Note: --deps-file needs to set to .DEPS.git for syncing to old
> > > +  # chromium versions
> > > +  gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
> > > +
> > > +  # Runhooks is disabled. Running the hooks will trigger the
> > > +  # gyp_chromium command replaced with cef_create_projects.sh by
> > > +  # cef3.
> > > +
> > > +  gclient sync -f --nohooks --revision src@${CHROMIUM_SRC_GIT_COMMIT}
> > > +
> > > +  # We need the lastchange hook to generate the LASTCHANGE revision info
> > > +  # file.
> > > +
> > > +  python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
> > > +}
> >
> >
> > small nit. formatting is not correct for this shell function.
> >
> 
> Z: Could you explain this in more details?

Use 4 spaces for indentation (everywhere).

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
  2015-05-28  7:14     ` Martin Jansa
@ 2015-06-03 19:26       ` Zoltan Kuscsik
  2015-06-03 19:50         ` Khem Raj
  0 siblings, 1 reply; 8+ messages in thread
From: Zoltan Kuscsik @ 2015-06-03 19:26 UTC (permalink / raw)
  To: openembedded-devel

Updated recipe to fetch Chromium from git.
LASTCHANGE file is now created using the proper
gclient hook.

Change-Id: Ie1790abec002a0dd39c8d0aae3d03f5ebe6f46aa
Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
---
 .../cef3/01_get_svn_version_from_LASTCHANGE.patch  | 32 ----------
 recipes-browser/chromium/cef3_280796.bb            | 73
+++++++++++++++-------
 2 files changed, 51 insertions(+), 54 deletions(-)
 delete mode 100644
recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch

diff --git
a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
b/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
deleted file mode 100644
index 9b14429..0000000
--- a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-Fetch the revision from LASTCHANGE file programmatically
-
-Upstream-Status: Pending
-
-Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
-
-diff --git a/cef/tools/make_version_header.py
b/cef/tools/make_version_header.py
-index 84d49f5..395c2cf 100644
---- a/cef/tools/make_version_header.py
-+++ b/cef/tools/make_version_header.py
-@@ -64,6 +64,8 @@ def write_svn_header(header, chrome_version,
cef_version, cpp_header_dir):
-       revision = svn.get_revision()
-     elif git.is_checkout('.'):
-       revision = git.get_svn_revision()
-+    elif os.path.isfile("../build/util/LASTCHANGE"):
-+      revision =
open("../build/util/LASTCHANGE").read().split("=")[1].strip()
-     else:
-       raise Exception('Not a valid checkout')
-
-diff --git a/cef/tools/revision.py b/cef/tools/revision.py
-index 1d94602..eb89e3b 100644
---- a/cef/tools/revision.py
-+++ b/cef/tools/revision.py
-@@ -16,6 +16,8 @@ if os.path.exists(os.path.join('.', '.svn')):
-   sys.stdout.write(svn.get_revision())
- elif os.path.exists(os.path.join('.', '.git')):
-   sys.stdout.write(git.get_svn_revision())
-+elif os.path.isfile("../build/util/LASTCHANGE"):
-+  revision = open("../build/util/LASTCHANGE").read().split("=")[1].strip()
- else:
-   raise Exception('Not a valid checkout')
diff --git a/recipes-browser/chromium/cef3_280796.bb
b/recipes-browser/chromium/cef3_280796.bb
index 9a9efe5..a3a4eca 100644
--- a/recipes-browser/chromium/cef3_280796.bb
+++ b/recipes-browser/chromium/cef3_280796.bb
@@ -7,21 +7,22 @@ RDEPENDS_${PN} += "pango cairo fontconfig pciutils
pulseaudio freetype fontconfi
 SRCREV_tools = "99bcb0e676eb396bcf8e1af3903aa4b578aeeee0"
 SRCREV_cef = "bbad53dfca9f98dddcb31a590410fece0a4f0234"
 SRCREV_egl = "a5b81b7617ba6757802b9b5f8c950034d5f961ec"
-SRCREV_FORMAT = "cef_egl_tools"
+SRCREV_FORMAT = "cef_egl"

-SRC_URI = "
http://people.linaro.org/~zoltan.kuscsik/chromium-browser/chromium_rev_${PV}.tar.xz
\
-           git://
github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=src/cef;branch=aura;name=cef
\
-           git://
github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=src/ui/ozone/platform/egl;branch=master;name=egl
\
-           git://
chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
\
-           file://01_get_svn_version_from_LASTCHANGE.patch \
-   file://cef-simple \
-  "
-SRC_URI[md5sum] = "9efbb50283b731042e62b9bd5e312b2f"
-SRC_URI[sha256sum] =
"f608e97dadf6ea4d885b24fd876896d46840fa39bf743ea2025075aee9fb348d"
+SRC_URI = "git://
chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
\
+           git://
github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=${S}/src/cef;branch=aura;name=cef
\
+           git://
github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=${S}/src/ui/ozone/platform/egl;branch=master;name=egl\
+           file://cef-simple\
+  "

-S = "${WORKDIR}/chromium_rev_${PV}"
+export CHROMIUM_SRC_GIT_URL="https://chromium.googlesource.com/chromium/src
"
+export CHROMIUM_SRC_GIT_COMMIT="e800fe7470fa87dc1ca5b148a7c2c41f603fdcbd"

-do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl SRCREV_tools"
+S = "${WORKDIR}/chromium_rev_${PV}/"
+
+B = "${S}/src/"
+
+do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl"

 GYP_ARCH_DEFINES_armv7a = " target_arch=arm arm_float_abi=hard"
 GYP_ARCH_DEFINES_i586 = " target_arch=ia32"
@@ -30,22 +31,50 @@ export GYP_GENERATORS="ninja"
 export BUILD_TARGET_ARCH="${TARGET_ARCH}"
 export GYP_DEFINES="${GYP_ARCH_DEFINES}
release_extra_cflags='-Wno-error=unused-local-typedefs' sysroot=''"

+do_fetch_chromium_source_from_git() {
+    cd ${S}
+    export PATH=${WORKDIR}/depot_tools:"$PATH"
+
+    # Note: --deps-file needs to set to .DEPS.git for syncing to old
+    # chromium versions
+    gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
+
+    # Runhooks is disabled. Running the hooks will trigger the
+    # gyp_chromium command replaced with cef_create_projects.sh by
+    # cef3.
+
+    gclient sync -f --nohooks --revision src@${CHROMIUM_SRC_GIT_COMMIT}
+
+    # We need the lastchange hook to generate the LASTCHANGE revision info
+    # file.
+
+    python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
+}
+
+addtask fetch_chromium_source_from_git after do_patch before do_configure
+
 do_configure_append() {
- export PATH=${WORKDIR}/depot_tools:"$PATH"
- # End of LD Workaround
- #-----------------------
- # Configure cef
- #------------------------
- cd cef
- ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth ../
- cd -
+    export PATH=${WORKDIR}/depot_tools:"$PATH"
+    # End of LD Workaround
+    #-----------------------
+    # Configure cef
+    #------------------------
+    cd ${S}/src/cef
+
+    # Fatal linker warnings need to be disalbed to avoid linker issu
+    # with GCC 4.9:
+    #
http://lists.openembedded.org/pipermail/openembedded-core/2014-August/095947.html
+
+    ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
../ -Ddisable_fatal_linker_warnings=
+    cd -
 }

 # Workaround to disable qa_configure
 do_qa_configure() {
- echo "do_qa_configure"
+    echo "do_qa_configure"
 }

 do_compile() {
- ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
+    cd ${S}/src
+    ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
 }
-- 
1.9.1


On 28 May 2015 at 09:14, Martin Jansa <martin.jansa@gmail.com> wrote:

> On Thu, May 28, 2015 at 09:09:24AM +0200, Zoltan Kuscsik wrote:
> > On 28 May 2015 at 02:56, Khem Raj <raj.khem@gmail.com> wrote:
> > > >
> > > > +do_fetch_chromium_source_from_git() {
> > > > +  cd ${S}
> > > > +  export PATH=${WORKDIR}/depot_tools:"$PATH"
> > > > +
> > > > +  # Note: --deps-file needs to set to .DEPS.git for syncing to old
> > > > +  # chromium versions
> > > > +  gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
> > > > +
> > > > +  # Runhooks is disabled. Running the hooks will trigger the
> > > > +  # gyp_chromium command replaced with cef_create_projects.sh by
> > > > +  # cef3.
> > > > +
> > > > +  gclient sync -f --nohooks --revision src@
> ${CHROMIUM_SRC_GIT_COMMIT}
> > > > +
> > > > +  # We need the lastchange hook to generate the LASTCHANGE revision
> info
> > > > +  # file.
> > > > +
> > > > +  python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
> > > > +}
> > >
> > >
> > > small nit. formatting is not correct for this shell function.
> > >
> >
> > Z: Could you explain this in more details?
>
> Use 4 spaces for indentation (everywhere).
>
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>



-- 
Zoltan Kuscsik

Senior Software Developer, Linaro Digital Home Group
email: zoltan.kuscsik@linaro.org
IRC: zoli_ on irc.linaro.org


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
  2015-05-28  7:09   ` Zoltan Kuscsik
  2015-05-28  7:14     ` Martin Jansa
@ 2015-06-03 19:47     ` Khem Raj
  1 sibling, 0 replies; 8+ messages in thread
From: Khem Raj @ 2015-06-03 19:47 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 356 bytes --]


> On May 28, 2015, at 12:09 AM, Zoltan Kuscsik <zoltan.kuscsik@linaro.org> wrote:
> 
>> Z: CEF is always built on clean chromium checkouts. CEF patches on
> Chromium are applied using cef's internal patch script. Here is an exampla:
> 
> https://github.com/kuscsik/chromiumembedded/commit/da9609deab1345fc480667ce04ef670f4c94ef62
> 
> 
> 

OK

[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git
  2015-06-03 19:26       ` Zoltan Kuscsik
@ 2015-06-03 19:50         ` Khem Raj
  0 siblings, 0 replies; 8+ messages in thread
From: Khem Raj @ 2015-06-03 19:50 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 8950 bytes --]


> On Jun 3, 2015, at 12:26 PM, Zoltan Kuscsik <zoltan.kuscsik@linaro.org> wrote:
> 
> Updated recipe to fetch Chromium from git.
> LASTCHANGE file is now created using the proper
> gclient hook.
> 

This patch seems to be badly formatted in patchwork. Can you send git format-patch + git send-mail
way. Alternatively, if your mailer is mutilating it then you can also send a github pull request.


> Change-Id: Ie1790abec002a0dd39c8d0aae3d03f5ebe6f46aa
> Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
> ---
> .../cef3/01_get_svn_version_from_LASTCHANGE.patch  | 32 ----------
> recipes-browser/chromium/cef3_280796.bb            | 73
> +++++++++++++++-------
> 2 files changed, 51 insertions(+), 54 deletions(-)
> delete mode 100644
> recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> 
> diff --git
> a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> b/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> deleted file mode 100644
> index 9b14429..0000000
> --- a/recipes-browser/chromium/cef3/01_get_svn_version_from_LASTCHANGE.patch
> +++ /dev/null
> @@ -1,32 +0,0 @@
> -Fetch the revision from LASTCHANGE file programmatically
> -
> -Upstream-Status: Pending
> -
> -Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> -
> -diff --git a/cef/tools/make_version_header.py
> b/cef/tools/make_version_header.py
> -index 84d49f5..395c2cf 100644
> ---- a/cef/tools/make_version_header.py
> -+++ b/cef/tools/make_version_header.py
> -@@ -64,6 +64,8 @@ def write_svn_header(header, chrome_version,
> cef_version, cpp_header_dir):
> -       revision = svn.get_revision()
> -     elif git.is_checkout('.'):
> -       revision = git.get_svn_revision()
> -+    elif os.path.isfile("../build/util/LASTCHANGE"):
> -+      revision =
> open("../build/util/LASTCHANGE").read().split("=")[1].strip()
> -     else:
> -       raise Exception('Not a valid checkout')
> -
> -diff --git a/cef/tools/revision.py b/cef/tools/revision.py
> -index 1d94602..eb89e3b 100644
> ---- a/cef/tools/revision.py
> -+++ b/cef/tools/revision.py
> -@@ -16,6 +16,8 @@ if os.path.exists(os.path.join('.', '.svn')):
> -   sys.stdout.write(svn.get_revision())
> - elif os.path.exists(os.path.join('.', '.git')):
> -   sys.stdout.write(git.get_svn_revision())
> -+elif os.path.isfile("../build/util/LASTCHANGE"):
> -+  revision = open("../build/util/LASTCHANGE").read().split("=")[1].strip()
> - else:
> -   raise Exception('Not a valid checkout')
> diff --git a/recipes-browser/chromium/cef3_280796.bb
> b/recipes-browser/chromium/cef3_280796.bb
> index 9a9efe5..a3a4eca 100644
> --- a/recipes-browser/chromium/cef3_280796.bb
> +++ b/recipes-browser/chromium/cef3_280796.bb
> @@ -7,21 +7,22 @@ RDEPENDS_${PN} += "pango cairo fontconfig pciutils
> pulseaudio freetype fontconfi
> SRCREV_tools = "99bcb0e676eb396bcf8e1af3903aa4b578aeeee0"
> SRCREV_cef = "bbad53dfca9f98dddcb31a590410fece0a4f0234"
> SRCREV_egl = "a5b81b7617ba6757802b9b5f8c950034d5f961ec"
> -SRCREV_FORMAT = "cef_egl_tools"
> +SRCREV_FORMAT = "cef_egl"
> 
> -SRC_URI = "
> http://people.linaro.org/~zoltan.kuscsik/chromium-browser/chromium_rev_${PV}.tar.xz
> \
> -           git://
> github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=src/cef;branch=aura;name=cef
> \
> -           git://
> github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=src/ui/ozone/platform/egl;branch=master;name=egl
> \
> -           git://
> chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
> \
> -           file://01_get_svn_version_from_LASTCHANGE.patch \
> -   file://cef-simple \
> -  "
> -SRC_URI[md5sum] = "9efbb50283b731042e62b9bd5e312b2f"
> -SRC_URI[sha256sum] =
> "f608e97dadf6ea4d885b24fd876896d46840fa39bf743ea2025075aee9fb348d"
> +SRC_URI = "git://
> chromium.googlesource.com/chromium/tools/depot_tools.git;protocol=https;destsuffix=depot_tools;branch=master;name=tools
> \
> +           git://
> github.com/kuscsik/chromiumembedded.git;protocol=https;destsuffix=${S}/src/cef;branch=aura;name=cef
> \
> +           git://
> github.com/kuscsik/ozone-egl.git;protocol=https;destsuffix=${S}/src/ui/ozone/platform/egl;branch=master;name=egl\
> +           file://cef-simple\
> +  "
> 
> -S = "${WORKDIR}/chromium_rev_${PV}"
> +export CHROMIUM_SRC_GIT_URL="https://chromium.googlesource.com/chromium/src
> "
> +export CHROMIUM_SRC_GIT_COMMIT="e800fe7470fa87dc1ca5b148a7c2c41f603fdcbd"
> 
> -do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl SRCREV_tools"
> +S = "${WORKDIR}/chromium_rev_${PV}/"
> +
> +B = "${S}/src/"
> +
> +do_fetch[vardeps] += "SRCREV_FORMAT SRCREV_cef SRCREV_egl"
> 
> GYP_ARCH_DEFINES_armv7a = " target_arch=arm arm_float_abi=hard"
> GYP_ARCH_DEFINES_i586 = " target_arch=ia32"
> @@ -30,22 +31,50 @@ export GYP_GENERATORS="ninja"
> export BUILD_TARGET_ARCH="${TARGET_ARCH}"
> export GYP_DEFINES="${GYP_ARCH_DEFINES}
> release_extra_cflags='-Wno-error=unused-local-typedefs' sysroot=''"
> 
> +do_fetch_chromium_source_from_git() {
> +    cd ${S}
> +    export PATH=${WORKDIR}/depot_tools:"$PATH"
> +
> +    # Note: --deps-file needs to set to .DEPS.git for syncing to old
> +    # chromium versions
> +    gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
> +
> +    # Runhooks is disabled. Running the hooks will trigger the
> +    # gyp_chromium command replaced with cef_create_projects.sh by
> +    # cef3.
> +
> +    gclient sync -f --nohooks --revision src@${CHROMIUM_SRC_GIT_COMMIT}
> +
> +    # We need the lastchange hook to generate the LASTCHANGE revision info
> +    # file.
> +
> +    python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
> +}
> +
> +addtask fetch_chromium_source_from_git after do_patch before do_configure
> +
> do_configure_append() {
> - export PATH=${WORKDIR}/depot_tools:"$PATH"
> - # End of LD Workaround
> - #-----------------------
> - # Configure cef
> - #------------------------
> - cd cef
> - ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth ../
> - cd -
> +    export PATH=${WORKDIR}/depot_tools:"$PATH"
> +    # End of LD Workaround
> +    #-----------------------
> +    # Configure cef
> +    #------------------------
> +    cd ${S}/src/cef
> +
> +    # Fatal linker warnings need to be disalbed to avoid linker issu
> +    # with GCC 4.9:
> +    #
> http://lists.openembedded.org/pipermail/openembedded-core/2014-August/095947.html
> +
> +    ./cef_create_projects.sh -I ${BUILD_TARGET_ARCH}_ozone.gypi --depth
> ../ -Ddisable_fatal_linker_warnings=
> +    cd -
> }
> 
> # Workaround to disable qa_configure
> do_qa_configure() {
> - echo "do_qa_configure"
> +    echo "do_qa_configure"
> }
> 
> do_compile() {
> - ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
> +    cd ${S}/src
> +    ninja -C out/${CHROMIUM_BUILD_TYPE} cefsimple
> }
> --
> 1.9.1
> 
> 
> On 28 May 2015 at 09:14, Martin Jansa <martin.jansa@gmail.com> wrote:
> 
>> On Thu, May 28, 2015 at 09:09:24AM +0200, Zoltan Kuscsik wrote:
>>> On 28 May 2015 at 02:56, Khem Raj <raj.khem@gmail.com> wrote:
>>>>> 
>>>>> +do_fetch_chromium_source_from_git() {
>>>>> +  cd ${S}
>>>>> +  export PATH=${WORKDIR}/depot_tools:"$PATH"
>>>>> +
>>>>> +  # Note: --deps-file needs to set to .DEPS.git for syncing to old
>>>>> +  # chromium versions
>>>>> +  gclient config ${CHROMIUM_SRC_GIT_URL} --deps-file=".DEPS.git"
>>>>> +
>>>>> +  # Runhooks is disabled. Running the hooks will trigger the
>>>>> +  # gyp_chromium command replaced with cef_create_projects.sh by
>>>>> +  # cef3.
>>>>> +
>>>>> +  gclient sync -f --nohooks --revision src@
>> ${CHROMIUM_SRC_GIT_COMMIT}
>>>>> +
>>>>> +  # We need the lastchange hook to generate the LASTCHANGE revision
>> info
>>>>> +  # file.
>>>>> +
>>>>> +  python src/build/util/lastchange.py -o src/build/util/LASTCHANGE
>>>>> +}
>>>> 
>>>> 
>>>> small nit. formatting is not correct for this shell function.
>>>> 
>>> 
>>> Z: Could you explain this in more details?
>> 
>> Use 4 spaces for indentation (everywhere).
>> 
>> --
>> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>> 
> 
> 
> 
> --
> Zoltan Kuscsik
> 
> Senior Software Developer, Linaro Digital Home Group
> email: zoltan.kuscsik@linaro.org
> IRC: zoli_ on irc.linaro.org
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-06-03 19:50 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-27 17:37 [meta-browser] [PATCH 2/2] chromium, cef: fetch chromium source from git Zoltan Kuscsik
2015-05-28  0:56 ` Khem Raj
2015-05-28  7:09   ` Zoltan Kuscsik
2015-05-28  7:14     ` Martin Jansa
2015-06-03 19:26       ` Zoltan Kuscsik
2015-06-03 19:50         ` Khem Raj
2015-06-03 19:47     ` Khem Raj
  -- strict thread matches above, loose matches on Subject: below --
2015-05-27 17:35 Zoltan Kuscsik

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.