Opened 5 months ago

Closed 4 months ago

#2364 closed enhancement (fixed)

Support Tomcat9

Reported by: mhatta Owned by: zzz
Priority: major Milestone: 0.9.39
Component: apps/jetty Version: 0.9.37
Keywords: Cc:
Parent Tickets:

Description

Debian will only have libtomcat9-java in Buster.

Looks like s/tomcat8/tomcat9/ will work in most cases, but need some check.

Subtickets

Change History (5)

comment:1 Changed 5 months ago by zzz

  • Component changed from unspecified to apps/jetty

Please test on the Debian side and report results.
I probably won't get to this unless/until Ubuntu removes 8, it's still in Disco for now:
https://packages.ubuntu.com/search?keywords=libtomcat&searchon=names&suite=all&section=all

Good info on Tomcat versions:
http://tomcat.apache.org/whichversion.html

Best test is to check all the webapps, especially susidns, susimail, and the bote plugin if you have it installed.

comment:3 Changed 5 months ago by mhatta

I uploaded 0.9.37-3 to Debian sid. It contains the following patch:

https://salsa.debian.org/debian/i2p/blob/debian/0.9.37-3/debian/patches/0003-tomcat9.patch

susinds and susimail seem to work. Let's see what happens.

comment:4 Changed 4 months ago by zzz

  • Milestone changed from soon to 0.9.39
  • Owner set to zzz
  • Status changed from new to accepted

I didn't include your patch in my 0.9.38 debian/ubuntu builds, in part because I forgot, and in part because tomcat8 still worked, it's still in sid, buster, and disco. If they remove tomcat8 (and the link in comment 2 says it won't be in buster), then my builds will break. I'll guess I'll deal with it when it happens.

What I'll probably do for .39 is add support in all the config and build.xml files for both 8 and 9, so we won't need any patch.

comment:5 Changed 4 months ago by zzz

  • Resolution set to fixed
  • Status changed from accepted to closed

In e70ab57c097b2669086ca15f9f939f709b0a59f8 to be 0.9.38-1
Adds with-libtomcat9-java build option as proposed in comment 4 above.
Build tested only by me, but mhatta has tested the running of it as noted in comment 3 above.
If there's any other issues, we'll find out soon enough.
Not clear atm if 8 will be removed from Disco, but 0.9.39 will be out before Disco is, so I don't foresee needing to re-release 0.9.38 in my PPA or our project deb disco repo.

Note: See TracTickets for help on using tickets.