[18:52] <fnatter> 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:53] <fnatter> Would someone be so kind to consider triggering the SRU process?
[19:29] <rbasak> 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] <rbasak> And thank you for preparing this for Ubuntu :)
[19:37] <tsimonq2> 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] <tsimonq2> fnatter: I personally consider the lack of a DEP-3 header in a patch a blocker to sponsorship, please fix that.
[19:39] <tsimonq2> Also, "Origin" should have e.g. "commit:5fc7559" rather than the full link I think.
[19:40] <tsimonq2> Lastly, you don't need to add your name to uploaders, and you just need to target the upload at "bionic"
[19:40] <tsimonq2> Otherwise, lgtm :)
[19:42] <rbasak> A full web-based URL is pretty normal for Origin.
[19:45] <tsimonq2> rbasak: Oh?
[19:45] <tsimonq2> I guess I've always preferred the full URL, but YMMV
[20:43] <rbasak> tsimonq2: that sounds like the inverse sense of what you just said above
[21:09] <fnatter> rbasak: I will fix this in the next days, I might not have time on the weekend, thanks!
[21:10] <fnatter> 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] <fnatter> and there are at least two forks
[21:14] <fnatter> I will fix the DEP3 header in debian/patches/disable-pom
[21:14] <fnatter> see you