#4 remove unnecessary dependency on parent POM
Merged 4 years ago by decathorpe. Opened 4 years ago by decathorpe.
Unknown source master  into  master

file modified
+7 -3
@@ -7,7 +7,7 @@

  

  Name:           netty

  Version:        4.1.13

- Release:        12%{?dist}

+ Release:        13%{?dist}

  Summary:        An asynchronous event-driven network application framework and tools for Java

  License:        ASL 2.0

  URL:            https://netty.io/
@@ -34,10 +34,8 @@

  BuildRequires:  mvn(org.codehaus.mojo:build-helper-maven-plugin)

  BuildRequires:  mvn(org.codehaus.mojo:exec-maven-plugin)

  BuildRequires:  mvn(org.fusesource.hawtjni:maven-hawtjni-plugin)

- BuildRequires:  mvn(org.javassist:javassist)

  BuildRequires:  mvn(org.jctools:jctools-core)

  BuildRequires:  mvn(org.slf4j:slf4j-api)

- BuildRequires:  mvn(org.sonatype.oss:oss-parent:pom:)

  %if %{without jp_minimal}

  BuildRequires:  mvn(com.fasterxml:aalto-xml)

  BuildRequires:  mvn(com.github.jponge:lzma-java)
@@ -84,6 +82,9 @@

  %patch3 -p1

  %endif

  

+ # remove unnecessary dependency on parent POM

+ %pom_remove_parent . bom dev-tools

+ 

  # Missing Mavenized rxtx

  %pom_disable_module "transport-rxtx"

  %pom_remove_dep ":netty-transport-rxtx" all
@@ -213,6 +214,9 @@

  %license LICENSE.txt NOTICE.txt

  

  %changelog

+ * Thu Nov 21 2019 Fabio Valentini <decathorpe@gmail.com> - 4.1.13-13

+ - Remove unnecessary dependency on parent POM.

+ 

  * Thu Jul 25 2019 Fedora Release Engineering <releng@fedoraproject.org> - 4.1.13-12

  - Rebuilt for https://fedoraproject.org/wiki/Fedora_31_Mass_Rebuild

  

Clean up dependencies and remove unnecessary dependency on parent POM.

Looks like it's not needed for anything, as the package builds successfully without it, and dependent packages also still build:
https://copr.fedorainfracloud.org/coprs/decathorpe/netty-no-parent-pr/monitor/

Pull-Request has been merged by decathorpe

4 years ago