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

Make location of ratpack.groovy configurable in Gradle plugin #675

Closed
cschlegelmilch opened this issue May 14, 2015 · 6 comments
Closed
Assignees

Comments

@cschlegelmilch
Copy link

I'd like to be able to change the current assumed directory structure from this:

  • build.gradle
  • src
    • main
    • ratpack
    • test

To this:

  • build.gradle
  • rest-api
    • src
      • main
      • ratpack
      • test

…such that the ratpack tasks (e.g. run) can still work. There appears to be an assumption/default that ratpack.groovy lives in ${project.projectDir}/src/ratpack/ratpack.groovy. It would nice if we could adjust the location of the ratpack directory or ratpack.groovy via the gradle DSL.

@cschlegelmilch cschlegelmilch changed the title Make location of ratpack.groovy configurable Make location of ratpack.groovy configurable in Gradle plugin May 14, 2015
@ldaley ldaley added this to the pre-1.0 milestone May 29, 2015
@ldaley
Copy link
Member

ldaley commented May 29, 2015

Anyone want to take this on? @johnrengelman ?

@johnrengelman johnrengelman self-assigned this Jun 2, 2015
@johnrengelman
Copy link
Member

@cschlegelmilch To satisfy my curiosity, how are you changing the location of the default source set in your build? (i.e. from src/main to rest-api/src/main)

@cschlegelmilch
Copy link
Author

@johnrengelman I wasn't really. I ended up using a parent gradle project at my project root then put a child gradle project beside the src directory. Its a little more complicated than needed for this project, but its seems to be working OK.

@johnrengelman
Copy link
Member

@alkemist I'm gonna bug you about this as soon as I see you again, but is the thought here to implement a true sourceSet instead of just adding a directory to the sourceSet.main?

@ldaley
Copy link
Member

ldaley commented Jun 11, 2015

I was just thinking we'd introduce a ratpack extension and have a property for this.

On 11 Jun 2015, at 15:02, John Engelman notifications@github.com wrote:

@alkemist I'm gonna bug you about this as soon as I see you again, but is the thought here to implement a true sourceSet instead of just adding a directory to the sourceSet.main?


Reply to this email directly or view it on GitHub.

@johnrengelman
Copy link
Member

Yeah, I just came to that same conclusion after looking at the code. We already have an extension for the dependency sugar, so it's just some renaming.

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

No branches or pull requests

3 participants