[GitHub] [maven] mthmulders commented on pull request #744: [MNG-7360] Fix xml transformation to ensure proper context

2022-06-14 Thread GitBox
mthmulders commented on PR #744: URL: https://github.com/apache/maven/pull/744#issuecomment-1155667002 > @mthmulders would you mind approving the PR for the record ? My pleasure! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [maven] mthmulders commented on pull request #744: [MNG-7360] Fix xml transformation to ensure proper context

2022-05-24 Thread GitBox
mthmulders commented on PR #744: URL: https://github.com/apache/maven/pull/744#issuecomment-1135517750 Ran it on Camel again, and this time, it succeeds! (OK, it fails, but for a completely different reason). Also, the [added IT](https://github.com/apache/maven-integration-testing/pull/167)

[GitHub] [maven] mthmulders commented on pull request #744: [MNG-7360] Fix xml transformation to ensure proper context

2022-05-19 Thread GitBox
mthmulders commented on PR #744: URL: https://github.com/apache/maven/pull/744#issuecomment-1132543244 Although the changes look sane, I'm not sure we're on the right path. I've tried building this branch of Maven manually (succeeded) and then use that `mvn` to build Camel. Still fails: