Bug 590522 - Review Request: plexus-cipher - Plexus Cipher: encryption/decryption Component
Review Request: plexus-cipher - Plexus Cipher: encryption/decryption Component
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Mat Booth
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-09 23:17 EDT by huwang
Modified: 2010-06-04 02:07 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-06-03 21:58:39 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mat.booth: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)
spec file (3.00 KB, text/plain)
2010-05-10 03:26 EDT, huwang
no flags Details
SRPM (30.43 KB, application/x-rpm)
2010-05-10 03:31 EDT, huwang
no flags Details

  None (edit)
Description huwang 2010-05-09 23:17:05 EDT
Spec URL: <spec info here>
SRPM URL: <srpm info here>
Description: Plexus Cipher: encryption/decryption Component
Comment 1 Alexander Kurtakov 2010-05-10 03:14:47 EDT
You haven't put links to your spec and srpm.
Comment 2 huwang 2010-05-10 03:26:33 EDT
Created attachment 412735 [details]
spec file
Comment 3 huwang 2010-05-10 03:31:04 EDT
Created attachment 412737 [details]
SRPM
Comment 4 huwang 2010-05-10 04:30:03 EDT
(In reply to comment #0)
> Spec URL: http://hotfile.com/dl/42162162/56b2406/plexus-cipher.spec.html
> SRPM URL: http://hotfile.com/dl/42162040/79e5c4c/plexus-cipher-1.4-1.src.rpm.html
> Description: Plexus Cipher: encryption/decryption Component
Comment 5 Mat Booth 2010-05-12 16:03:49 EDT
I will take this.

I'm happy for you to continue putting your SPEC and SRPM on the bug as attachments, by the way. :-)
Comment 6 Mat Booth 2010-05-13 17:08:59 EDT
Those parent poms, are there any other plexus modules that might need these to build? I'm just wondering if it is worth getting these poms into the maven2-common-poms package like we did with the Apache commons-parent pom...
Comment 7 Guido Grazioli 2010-05-13 17:31:31 EDT
(In reply to comment #6)
> Those parent poms, are there any other plexus modules that might need these to
> build? I'm just wondering if it is worth getting these poms into the
> maven2-common-poms package like we did with the Apache commons-parent pom...    

Could be useful, take a look here -> bug 591454
Comment 8 Mat Booth 2010-05-13 17:41:11 EDT
Ah, well spotted. Huwang, what do you think about making this depend on the spice-parent package instead of bundling the pom here?
Comment 9 huwang 2010-05-13 22:27:36 EDT
(In reply to comment #8)
> Ah, well spotted. Huwang, what do you think about making this depend on the
> spice-parent package instead of bundling the pom here?    

There are 3 modules depend on spice-parent package: plexus-cipher, plexus-build-api and plexus-sec-dispatcher. So I added spice-parent package which is only a pom file.
Comment 10 Mat Booth 2010-05-15 08:15:55 EDT
Excellent, please update your SPEC/SRPM and I will conduct a full review.
Comment 11 Alexander Kurtakov 2010-05-19 09:11:05 EDT
huwang, Please do what Mat requested
Comment 12 huwang 2010-05-20 04:49:22 EDT
(In reply to comment #10)
> Excellent, please update your SPEC/SRPM and I will conduct a full review.    

Sorry for the delay, as I updated it to the latest version, please review again, thanks a lot.

spec URL: http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher.spec&can=2&q=#makechanges

SRPM URL:  http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher-1.5-1.src.rpm&can=2&q=#makechanges

Note about this package:
 It built with dist-f14-maven221 target, koji building link is http://koji.fedoraproject.org/koji/taskinfo?taskID=2198806
Comment 13 huwang 2010-05-20 07:26:38 EDT
Rebuilt in koji with dist-f14-maven221 target, the link is http://koji.fedoraproject.org/koji/taskinfo?taskID=2198918
Comment 14 Mat Booth 2010-05-21 07:42:21 EDT
Hmm, I rebuilt your SPRM again myself, and this time it failed:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2200828

Also, in both our builds, it fails to get this pom file:
JPP/maven2/default_poms/JPP-apache-commons-logging.pom

Seems like it should be finding this one first anyway:
JPP/maven2/poms/JPP-commons-logging.pom

What's going on there?

Also, please provide a valid url in the spec file URL tag.
Comment 15 Stanislav Ochotnicky 2010-05-21 07:58:00 EDT
I am sorry, this is actually a problem in commons-logging. I am fixing it right now. Sorry for causing trouble...
Comment 16 Mat Booth 2010-05-21 08:05:27 EDT
No problem, thanks for your attention Stanislav :-)
Comment 17 Stanislav Ochotnicky 2010-05-21 08:18:36 EDT
Just to explain (might be useful for future):
One of the dependencies that plexus-cipher pulls in is using old jakarta-commons-logging package I assume. It had wrong groupId (commons-logging instead of org.apache.commons) so I had to fix it when I renamed package to apache-commons-logging. I tried to keep backward compatibility but I forgot to change one call to use correct name...
Comment 18 Mat Booth 2010-05-21 11:05:41 EDT
Ok the commons-logging thing has been fixed, but it still fails to build for me:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2201405


[INFO] ------------------------------------------------------------------------
[ERROR] BUILD ERROR
[INFO] ------------------------------------------------------------------------
[INFO] Internal error in the plugin manager executing goal 'org.apache.maven.plugins:maven-javadoc-plugin:2.4:javadoc': Unable to find the mojo 'org.apache.maven.plugins:maven-javadoc-plugin:2.4:javadoc' in the plugin 'org.apache.maven.plugins:maven-javadoc-plugin'
org.apache.maven.doxia.logging.LogEnabled
[INFO] ------------------------------------------------------------------------


I'm not sure what "Unable to find the mojo" means, is it a problem with this package or a problem in the maven javadoc plugin?
Comment 19 huwang 2010-05-24 22:43:45 EDT
(In reply to comment #18)
> Ok the commons-logging thing has been fixed, but it still fails to build for
> me:
> 
> http://koji.fedoraproject.org/koji/taskinfo?taskID=2201405
> 
> 
> [INFO] ------------------------------------------------------------------------
> [ERROR] BUILD ERROR
> [INFO] ------------------------------------------------------------------------
> [INFO] Internal error in the plugin manager executing goal
> 'org.apache.maven.plugins:maven-javadoc-plugin:2.4:javadoc': Unable to find the
> mojo 'org.apache.maven.plugins:maven-javadoc-plugin:2.4:javadoc' in the plugin
> 'org.apache.maven.plugins:maven-javadoc-plugin'
> org.apache.maven.doxia.logging.LogEnabled
> [INFO] ------------------------------------------------------------------------
> 
> 
> I'm not sure what "Unable to find the mojo" means, is it a problem with this
> package or a problem in the maven javadoc plugin?    

I think this isn't this package's problem. I built again, it isn't failed, the link is http://koji.fedoraproject.org/koji/taskinfo?taskID=2207394

spec URL:
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher.spec&can=2&q=#makechanges

SRPM URL: 
http://code.google.com/p/tobepackagemaintaner/downloads/detail?name=plexus-cipher-1.5-1.src.rpm&can=2&q=#makechanges

please review again, thanks.
Comment 20 Alexander Kurtakov 2010-05-28 03:25:22 EDT
Mat, 
Can you finish this?
If not please drop yourself and I'll do it. It's blocking other packages.
Comment 21 Mat Booth 2010-05-28 05:51:17 EDT
Sorry, I have been very busy this week. I can finish it at the weekend, if that is ok.
Comment 22 Mat Booth 2010-05-31 12:00:11 EDT
Hi,

This builds for me now finally, thanks.

Rpmlint output:

  W: spelling-error Summary(en_US) decryption
  W: spelling-error %description -l en_US decryption

Ignore these spelling warnings, "decryption" is a valid noun form of the verb "decrypt." What a terrible dictionary it's using.

  W: non-conffile-in-etc /etc/maven/fragments/plexus-cipher

This is normal and may also be ignored.

  W: no-documentation

Please include the NOTICE.txt file as documentation in the main package.

  W: invalid-url URL: http://spice.sonatype.org/plexus-cipher HTTP Error 404: Not Found
  W: invalid-url URL: http://spice.sonatype.org/plexus-cipher HTTP Error 404: Not Found

Please include a valid URL in the URL tag. I've tried this in my browser, there is definitely no page at this location.

---

I've been through the guidelines and the review check list and I believe the package is acceptable except for the two items from the rpmlint output that I've highlighted above.
Comment 23 huwang 2010-05-31 21:32:37 EDT
(In reply to comment #22)
> Hi,
> 
> This builds for me now finally, thanks.
> 
> Rpmlint output:
> 
>   W: spelling-error Summary(en_US) decryption
>   W: spelling-error %description -l en_US decryption
> 
> Ignore these spelling warnings, "decryption" is a valid noun form of the verb
> "decrypt." What a terrible dictionary it's using.
> 
>   W: non-conffile-in-etc /etc/maven/fragments/plexus-cipher
> 
> This is normal and may also be ignored.
> 
>   W: no-documentation
> 
> Please include the NOTICE.txt file as documentation in the main package.
Fixed.
> 
>   W: invalid-url URL: http://spice.sonatype.org/plexus-cipher HTTP Error 404:
> Not Found
>   W: invalid-url URL: http://spice.sonatype.org/plexus-cipher HTTP Error 404:
> Not Found
> 
> Please include a valid URL in the URL tag. I've tried this in my browser, there
> is definitely no page at this location.
Fixed.
> 
> ---
> 
> I've been through the guidelines and the review check list and I believe the
> package is acceptable except for the two items from the rpmlint output that
> I've highlighted above.    

Please review again, thanks.

Spec URL: http://huwang.fedorapeople.org/packages/plexus_cipher/plexus-cipher.spec
SRPM URL: http://huwang.fedorapeople.org/packages/plexus_cipher/plexus-cipher-1.5-3.src.rpm

here is the build link: http://koji.fedoraproject.org/koji/taskinfo?taskID=2221097
Comment 24 Mat Booth 2010-06-01 04:56:53 EDT
Nice one, that looks fine to me.

APPROVED
Comment 25 huwang 2010-06-01 06:05:54 EDT
New Package CVS Request
=======================
Package Name: plexus-cipher
Short Description: Plexus Cipher: encryption/decryption Component
Owners: huwang
Branches: 
InitialCC:
Comment 26 Kevin Fenzi 2010-06-03 16:13:49 EDT
CVS done (by process-cvs-requests.py).

Please don't put versions in the Summary of review tickets... ;)
Comment 27 huwang 2010-06-03 21:57:53 EDT
Built in koji : http://koji.fedoraproject.org/koji/taskinfo?taskID=2228830

Note You need to log in before you can comment on or make changes to this bug.