-
Notifications
You must be signed in to change notification settings - Fork 3
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
Problem when creating default
configuration
#12
Comments
I could verify locally that this solves the problem, but it continues with:
I can successfully integrate it my project if I rename the tasks, i.e.
What do you think? Should I open a PR for that change? [edit] Seems to be somewhat the default, e.g. for the python gradle plugin:
|
Hey, thanks for the feedback!
The This behavior isn't ideal but was the one I deemed sufficient enough with my at-the-time limited knowledge of proper Gradle ways, to work in all cases, albeit with a bit of refactoring. As per your proposals:
I'll try and look into how I can improve the developer experience with gradle-rust based on that second point (i.e. not splitting the project in two plugins), but do note that it'll probably take a bit of time as I'm taking a break from programming to handle changes and life in general 👍 You can also try and separate your module layout to work in the current gradle-rust fashion if that's possible for your workflow; there isn't much in terms of proper documentation, only this example project which should serve as a bare-bones template. |
Hey, I am trying to configure the plugin in an existing Gradle project. Your example project runs just fine. However, when I add the plugin to the existing project, I get:
Which is caused by:
gradle-rust/src/main/kotlin/fr/stardustenterprises/gradle/rust/wrapper/WrapperPlugin.kt
Line 24 in 91c333f
If a default configuration already exists (which is the case here), it throws.
I am not sure if creating a configuration is strictly necessary, but if so, we could change the call to
.maybeCreate
which has create-if-not-exists semantics. wdyt?The text was updated successfully, but these errors were encountered: