2015-10-26 13:15 GMT+01:00 Julian Reschke <[email protected]>: > On 2015-10-26 12:13, Chetan Mehrotra wrote: >> >> Looking at history of oak-core/pom.xml this change was done in [1] >> for OAK-1708 most like to support loading of various DB drivers from >> within Oak Core and probably a temp change which was not looked back >> again. That might not be required now as the DataSource gets injected >> and oak-core need not be aware of drivers etc. So we can get rid of >> that >> >> @Julian - Thoughts? >> ... > > > No thoughts other than "I do not understand Maven dependencies sufficiently > well to comment". > > How about changing it and see whether anything breaks? >
It would be a good starting point if you would remember why the change was made in the first place, so we could have something more specific to look after. > Best regards, Julian
