Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Correctly specify JAR file containing executing net.sourceforge.schemaspy.Config class #10

Merged

Conversation

sturton
Copy link
Contributor

@sturton sturton commented May 20, 2012

Currently, using the plugin fails with the schemas tag (with single or multiple specified schemas) for 2 reasons:-
the generated command-lines reference the wrong JAR file, simply being the first JAR file in the classpath
the generated command-lines do not specify the class to invoke and the generated JAR file does not specify net.sourceforge.schemaspy.Main as the main JAR class and so is not invoked.

This change corrects the first problem: it is upstream compatible up to schemaspy.sourceforge.net

… is called from external frameworks for multiple schemas
@sturton
Copy link
Contributor Author

sturton commented May 20, 2012

This pull request relates to Issue #1.

@wakaleo wakaleo merged commit a933cb7 into wakaleo:master May 20, 2012
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants