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

Ported build over to Gradle #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Mar 17, 2014

Hello! Some of us here at Puppet Labs are using ci-eye for our build radiators and I'd like to start extending its functionality a bit, but before I do that I'd really like to use a modern build system. This PR converts the build to gradle and the following commands will be the ones a person would need for development:

gradle test - Run the unit tests
gradle fatJar - Build the fat jar and place it in the ./build/lib directory.
gradle idea - Generate Idea project files.
gradle eclipse - Generate Eclipse project files.

Note that with a modern version of Idea, instead of running gradle idea, you can actually just open the gradle.build file directly and the project will be opened.

@scarytom
Copy link
Member

Sadly, when Gradle generates the eclipse .project file, it doesn't generate the filter rules, and when it generates the eclipse .classpath file, it doesn't generate the access rules. Do you have any way to correct this?

@ghost
Copy link
Author

ghost commented Mar 18, 2014

Hmm, well I will look around for some way around this. If I can't find any way to do this I will remove that commit from this PR.

@ghost
Copy link
Author

ghost commented Mar 19, 2014

@scarytom, What I ended up doing was leaving the Eclipse .project and .settings/ directory. The Gradle Eclipse plugin will now append a couple things to them, if necessary, but leave everything else intact. The only thing that's generated from scratch is the .classpath file (the most important part of the Gradleizing) which now has the proper filter applied.

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 this pull request may close these issues.

1 participant