-
Notifications
You must be signed in to change notification settings - Fork 583
Move build system to Maven (retains Ant tasks for backwards compatibility) #122
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
Conversation
Fixes #37. |
The SSL testsuite does bot work because |
A bit more details about my last commit: what was done in This probably remained unnoticed because default values for Our testsuite didn't handle well failures from Finally, the new |
Hmm, there is one more thing to fix: |
Source files are moved from `src` to `src/main/java`. Tests are moved from `test` to `src/test/java`. `build.properties` is moved and split into two files: src/test/resources/build.properties src/test/resources/config.properties build.xml is replaced by pom.xml. Required system properties (eg. `make.bin`) are now verified in AbstractRMQTestSuite.java. Properties are verified before the tests start. This avoids to have a testsuite hanging because `make` is not GNU Make for instance. Fixes #37.
0e84540
to
fc94c0f
Compare
Move build system to Maven (retains Ant tasks for backwards compatibility)
@PatrickSauts: Merged, thank you very much! |
@dumbbell Have the right
To release a new version the maven commands would be: It would set the version in the pom and use that version in the generated java file. By the way you can also run the perf tests like this or with arguments
Basically any class with a main inside package com.rabbitmq.examples without the need to moving any jar around, the classpath is defined by maven. I added a One last thing I can write an assembly to build a zip to run perf tests that would contain all the necessary files. It would contain a single jar and the necessary .sh/.bat files |
@PatrickSauts note that we also distribute the JAR directly from GitHub and rabbitmq.com. But
certainly sounds right up our alley, thank you! |
@michaelklishin You probably want to build the jars using maven too right ?
I'm pretty sure that was implicit in your comment but just in case. So moving ~completely away from ant you could get rid of Your very welcome. Glad I could help. |
@PatrickSauts sure, thank you. We will ping you once we have more Maven questions (which should be around the time |
As the activation cannot use properties in the file tag. Issue rabbitmq#122
Originally submitted by @PatrickSauts as #119.
This moves build system to Maven while retaining backwards compatibility by offering "proxy" Ant tasks.