#1 update to version 3.1.1
Merged a month ago by decathorpe. Opened 2 months ago by decathorpe.

file modified
+7 -3

@@ -1,8 +1,11 @@

- plexus-io-1.0-alpha-5.tar.bz2

+ /results_*

+ /*.src.rpm

+ 

+ /plexus-io-1.0-alpha-5.tar.bz2

  /plexus-io-1.0.1.tar.xz

- plexus-io-2.0.1.tar.xz

+ /plexus-io-2.0.1.tar.xz

  /plexus-io-2.0.2.tar.xz

- sonatype-plexus-io-plexus-io-2.0.4-0-g2767dfe.tar.gz

+ /sonatype-plexus-io-plexus-io-2.0.4-0-g2767dfe.tar.gz

  /sonatype-plexus-io-plexus-io-2.0.5-0-g4a31728.tar.gz

  /plexus-io-2.0.5.tar.gz

  /LICENSE-2.0.txt

@@ -20,3 +23,4 @@

  /plexus-io-2.6.tar.gz

  /plexus-io-2.7.1.tar.gz

  /plexus-io-3.0.0.tar.gz

+ /plexus-io-3.1.1.tar.gz

file modified
+19 -8

@@ -1,18 +1,18 @@

  Name:           plexus-io

- Version:        3.0.0

- Release:        5%{?dist}

  Summary:        Plexus IO Components

+ Version:        3.1.1

+ Release:        1%{?dist}

  License:        ASL 2.0

- URL:            https://github.com/codehaus-plexus/plexus-io

- BuildArch:      noarch

  

- Source0:        https://github.com/codehaus-plexus/plexus-io/archive/plexus-io-%{version}.tar.gz

+ URL:            https://github.com/codehaus-plexus/%{name}

+ Source0:        %{url}/archive/%{name}-%{version}.tar.gz

  Source1:        http://www.apache.org/licenses/LICENSE-2.0.txt

  

+ BuildArch:      noarch

+ 

  BuildRequires:  maven-local

  BuildRequires:  mvn(com.google.code.findbugs:jsr305)

  BuildRequires:  mvn(commons-io:commons-io)

- BuildRequires:  mvn(org.apache.maven.plugins:maven-enforcer-plugin)

  BuildRequires:  mvn(org.codehaus.plexus:plexus-container-default)

  BuildRequires:  mvn(org.codehaus.plexus:plexus:pom:)

  BuildRequires:  mvn(org.codehaus.plexus:plexus-utils)

@@ -21,10 +21,10 @@

  Plexus IO is a set of plexus components, which are designed for use

  in I/O operations.

  

- %package javadoc

+ %package        javadoc

  Summary:        Javadoc for %{name}

  

- %description javadoc

+ %description    javadoc

  API documentation for %{name}.

  

  

@@ -33,14 +33,21 @@

  cp %{SOURCE1} .

  

  %pom_remove_plugin :animal-sniffer-maven-plugin

+ %pom_remove_plugin :maven-enforcer-plugin

+ 

+ # junit isn't actually used

+ %pom_remove_dep :junit

+ 

  

  %build

  %mvn_file  : plexus/io

  %mvn_build

  

+ 

  %install

  %mvn_install

  

+ 

  %files -f .mfiles

  %license NOTICE.txt LICENSE-2.0.txt

  

@@ -49,6 +56,9 @@

  

  

  %changelog

+ * Mon Jul 29 2019 Fabio Valentini <decathorpe@gmail.com> - 3.1.1-1

+ - Update to version 3.1.1.

+ 

  * Fri Jul 26 2019 Fedora Release Engineering <releng@fedoraproject.org> - 3.0.0-5

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

  

@@ -190,3 +200,4 @@

  

  * Wed Dec 23 2009 Alexander Kurtakov <akurtako@redhat.com> 1.0-0.1.a5.1

  - Initial package

+ 

file modified
+1 -1

@@ -1,2 +1,2 @@

- SHA512 (plexus-io-3.0.0.tar.gz) = 5f1d028d963bf782168d32ff71633932e0c257a6bded656f5a35020df5285f2a3d4ac60fe6165d927c2f961509ea6d34507161897e1ba2f6ca4c9aaefeba9e18

+ SHA512 (plexus-io-3.1.1.tar.gz) = 14a3c09d9e5f3218a481de56981701cd8ce56090855ba5c909f00ba8bbb9443ff8b55f320725fce06d246d39ff6346772aeb9f63dc2f53f1ea1b26b0277a2107

  SHA512 (LICENSE-2.0.txt) = 98f6b79b778f7b0a15415bd750c3a8a097d650511cb4ec8115188e115c47053fe700f578895c097051c9bc3dfb6197c2b13a15de203273e1a3218884f86e90e8

Test rebuilds of dependent packages were done in COPR:

https://copr.fedorainfracloud.org/coprs/decathorpe/plexus-io-pr/monitor/

There are two additional build failures on top of the current state of rawhide (the usual suspects):

  • java-uuid-generator: always FTBFS in COPR, builds fine in koji
  • openhft-affinity: same

The update to version 3.1.1 is already available from modular stream branches:

So this PR brings rawhide up to speed with modular versions of this package.

rebased onto a29e212

2 months ago

↑ rebased for after the mass rebuild

Minor: The modular version does not include junit as BR.

Though junit 4.12 is listed as a dep in pom.xml, I don't see any junit testcases being run. Do we need to include this?

Other than my above comment, this PR LGTM! :thumbsup:

Thanks for the PR, @decathorpe

rebased onto 3658975

2 months ago

@dmoluguw you're absolutely right, junit isn't actually used. I've removed the dependency.

Noisy diff, but I'll ACK too.

Pull-Request has been merged by decathorpe

a month ago