[GitHub] [geode-examples] rhoughton-pivot opened pull request #86: Add Gradle option to consume Geode as an includeBuild

["rhoughton-pivot (GitHub)" <gi...@apache.org>]

Building examples has required a published Geode tgz and jars. Use
Gradle's `includeBuild` feature to allow mapping to a custom Geode clone
for integration and API testing. Invoke with:
`./gradlew -Dcomposite -PgeodeCompositeDirectory=../geode build`

[ Full content available at: https://github.com/apache/geode-examples/pull/86 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-examples] onichols-pivotal commented on issue #86: Add Gradle option to consume Geode as an includeBuild

["onichols-pivotal (GitHub)" <gi...@apache.org>]

out of curiosity, I tried introducing compile error in geode-management, which is a module that geode-core depends on, and it was not picked up.   So does the "composite" feature only apply to the 3 modules explicitly listed, not their dependencies?

Also this should be merged to develop, probably not master although both is probably harmless enough.

[ Full content available at: https://github.com/apache/geode-examples/pull/86 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-examples] rhoughton-pivot closed pull request #86: Add Gradle option to consume Geode as an includeBuild

["rhoughton-pivot (GitHub)" <gi...@apache.org>]

[ pull request closed by rhoughton-pivot ]

[ Full content available at: https://github.com/apache/geode-examples/pull/86 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-examples] rhoughton-pivot commented on issue #86: Add Gradle option to consume Geode as an includeBuild

["rhoughton-pivot (GitHub)" <gi...@apache.org>]

@onichols-pivotal I'll test that more on my side. `-Dcomposite` should pick up such compile-time errors on all sub-projects of the dependencies we name (an error in geode-management should cause a problem for geode-core, and therefore in geode-examples). 


[ Full content available at: https://github.com/apache/geode-examples/pull/86 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-examples] onichols-pivotal commented on issue #86: Add Gradle option to consume Geode as an includeBuild

["onichols-pivotal (GitHub)" <gi...@apache.org>]

oh, it was user error on my part...I was in geode-web-management not geode-management.  It does pick up changes to geode-management just fine :)

[ Full content available at: https://github.com/apache/geode-examples/pull/86 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org