=== lifeless_ is now known as lifeless [18:52] hi ubuntu-motu, I have filed an SRU (a sync of an unstable bug fix to bionic): https://bugs.launchpad.net/ubuntu/+source/libjsyntaxpane-java/+bug/1770809 [18:52] Launchpad bug 1770809 in libjsyntaxpane-java (Ubuntu Bionic) "[bionic SRU] Port libjsyntaxpane-java 0.9.6~r156-7 java9 crash to bionic" [Undecided,New] [18:53] Would someone be so kind to consider triggering the SRU process? [19:29] fnatter: looks good in principle but I don't have time to review/sponsor right now. Please ping me tomorrow if you don't find anyone before then. [19:29] And thank you for preparing this for Ubuntu :) [19:37] I'll do what I can right now, but there's a chance I'll defer the actual patch review to someone else. [19:39] fnatter: I personally consider the lack of a DEP-3 header in a patch a blocker to sponsorship, please fix that. [19:39] Also, "Origin" should have e.g. "commit:5fc7559" rather than the full link I think. [19:40] Lastly, you don't need to add your name to uploaders, and you just need to target the upload at "bionic" [19:40] Otherwise, lgtm :) [19:42] A full web-based URL is pretty normal for Origin. [19:45] rbasak: Oh? [19:45] I guess I've always preferred the full URL, but YMMV [20:43] tsimonq2: that sounds like the inverse sense of what you just said above [21:09] rbasak: I will fix this in the next days, I might not have time on the weekend, thanks! [21:10] tsimonq2: "Origin" contains a full URL because the repo is a fork of the original jsyntaxpane repo, that the package was originally created for [21:10] and there are at least two forks [21:14] I will fix the DEP3 header in debian/patches/disable-pom [21:14] see you === lifeless_ is now known as lifeless