rhoughton-pivot commented on a change in pull request #6601:
URL: https://github.com/apache/geode/pull/6601#discussion_r650127610
##########
File path: geode-gfsh/src/test/resources/expected-pom.xml
##########
@@ -16,6 +16,11 @@
See the License for the specific language governing permissions and
limitations under the License.
-->
+ <!-- This module was also published with a richer model, Gradle metadata,
-->
+ <!-- which should be used instead. Do not delete the following line which
-->
+ <!-- is to indicate to Gradle or any Gradle module metadata file consumer
-->
+ <!-- that they should prefer consuming it instead. -->
+ <!-- do_not_remove: published-with-gradle-metadata -->
Review comment:
Gradle started its own alternative to POM files for other Gradle
projects. When Geode moved to Gradle 6.x, we started publishing them to our
snapshot repository, and this XML tag in the POM is a bread-crumb to Gradle
projects to look-aside for that file.
Problem is, that the Gradle metadata does not handle all our dependency
declarations (I do not remember the actual failure at the moment) and
downstream projects were having an issue when the pulled the metadata file
instead of the POM. So we stopped publishing the metadata until there is a push
to restore it.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
