slachiewicz opened a new pull request, #315: URL: https://github.com/apache/maven-source-plugin/pull/315
This plugin was still on `4.0.0-beta-3`, the furthest behind of the 4-native plugins, and it is a migration rather than a version bump. **Dependencies.** `maven-api-meta` and `maven-api-impl` no longer exist; they become `maven-api-annotations` and `org.apache.maven:maven-testing` respectively, matching what maven-jar-plugin already does. `maven-archiver` moved to `org.apache.maven.shared`. **Source roots.** `ProjectManager.getCompileSourceRoots(project, scope)` and `getResources(project, scope)` are gone; resources and source roots are unified into `SourceRoot`: | before | after | |---|---| | `getCompileSourceRoots(p, MAIN)` | `getEnabledSourceRoots(p, MAIN, Language.JAVA_FAMILY).map(SourceRoot::directory)` | | `getResources(p, MAIN)` | `getEnabledSourceRoots(p, MAIN, Language.RESOURCES)` | | `Resource.getDirectory()` | `SourceRoot.directory()` | | `Resource.getIncludes()` / `getExcludes()` | `SourceRoot.includes()` / `excludes()` | | `Resource.getTargetPath()` | `SourceRoot.targetPath()` | | `session.createArtifact(...)` | `session.createProducedArtifact(...)` | `attachArtifact` now takes a `ProducedArtifact`. The `maven-shared-archive-resources` special case moves onto the same `SourceRoot` stream. Include/exclude and target-path handling are preserved, so which files land in the sources jar should not change. **site.xml.** The `<skin>` workaround is removed — its own comment said it was only needed on beta-3 and is redundant from rc-5 onwards. **Verification.** `mvn test`: 10 tests, 0 failures. `mvn verify -Prun-its`: 16 ITs pass, `MSOURCES-140` fails. That one failure is **not** introduced here — checking out unmodified `master` and running the same IT under the same Maven fails identically. Two separate things are wrong with it: 1. Its assertion cannot ever match. `verify.groovy:22` is a Groovy slashy-string regex containing the literal text `" + File.separator + "` — Java string concatenation written inside a regex literal. 2. Even with that corrected, it still fails: the expected `already attached ... ignoring same re-attach` line is absent from the build log entirely, so the re-attach path is not being reached. I have the regex correction ready but have not opened it, since on its own it does not turn the IT green and the second point wants a maintainer's eye. Happy to send it if useful. Part of apache/maven#12676. -- 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]
