On 09/24/2012 10:25 AM, Srinivasa Sarva wrote:
Please review the changes for
7200300 Upgrade sox to 14.4.0
Patches 1 to 7 are header changes only.
Patch 8 is new.
Webrev
http://jurassic.us.oracle.com/net/ssarva-us/export/home/srini/S11U2/WS4/webrev/
Why change all the patches to level 0? It doesn't gain you anything.
It's perfectly fine if the first part of the pathname doesn't match the
current version. Lots of components have these sort of patches.
I would suggest not putting the caseid all on one line in the .p5m file.
Leave it the way it was. It's a gratuitous change that's not needed.
If you just changed the things that really needed fixing your webrev would
be much smaller and it would be much easier to see what had really
changed and get a review.
Did you build it on x86 and SPARC? Can you point me at your build logs?
How did you test it? Can you point me at your results?
I see the NO_TESTS in the Makefile, but surely you testing
this in some way ...
The Bugster CR is still in a Dispatched state. Are you going to add an
Evaluation, Suggested Fix and adjust it to "Fix In progress"? Adding how
you tested it (and the test results) to the bug would also be good.
_______________________________________________
userland-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/userland-discuss
