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

jOOQ-codegen-gradle should resolve relative directories with respect to the gradle project, not the running process #15944

Closed
lukaseder opened this issue Dec 18, 2023 · 4 comments

Comments

@lukaseder
Copy link
Member

lukaseder commented Dec 18, 2023

There's an issue with the resolution of relative paths in the new jOOQ-codegen-gradle plugin (#12985), as reported here:

When not providing jOOQ with absolute target paths, then the relative directory is resolved with respect to the running process, e.g.:

  target dir             : C:\Users\lukas\.gradle\daemon\8.5\target\generated-sources\jooq

Instead of the gradle project. We do this already for the maven plugin, so let's do it as well for the gradle plugin. The default should be build/generated-sources/jooq instead of target/generated-sources/jooq

@lukaseder
Copy link
Member Author

3.20 Other improvements automation moved this from To do to Done Dec 18, 2023
lukaseder added a commit that referenced this issue Dec 18, 2023
directories with respect to the gradle project, not the running process
lukaseder added a commit that referenced this issue Dec 18, 2023
directories with respect to the gradle project, not the running process
@mattupstate
Copy link

Would it be possible to use Gradle's filesystem types, such as org.gradle.api.file.Directory, instead of String for file oriented values?

@lukaseder
Copy link
Member Author

lukaseder commented Dec 19, 2023

Can you please be more specific. Where specifically should jOOQ use those types?

Also, it would be helpful if you could state why you need this.

@mattupstate
Copy link

Forgive me. You can ignore this comment.

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

No branches or pull requests

2 participants