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

better support for others libraries management tools #28

Closed
rlagoue opened this issue Mar 3, 2018 · 5 comments
Closed

better support for others libraries management tools #28

rlagoue opened this issue Mar 3, 2018 · 5 comments
Labels

Comments

@rlagoue
Copy link

rlagoue commented Mar 3, 2018

the packaging value for the module vue-gwt-processors is "jar" (https://search.maven.org/remotecontent?filepath=com/axellience/vue-gwt-processors/1.0-beta-6/vue-gwt-processors-1.0-beta-6.pom)

Unfortunately the value is "gwt-lib" for the modules vue-gwt and vue-gwt-core (https://search.maven.org/remotecontent?filepath=com/axellience/vue-gwt/1.0-beta-6/vue-gwt-1.0-beta-6.pom, https://search.maven.org/remotecontent?filepath=com/axellience/vue-gwt-core/1.0-beta-6/vue-gwt-core-1.0-beta-6.pom). It causes resolve tasks to fail when working with apache ivy.

It would be nice to rename it to "jar" too

@adrienbaron adrienbaron added this to the Beta 7 milestone Mar 4, 2018
@rlagoue
Copy link
Author

rlagoue commented Mar 7, 2018

hey @adrienbaron

can I fix it by myself, since I am waiting for this to start with vue-gwt?

Best regards

@adrienbaron
Copy link
Collaborator

adrienbaron commented Mar 7, 2018

@rlagoue Sure you can try to clone our repo locally on the master fix it and do mvn clean install.
The actual fix should be done on the develop for Beta 7, but the documentation for Beta 7 is not up yet so you should rather use Beta 6 for your project for now.
We are using tbroyer GWT maven plugin (https://github.com/tbroyer/gwt-maven-plugin) so that's where the gwt-lib comes from. Just replacing by jar might break some things. You can ask on the Gitter also, maybe Colin or James will have an idea.

@rlagoue
Copy link
Author

rlagoue commented Mar 7, 2018

Ok I was supposing that the solution would be easy as just changing it. So I will continue to fight with Beta 6 usage. :-)

@adrienbaron adrienbaron removed this from the Beta 7 milestone Mar 16, 2018
@adrienbaron
Copy link
Collaborator

Removing the gwt-lib would break the build with the tbroyer GWT maven plugin to my best knowledge.
So for now we have to keep it even it breaks compatibility with Apache Ivy.
So we won't fix for now, solution might be to look how to make Apache Ivy accept gwt-lib as it's fairly popular amongst modern GWT librairies.

@rlagoue
Copy link
Author

rlagoue commented Mar 17, 2018

ok, I have found a away to declare it properly in the dependency, thank you

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

No branches or pull requests

2 participants