dima767
commented
12 years ago Thank you for the pull request!
Will merge a bit later - right now I'm at the Jasig 2012 conference :-)
Sent from my iPhone
On Jun 11, 2012, at 14:32, Eric Piercereply@reply.github.com wrote:
I added the groovy-eclipse-compiler plugin so maven will build the groovy components as well as the java. I also fixed a typo in JsonServiceRegistryDao (the ObjectMapper was being called with the wrong variable name) Everything now compiles cleanly and seems to be working fine.
Thanks for sharing this module - this looks like it will be perfect for our needs.
-Eric
You can merge this Pull Request by running:
git pull https://github.com/epierce/cas-addons master
Or you can view, comment on it, or merge it online at:
https://github.com/Unicon/cas-addons/pull/1
-- Commit Summary --
- Add maven plugin for groovy compilation and variable name fix
- Variable fix
-- File Changes --
M pom.xml (26) M src/main/java/net/unicon/cas/addons/serviceregistry/JsonServiceRegistryDao.groovy (2)
-- Patch Links --
https://github.com/Unicon/cas-addons/pull/1.patch https://github.com/Unicon/cas-addons/pull/1.diff
Reply to this email directly or view it on GitHub: https://github.com/Unicon/cas-addons/pull/1
I added the groovy-eclipse-compiler plugin so maven will build the groovy components as well as the java. I also fixed a typo in JsonServiceRegistryDao (the ObjectMapper was being called with the wrong variable name) Everything now compiles cleanly and seems to be working fine.
Thanks for sharing this module - this looks like it will be perfect for our needs.
-Eric