/srv/irclogs.ubuntu.com/2019/03/12/#ubuntu-java.txt

sbeattieokay, other than the android-platform-libnativehelper/android-framework-23 issue I mentioned, everything else in https://launchpad.net/~openjdk-11-transition/+archive/ubuntu/android-tools/ is +1 to go to bionic-proposed.02:47
sbeattie(and I'd be +1 on android-platform-libnativehelper and android-framework-23 if someone can explain to me why the dropped JniConstants are not an issue)02:51
sbeattielooks like https://android.googlesource.com/platform/libcore/+/72a46b24e87902908d544c4ba0bdd218368a280e and https://android.googlesource.com/platform/libcore/+/59cd43352e5fd4907303d06f9b92f01c943375f7 got rid of the references upstream.03:27
dokosbeattie: forwarded to #debian-java10:19
dokosbeattie: please review android-platform-build in android-tools. should be the last one there10:20
dokotdaitx: sweethome3d, running on my laptop, bionic, no chroot. so no bind-mount issue10:23
dokosbeattie: https://launchpad.net/~openjdk-11-transition/+archive/ubuntu/test/+build/16486471 has a a-f-23 test build10:25
dokohttps://launchpad.net/~openjdk-11-transition/+archive/ubuntu/test/+sourcepub/9968168/+listing-archive-extra10:26
doko<seamlik> @doko The build script for "android.jar" is not easy to maintain and updating it is so annoying. I have been experimenting a way to build it using upstream's build system.12:01
doko<seamlik> "android.jar" only contains declarations, the contents of methods are all erased. So it doesn't care about those missing APIs.12:01
dokosbeattie: ^^^12:01
dokosbeattie, tdaitx, huehner:feedback appreciated for the announcement mail to u-d-a: https://paste.ubuntu.com/p/nHvTqf5c8n/13:02
dokotdaitx: I copied the ppa packages to the cosmic pocket. first copied every package, then removed every package which showed a superseded version. So that should be safe. will you update the ti arch-any packages? but please don't re-upload those which I already removed13:27
tdaitxdoko: sure13:56
=== huehner_ is now known as huehner
dokosbeattie: any change to review the one missing package, and reply to the comments above?16:31
dokoany chance even ...16:31
sbeattiedoko: yes, looking now.16:32
sbeattie+1 on  android-platform-build16:32
dokosbeattie: and the issue about the "missing" references?16:39
sbeattiedoko: to be clear, based on seamlik's comments, we'd keep things as-is and not backport those two commits like in the test build?16:42
sbeattie(just want to make sure I understand scrollback)16:42
dokoyes16:43
dokoyes, keept it as is16:43
dokosbeattie: ?16:49
sbeattieI'm still looking16:49
sbeattie(trying a local unmodified test rebuild)16:49
dokosorry to pester you, but I'm trying to get this ready, so we can copy these packages, and then send the announcement email16:50
sbeattieyeah, got you.16:52
sbeattieokay, I'm +1 on android-platform-libnativehelper and android-framework-23, so +1 on everything in the android-tools ppa.16:52
sbeattiedoko: ^16:52
dokosbeattie: ta. and I checked, that the android.jar is the same in both packages16:54
dokosil2100: ^^^ please copy ppa:android-tools16:54
sil2100doko: whole PPA?16:55
dokosil2100: yes16:55
sil2100ACK16:55
dokotdaitx_: looking at your list of of openjdk 10 ftbfs: libgpars-groovy-java apache-log4j2. Should we update these to disco versions as well?17:08
tdaitx_libgpars-groovy-java is in stage317:09
dokook17:10
tdaitx_let me look at apache-log4j217:10
=== tdaitx_ is now known as tdaitx
dokosil2100: please also update the hints for the packages uploaded earlier today, and add those for the new ppa17:15
sil2100Ok, will do, for now I'm slowly approving the newly copied packages17:17
tdaitxso apache-log4j2 has a single fix for its own FTBFS + it adds a java 9 module17:18
tdaitxof its reverse (b-)deps we only updates jabref, which works fine with the current apache-log4j2 version in bionic17:18
tdaitxthus the only reason to update it is to fix the FTBFS as far as I can tell17:18
tdaitxdoko: ^ do you see any other problem besides apache-log4j2 FTBFS in bionic? are we updating non-runtime issues at this step?17:20
tdaitxif so, yes, we should update it17:20
huehnertdaitx: sorry to jump in here... upstream precompiled jar files of log4j2 use a feature called multi-release jar-files (several verions of .class file targeting different jvm versions auto-selected at runtime) and current packaged ant has some bug (missing support) causing failures at runtime17:20
huehnertdaitx: not sure if can issue also with your package17:21
huehnerhttps://bugs.launchpad.net/ubuntu/+source/ant/+bug/180838317:21
huehnerfixed upstream by ant but no release done yet :(17:21
tdaitxoh yeah, did the ant we backported included this fix?17:21
tdaitxI forgot to look at that17:21
tdaitxit would be a good time to include this17:21
huehnertdaitx: afaik not... we tied to push upstreadm (ant) to make a 1.10.6 release but they maybe forgot (weeks after not yet)17:21
huehnerso afaik unfixed everywhere in ubuntu17:22
tdaitxhuehner: any change this patch can introduce issues or require other packages to be updated or rebuild?17:23
dokotdaitx: yes, updating anything else would be ok, but not a priority for today's email17:23
dokotdaitx: I just checked for these openjdk-10 ftbfs in our progress document17:23
dokohuehner: please attach a patch and subscribe ubuntu-sponsors17:25
huehnertdaitx: from reading upstream commit should be quite safe, as essentially switches to newer openjdk api understand new java feature, however they did bit of code churn +adding test-cases to diff not super small17:28
huehneri'll check if upstream commit applies on top of 1.10.5 (we have packaged) and attach it17:28
huehnerwill be in evening... outside dayjob17:28
tdaitxhuehner: thanks, is this something that affects other packages builds or basically helps users that have to deal with mr-jars?17:31
huehnertdaitx: only mrjar when loaded via ant-classloader (your custom ant task running inside ant jvm)17:37
huehnertdaitx: so very like that for 95% of people they never touch it17:37
tdaitxack17:37
huehnertdaitx: there is workaround flag for ANT_OPTS... but i would prefer fixing it inside ant if we can17:37
huehnertdaitx: just forgot to prod upstream more to do a 1.10.6 release including it (they wanted in january already but maybe forgot)17:38
sil2100Damn those LP timeouts17:51
dokothe issue is too big17:54
dokobut we'll have more fun adding the cosmic tasks ...17:54
sil2100Ok, all copied and approved, will update the hints17:58
sil2100Should be updated18:05
huehnertdaitx, doko: i am preparing ant patches we talked about: so far, 2 new quilt patches (1 per upstream commit) just with quilt import, 1patch mofified to remove hunk for patching WHATNEWS (does not apply as 1.10.6 section does not exist in 1.10.5) small changelog entry examples, will be tested by us on top of bionic-proposed 1.10.5 (idea to add explanation of commit + risk comments in lp bug)21:16
huehnerdoes that fix what you need or you prefer something different?21:16

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!