On Wed, Nov 6, 2019 at 2:19 PM Maxim Uvarov maxim.uvarov@linaro.org wrote:
Yes, I did.
On Wed, 6 Nov 2019 at 15:58, Nicolas Dechesne nicolas.dechesne@linaro.org wrote:
hi Maxim,
your original email didn't make it to the mailing list.. not sure why, you might want to check. I am adding the list here again.
On Wed, Nov 6, 2019 at 10:54 AM Maxim Uvarov maxim.uvarov@linaro.org wrote:
looks like it is. after this patch: http://lists.openembedded.org/pipermail/bitbake-devel/2017-November/009097.h... I see error: ERROR: ExpansionError during parsing /home/maxim.uvarov/build-test-update/build-rpb/conf/../../layers/meta-linaro/meta-aarch64/recipes-core/openjdk/openjdk-8_0.1.bb Traceback (most recent call last): File "Var <SRCPV>", line 1, in <module> File "/home/maxim.uvarov/build-test-update/bitbake/lib/bb/fetch2/__init__.py", line 768, in get_srcrev(d=<bb.data_smart.DataSmart object at 0x7faa98f59e48>, method_name='sortable_revision'): raise FetchError("The SRCREV_FORMAT variable must be set when multiple SCMs are used.\n"\ > "The SCMs are:\n%s" % '\n'.join(scms))
bb.data_smart.ExpansionError: Failure expanding variable SRCPV, expression was ${@bb.fetch2.get_srcrev(d)} which triggered exception FetchError: Fetcher failure: The SRCREV_FORMAT variable must be set when multiple SCMs are used.
I don't understand the situation. do we have a problem or did you fix it?
and btw, are we really still using this recipe from 2014? or did you find that by accident?
yea fresh version of openjdk is in meta-java. But it has some issues to compile on aarch64.
Issue with meta-linaro can be fixed in a few ways:
- add SRCREV_FORMAT = "jdk8" and apply this patch
http://lists.openembedded.org/pipermail/bitbake-devel/2017-November/009097.h...
or
- HG can allow to download tar balls. So we can switch from hg:// to https://.
or
- Simple drop openjdk from meta-linaro and fix meta-java version.
For my issue meta-linaro/aarch64 was in ledge rp build. And parser of .bb files started to fail on that recipe after we switched to zeus branch. Because we don't need openjdk I simply removed it from ledge rp layer. That fixed my current issue. But in general it will be good to fix it.
I think we should fix upstream. I think we used this recipe initially for the bring up, but it should be removed now. Would be nice to have some background on this recipe.. or at least someone who can confirm it's safe to be removed now.
Maxim.
The SCMs are: hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/jdk8;name=jdk8;module=jdk8 hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/corba;name=corba;module=jdk8/corba hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/hotspot;name=hotspot;module=jdk8/hotspot hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/jaxp;name=jaxp;module=jdk8/jaxp hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/jaxws;name=jaxws;module=jdk8/jaxws hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/jdk;name=jdk;module=jdk8/jdk hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/langtools;name=langtools;module=jdk8/langtools hg://hg.openjdk.java.net/aarch64-port;protocol=http;destsuffix=hg/nashorn;name=nashorn;module=jdk8/nashorn
On Wed, 6 Nov 2019 at 12:33, Maxim Uvarov maxim.uvarov@linaro.org wrote:
Hi Nicolas,
is that known bug?
---------- Forwarded message --------- From: Maxim Uvarov maxim.uvarov@linaro.org Date: Tue, 5 Nov 2019 at 23:09 Subject: openjdk is broken on zeus branch To: openembedded@lists.linaro.org
It looks like ${AUTOREV} with hg tool project is broken. openjdk from meta-java also does not use AUTOREV. I'm not sure what is better fix for that - remove AUTOREV or fix it?
BR, Maxim.
WARNING: /home/maxim.uvarov/build-test-update/build-rpb/conf/../../layers/meta-linaro/meta-aarch64/recipes-core/openjdk/openjdk-8_0.1.bb: Exception during build_dependencies for AUTOREV | ETA: --:--:-- WARNING: /home/maxim.uvarov/build-test-update/build-rpb/conf/../../layers/meta-linaro/meta-aarch64/recipes-core/openjdk/openjdk-8_0.1.bb: Error during finalise of /home/maxim.uvarov/build-test-update/build-rpb/conf/../../layers/meta-linaro/meta-aarch64/recipes-core/openjdk/openjdk-8_0.1.bb ERROR: ExpansionError during parsing /home/maxim.uvarov/build-test-update/build-rpb/conf/../../layers/meta-linaro/meta-aarch64/recipes-core/openjdk/openjdk-8_0.1.bb Traceback (most recent call last): File "/home/maxim.uvarov/build-test-update/bitbake/lib/bb/fetch2/__init__.py", line 1302, in FetchData.setup_revisions(d=<bb.data_smart.DataSmart object at 0x7fddc1423b38>): for name in self.names: > self.revisions[name] = srcrev_internal_helper(self, d, name)
File "/home/maxim.uvarov/build-test-update/bitbake/lib/bb/fetch2/__init__.py", line 1167, in srcrev_internal_helper(ud=<bb.fetch2.FetchData object at 0x7fddc17d8e80>, d=<bb.data_smart.DataSmart object at 0x7fddc1423b38>, name='jdk8'): if srcrev == "AUTOINC": > srcrev = ud.method.latest_revision(ud, d, name)
File "/home/maxim.uvarov/build-test-update/bitbake/lib/bb/fetch2/__init__.py", line 1558, in Hg.latest_revision(ud=<bb.fetch2.FetchData object at 0x7fddc17d8e80>, d=<bb.data_smart.DataSmart object at 0x7fddc1423b38>, name='jdk8'): revs = bb.persist_data.persist('BB_URI_HEADREVS', d) > key = self.generate_revision_key(ud, d, name) try: File "/home/maxim.uvarov/build-test-update/bitbake/lib/bb/fetch2/__init__.py", line 1570, in Hg.generate_revision_key(ud=<bb.fetch2.FetchData object at 0x7fddc17d8e80>, d=<bb.data_smart.DataSmart object at 0x7fddc1423b38>, name='jdk8'): def generate_revision_key(self, ud, d, name): > key = self._revision_key(ud, d, name) return "%s-%s" % (key, d.getVar("PN") or "") File "/home/maxim.uvarov/build-test-update/bitbake/lib/bb/fetch2/hg.py", line 225, in Hg._revision_key(ud=<bb.fetch2.FetchData object at 0x7fddc17d8e80>, d=<bb.data_smart.DataSmart object at 0x7fddc1423b38>, name='jdk8'): """ > return "hg:" + ud.moddir
bb.data_smart.ExpansionError: Failure expanding variable SRCPV, expression was ${@bb.fetch2.get_srcrev(d)} which triggered exception AttributeError: 'FetchData' object has no attribute 'moddir'