You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@grails.apache.org by "jamesfredley (via GitHub)" <gi...@apache.org> on 2025/09/18 19:33:51 UTC
[PR] Add 'migrations' to grailsAppResourceDirs [grails-core]
jamesfredley opened a new pull request, #15084:
URL: https://github.com/apache/grails-core/pull/15084
The 'migrations' directory is now included in grailsAppResourceDirs, allowing it to be treated as a resource directory alongside 'views', 'i18n', and 'conf'.
--
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: notifications-unsubscribe@grails.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add 'migrations' to grailsAppResourceDirs [grails-core]
Posted by "jamesfredley (via GitHub)" <gi...@apache.org>.
jamesfredley commented on PR #15084:
URL: https://github.com/apache/grails-core/pull/15084#issuecomment-3312755488
```
sourceSets {
main {
resources {
srcDir 'grails-app/migrations'
}
}
}
```
which historically was added to **build.gradle**, but was not auto generated by forge, to accomplish adding /grails-app/migrations/* to /build/resources/main or the generated war/jar, also creates the duplicates issue, so I am moving forward with fixing that also as part of this PR too
--
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: notifications-unsubscribe@grails.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add 'migrations' to grailsAppResourceDirs resolve duplicates [grails-core]
Posted by "jamesfredley (via GitHub)" <gi...@apache.org>.
jamesfredley merged PR #15084:
URL: https://github.com/apache/grails-core/pull/15084
--
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: notifications-unsubscribe@grails.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add 'migrations' to grailsAppResourceDirs [grails-core]
Posted by "jamesfredley (via GitHub)" <gi...@apache.org>.
jamesfredley commented on PR #15084:
URL: https://github.com/apache/grails-core/pull/15084#issuecomment-3309885864
This fixed bootRun, bootJar and bootWar, but causes a duplication issue with dbm-update
--
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: notifications-unsubscribe@grails.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add 'migrations' to grailsAppResourceDirs [grails-core]
Posted by "jdaugherty (via GitHub)" <gi...@apache.org>.
jdaugherty commented on PR #15084:
URL: https://github.com/apache/grails-core/pull/15084#issuecomment-3310618695
dbm-update has never worked for me in the past. It doesn't load the same context that bootRun does. I'd argue dbm-update should be replaceed by something that just starts the server & shuts down.
--
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: notifications-unsubscribe@grails.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add 'migrations' to grailsAppResourceDirs [grails-core]
Posted by "jamesfredley (via GitHub)" <gi...@apache.org>.
jamesfredley commented on PR #15084:
URL: https://github.com/apache/grails-core/pull/15084#issuecomment-3314287819
Resolved duplicates issue and updated docs
--
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: notifications-unsubscribe@grails.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add 'migrations' to grailsAppResourceDirs [grails-core]
Posted by "jamesfredley (via GitHub)" <gi...@apache.org>.
jamesfredley commented on PR #15084:
URL: https://github.com/apache/grails-core/pull/15084#issuecomment-3309899229
```
> Task :dbmUpdate
. ____ _ __ _ _
/\\ / ___'_ __ _ _(_)_ __ __ _ \ \ \ \
( ( )\___ | '_ | '_| | '_ \/ _` | \ \ \ \
\\/ ___)| |_)| | | | | || (_| | ) ) ) )
' |____| .__|_| |_|_| |_\__, | / / / /
=========|_|==============|___/=/_/_/_/
:: Spring Boot :: (v3.5.6)
Grails application running at http://localhost:0 in environment: development
ERROR: Exception Details
ERROR: Exception Primary Class: IOException
ERROR: Exception Primary Reason: Found 2 files with the path 'test-domain.groovy':
- file:///C:/Users/james/Documents/IdeaProjects/database-migration-test/grails-app/migrations/test-domain.groovy
- file:///C:/Users/james/Documents/IdeaProjects/database-migration-test/build/resources/main/test-domain.groovy
Search Path:
- C:\Users\james\Documents\IdeaProjects\database-migration-test\grails-app\migrations
You can limit the search path to remove duplicates with the liquibase.searchPath setting. Or, if you KNOW these are the exact same file you can set liquibase.duplicateFileMode=WARN.
ERROR: Exception Primary Source: H2 2.3.232 (2024-08-11)
```
--
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: notifications-unsubscribe@grails.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org