Skip to content
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

Specifying multiple commands on the command line #353

Closed
flyway opened this issue Jun 25, 2013 · 3 comments
Closed

Specifying multiple commands on the command line #353

flyway opened this issue Jun 25, 2013 · 3 comments

Comments

@ghost
Copy link
Collaborator

@ghost ghost commented Jun 25, 2013

Original author: jonathan...@gmail.com (September 18, 2012 13:46:47)

It would be nice to be able to do the following:

./flyway.sh clean init migrate

as opposed to:

./flyway.sh clean
./flyway.sh init
./flyway.sh migrate

Currently this:

./flyway.sh clean init migrate

will just run clean.

Original issue: http://code.google.com/p/flyway/issues/detail?id=331

@ghost
Copy link
Collaborator Author

@ghost ghost commented Jun 25, 2013

From axel.fontaine.business@gmail.com on September 18, 2012 15:03:01
Hi Jonathan,

thanks for your suggestion. I agree with you. This would make life easier and this is in fact already quite similar to the way people use the maven plugin today.

Axel

@ghost
Copy link
Collaborator Author

@ghost ghost commented Jun 25, 2013

From jonathan...@gmail.com on September 18, 2012 15:25:46
I've patched the flyway.sh script at the moment to do this as follows:

CLASSPATH=bin/flyway-commandline-1.7.jar:bin/flyway-core-1.7.jar:bin/spring-jdbc-2.5.6.jar:bin/commons-logging-1.1.1.jar:bin/spring-beans-2.5.6.jar:bin/spring-core-2.5.6.jar:bin/spring-context-2.5.6.jar:bin/aopalliance-1.0.jar:bin/spring-tx-2.5.6.jar:bin/log4j-1.2.16.jar:sql

for cmd in "$@"
do
$JAVA_CMD -cp $CLASSPATH com.googlecode.flyway.commandline.Main $cmd
done

The other way is modifying the Main class to pick up on these, but this does the trick for me currently. Thanks for getting back though.

@ghost
Copy link
Collaborator Author

@ghost ghost commented Jun 25, 2013

From herve.ma...@gmail.com on March 08, 2013 11:01:32
I've posted a patch for this issue, see issue #446.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
0 participants
You can’t perform that action at this time.