Sounds good to me. Ultilately this managed package could also disappear I guess since it was mainly for geronimo and tomee which disappeared or dont use it anymore.
Le sam. 31 déc. 2022 à 12:25, Richard Zowalla <r...@apache.org> a écrit : > I guess, that it would need some tests to show, that it is working > correctly? Otherwise it would raise similar concerns as for the PR > using Maven Shading to achieve the transition. > > Gruß > Richard > > P.S. Build is currently failing for that one. > > > Am Samstag, dem 31.12.2022 um 12:15 +0100 schrieb Emmanuel Bourg: > > I've experimented with another approach, a single version > > supporting both javax.transaction and jakarta.transaction: > > > > https://github.com/ebourg/commons-dbcp/tree/dual-javax-jakarta > > > > The only classes using javax.transaction are in the > > org.apache.commons.dbcp2.managed > > package. This package is copied at build time to > > org.apache.commons.dbcp2.managed.jakarta > > with the javax imports replaced. > > > > This solution is easier than maintaining two versions is parallel. > > > > What do you think? > > > > Emmanuel Bourg > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
