Closed jbasney closed 11 years ago
To me it looks like 5 independent changes. Could you split it to 5 poll requests or at least 5 commits? This will make review and merge process much simpler.
I briefly reviewed the 4 effected classed (MyProxy.java, MyProxyCLI.java, MyProxyException.java, Base64.java) and merged with my local checkin. If there are no objections, I would like to merge this PR and test it more with a higher level test suite - https://svn.apache.org/repos/asf/airavata/sandbox/grid-tools/gsi-myproxy-client/.
Hi Jim, Can you please also do a pull request on the 2.0.6 branch and sync all these JGlobus 2 fixes from CILogin CVS?
Suresh, I don't know how to re-issue this pull request for the 2.0.6 branch. I tried for an hour now to figure it out without success.
Hi Jim, I tried the same yesterday without much success. I finally cherry picked'ed Amila's patches. I will probably do the same with your PR as well. I will attempt it later today.
Jim, I cherry picked your commit into 2.0.6. I will now proceed with staging the RC.
MyProxy JGlobus 2.0 fixes from CILogon CVS: