Skip to content

NIFI-15633 - NiFi Maven - address deprecations#46

Merged
exceptionfactory merged 2 commits intoapache:mainfrom
pvillard31:NIFI-15633
Feb 23, 2026
Merged

NIFI-15633 - NiFi Maven - address deprecations#46
exceptionfactory merged 2 commits intoapache:mainfrom
pvillard31:NIFI-15633

Conversation

@pvillard31
Copy link
Contributor

Deprecations fixed

  • @Component annotation (10+ occurrences). Replaced with JSR-330 @Inject / @Named across NarMojo, NarDuplicateDependenciesMojo, and NarProvidedDependenciesMojo
  • ArtifactFactory, Replaced with org.apache.maven.repository.RepositorySystem in NarMojo
  • MavenArchiver.configureReproducible(), Replaced with configureReproducibleBuild() plus parseBuildOutputTimestamp() in NarMojo
  • AbstractArchiver.addDirectory(). Replaced with addFileSet(DefaultFileSet) in NarMojo
  • MavenProject.getDependencyArtifacts(). Replaced with getDependencies() in NarMojo and NarDependencyUtils

Deprecations not addressed (no fix available in Maven 3.9.x)

  • ArtifactRepository. ProjectBuildingRequest and ArtifactResolutionRequest still require this type; there is no Aether-based alternative for these APIs until Maven 4.x
  • ArtifactHandlerManager.addHandlers(). Deprecated with no replacement API provided; needed to register the custom NAR artifact handler at runtime

Copy link
Contributor

@exceptionfactory exceptionfactory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for addressing these deprecations @pvillard31, the approach looks good using the new JSR annotations and associated plugin. If you can rebase, I will proceed with merging.

@exceptionfactory exceptionfactory merged commit 11c57bc into apache:main Feb 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants