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

Migrate support for OS MINGW64 #94

Closed
jpenalver10 opened this issue Oct 17, 2017 · 3 comments · May be fixed by mojohaus/appassembler#94
Closed

Migrate support for OS MINGW64 #94

jpenalver10 opened this issue Oct 17, 2017 · 3 comments · May be fixed by mojohaus/appassembler#94

Comments

@jpenalver10
Copy link

Hello guys!

I'm currently using MinGW (http://www.mingw.org/) as my default shell environment over windows 10. It's currently distributed with GIT client for windows and very useful for windows users.

When I try to run migrate in this bash, I receive this error:

jpenalver@JOAQUINPENALVER MINGW64 /c/git/wtn-geo-searcher/migrations (develop) $ migrate info
Error: could not find or load main class org.apache.ibatis.migration.Migrator

I historically solve this problem editing the migrate script *nix, adding the next line in the resolution of OS specific support to support MINGW64:

case " `uname`" in
  CYGWIN*) cygwin=true ;;
  MINGW64*) cygwin=true ;;
  Darwin*) darwin=true

The system identification in mingw is:

$ uname -a
MINGW64_NT-10.0 JOAQUINPENALVER 2.9.0(0.318/5/3) 2017-10-05 15:05 x86_64 Msys

I dont know if this is the best solution and if is reasonable to add this support.

Thanks!

@harawata
Copy link
Member

This may be a duplicate of #58 .
The script is generated by Appassembler maven plugin.
I already reported the issue on their tracker, but as you have a workaround, I would suggest you to send a PR and see if it's accepted.

@jpenalver10
Copy link
Author

ooops sorry! Ok, maybe I send PR to Appassembler.

thanks!

@jpenalver10
Copy link
Author

I close this issue for duplication.

Thanks.

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

Successfully merging a pull request may close this issue.

2 participants