dlehammer opened a new issue, #14703:
URL: https://github.com/apache/grails-core/issues/14703

   Symptom discovered during migration of web-application from Grails v2.4.4 to 
v3.3.5, symptom still present in Grails v3.3.8.
   
   ### Task List
   
   - [x] Steps to reproduce provided
   - [x] Full description of the issue provided (see below)
   
   ### Steps to Reproduce
   
   1. create a Grails app (symptom is also present for plugins etc.)
   ```
   $ grails create-app my-app
   ```
   
   2. commit `create-app` to version control, ex. using Git.
   ```
   my-app$ git init
   my-app$ git add .
   my-app$ git commit --message="grails create-app my-app"
   ```
   
   3. execute a Gradle/Grails command, ex. `-version` where side-effects are 
unexpected (symptom is also present for `create-service` etc.)
   ```
   my-app$ ./grailsw -version
   | Grails Version: 3.3.8
   | Groovy Version: 2.4.7
   | JVM Version: 1.8.0_172
   ```
   
   4. now `gradle.properties` has been modified.
   ```
   my-app$ git status 
   On branch master
   Changes not staged for commit:
     (use "git add <file>..." to update what will be committed)
     (use "git checkout -- <file>..." to discard changes in working directory)
   
        modified:   gradle.properties
   ```
   
   ```
   my-app$ git diff gradle.properties
   diff --git a/gradle.properties b/gradle.properties
   index df75075..c19fdd5 100644
   --- a/gradle.properties
   +++ b/gradle.properties
   @@ -1,3 +1,5 @@
   +#Sat Dec 01 14:08:20 CET 2018
    grailsVersion=3.3.8
   +grailsWrapperVersion=1.0.0
    gormVersion=6.1.10.RELEASE
    gradleWrapperVersion=3.5
   ```
   
   ### Expected Behaviour
   
   `gradle.properties` isn't expected to be modified by Gradle/Grails after 
initial `create-app`.
   
   The Grails wrapper is expected to be self-contained, ie. it's unexpected 
that redundant configuration leaks into `gradle.properties`.
   In my experience the `grailsWrapperVersion` isn't needed in-order for 
Gradle/Grails wrapper to function as expected.
   
   ### Actual Behaviour
   
   `gradle.properties` is modified by Gradle/Grails, se also Steps to Reproduce
   
   ### Work-around(s)
   
   A. explicitly add the `grailsWrapperVersion=1.0.0` to `gradle.properties` 
and commit.
   
   B. mark `gradle.properties` as readonly.
   
   ### Environment Information
   
   - **Operating System**: Ubuntu 16.04 (64-bit)
   - **Grails Version:** 3.3.8
   - **JDK Version:** OpenJDK Runtime Environment (Zulu 8.30.0.1-linux64) 
(build 1.8.0_172-b01)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to