+1 on removing the conditional build, and including it in the docker image.
My only concern is the jdbc driver license. It'd be fine if we used postgres one. BSD 2-clause is permissive[1][2]. [1] https://mvnrepository.com/artifact/org.postgresql/postgresql/42.7.4 [2] https://jdbc.postgresql.org/license/. Yufei On Fri, Apr 18, 2025 at 11:17 PM Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > Yes, that's the plan (cleanup the conditional build steps). > > If no objections, I will go ahead on a PR. > > Let's see what the others are thinking. > > Regards > JB > > On Fri, Apr 18, 2025 at 11:08 PM Dmitri Bourlatchkov <di...@apache.org> > wrote: > > > > I agree that including the EclipseLink-based Persistence into binary > > distributions is a good idea. > > > > I'd even take this one step further and remove all conditional build > steps > > for EclipseLink and include it in regular builds too. > > > > Cheers, > > Dmitri. > > > > On Fri, Apr 18, 2025 at 2:12 AM Jean-Baptiste Onofré <j...@nanthrax.net> > > wrote: > > > > > Hi folks > > > > > > As discussed yesterday during the community meeting, I propose we > include > > > eclipselink module by default in Polaris server distribution and docker > > > image. > > > > > > I think it would help our users to easily start with Polaris. > > > I would like to include this for 0.10.0-beta-incubating release. > > > > > > Thoughts ? > > > > > > Regards > > > JB > > > >
