* [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.