-
Notifications
You must be signed in to change notification settings - Fork 25
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 Morf Data Copier #8
Comments
@tsg21 is making progress on this. Assigned |
I've spent a while trying to progress this in my fork. The main stumbling block is that the UI depends on Eclipse's SWT and JFace, and these are not cleanly available on Maven-central. They were added earlier this year it seems but I'm having trouble getting this to work. SWT is funky in the way it provides builds for multiple platforms (with native bindings), and it seems the way they've published the jars to Maven confuses m2e. I keep getting this in the Maven logs:
And also this in the problems view (amongst other things):
|
Useful discussions on exactly this problem: Doesn't seem to supply a way of getting JFace to work though. |
IMHO I think all we really need is a the command line version. Not sure the SWT UI adds much to the party. |
Since it’s a really simple wizard UI, shall we replace it with JavaFX so that we don’t need the extra dependencies? |
@badgerwithagun : given how much use the UI gets within Alfa, I think it does add value. A command-line version is necessary too, but people like UIs. @venushka : Possibly. I'm not against. The existing UI is pretty good though. Re-writing in JavaFX seems wasteful when there's a perfectly functional UI already there. |
Functional but desperately in need a rewrite, in all honesty - particularly to decouple from the JDBC URL generation code and allow support for arbitrary additional dialects. |
Absolutely, but that's a pretty small refactor (in my eyes) compared to completely rebuilding the UI in new tech. (I don't have the knowledge of Java FX to rebuild the UI. Maybe someone else does?) |
Yeah, it’s pretty straightforward as long as you have simple requirements. Want a taskbar icon? Back to AWT with you. Hook into system level events? Ha! One window wizard? Simples. |
I've taken a look at rewriting from SWT to JavaFX, and it doesn't look too bad. From experience with progress bars in SWT, that might be a little trickier, but no insurmountable. Will try to get to this over Christmas. |
Just to re-iterate, why do we think the UI is that important? @tsg21 ? Could we not get the command line version out first and build a UI later? |
It's not essential, by any means (if a good command line alternative is provided), however:
So I think it would be valuable to have as an option. @gilleain Please feel free to have a go! |
We have a simple command-line/UI tool built over Morf which allows end users to copy arbitrary database schemas and data between servers, platforms and cross-platform XML "dumps".
This is an extremely useful and unique tool and also a nice, clean example of how to use Morf in the real world. We want that published, but it also needs some extrication from our code stack.
The text was updated successfully, but these errors were encountered: