-
Notifications
You must be signed in to change notification settings - Fork 433
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Publish ESRi, Oracle modules on Maven #439
Comments
Actually to run (at least some of) the testcases a developer also needs to have an oracle instance running |
This is a serious pain. Now that Oracle drivers are in Maven Central is there any reason this can't be done? |
@yeroc @andyjefferson Creating a PR that updates |
Yes, that is a serious impediment to maintaining and publishing these modules, IMO. I am no longer in a position to run the tests for these modules . I suppose the Oracle module could be released in a "compile-only" way, if that is worth anything. The unit tests would have to be disabled in some way for CI, I think? Other ideas welcome. |
At Vivid we had an environment in which to develop and run the tests. |
FWIW Oracle JDBC driver(s) are in Maven central. Change https://github.com/locationtech/jts/blob/master/pom.xml#L116 Change https://github.com/locationtech/jts/blob/master/modules/io/ora/pom.xml#L40 Change https://github.com/locationtech/jts/blob/master/modules/io/ora/README.md That presumably would allow anyone to build it. You'd have to disable tests without a working Oracle instance, clearly. I tried "mvn clean install -DskipTests" followed by "mvn install -Poracle -DskipTests" but got some Maven whinges on the second command about "Failure to find org.locationtech.jts:jts-io:pom:1.18.0-SNAPSHOT". The parent seems to be set to 1.18.0-SNAPSHOT rather than the current 1.18.2-SNAPSHOT, so perhaps change that. Oh, and https://github.com/locationtech/jts/blob/master/modules/io/ora/pom.xml will also need the dependency
adding. Do all of that and you can finally do "mvn install -Poracle -DskipTests" |
These have non-open source dependencies but, unless they are published, they will not be useable (without inflicting on developers to "go off and build it yourself").
In the case of Oracle the only additional work required (by the Maven releaser) is to install an Oracle JDBC driver jar in their local Maven repo, and then build/release the JTS module (in a suitable place in Maven).
FWIW Vividsolutions managed to publish such modules without apparent problem.
The text was updated successfully, but these errors were encountered: