Re: [zones-discuss] Solaris10-Branded Zones Webrev: CR 6882732

2009-12-10 Thread Frank Batschulat (Home) 
On Wed, 09 Dec 2009 23:54:05 +0100, Jordan Vaughan jordan.vaug...@sun.com 
wrote:

 I need someone to review my fix for

 6882732 unpacking archive with extended file attributes reports errors

 The webrev is accessible via

 http://cr.opensolaris.org/~flippedb/onnv-s10c

looks good to me.

cheers
---
frankB
___
zones-discuss mailing list
zones-discuss@opensolaris.org

Re: [zones-discuss] Solaris10-Branded Zones Webrev: CR 6882732

2009-12-09 Thread Edward Pilatowicz 
On Wed, Dec 09, 2009 at 02:54:05PM -0800, Jordan Vaughan wrote:
 I need someone to review my fix for

 6882732 unpacking archive with extended file attributes reports errors

 The webrev is accessible via

 http://cr.opensolaris.org/~flippedb/onnv-s10c


lgtm.
ed
___
zones-discuss mailing list
zones-discuss@opensolaris.org

Re: [zones-discuss] Solaris10-Branded Zones Webrev: CR 6882732

2009-12-09 Thread Jerry Jelinek 

Jordan Vaughan wrote:

I need someone to review my fix for

6882732 unpacking archive with extended file attributes reports errors

The webrev is accessible via

http://cr.opensolaris.org/~flippedb/onnv-s10c


Jordan,

Nice job, this looks good to me.

Thanks,
Jerry
___
zones-discuss mailing list
zones-discuss@opensolaris.org

Re: [zones-discuss] Solaris10-Branded Zones Webrev: CR 6882732

2009-12-09 Thread Jordan Vaughan 

On 12/ 9/09 03:04 PM, Edward Pilatowicz wrote:

On Wed, Dec 09, 2009 at 02:54:05PM -0800, Jordan Vaughan wrote:

I need someone to review my fix for

6882732 unpacking archive with extended file attributes reports errors

The webrev is accessible via

http://cr.opensolaris.org/~flippedb/onnv-s10c



lgtm.
ed


Thanks Ed.
___
zones-discuss mailing list
zones-discuss@opensolaris.org

Re: [zones-discuss] Solaris10-Branded Zones Webrev: CR 6882732

2009-12-09 Thread Jordan Vaughan 

On 12/ 9/09 03:45 PM, Jerry Jelinek wrote:

Jordan Vaughan wrote:

I need someone to review my fix for

6882732 unpacking archive with extended file attributes reports errors

The webrev is accessible via

http://cr.opensolaris.org/~flippedb/onnv-s10c


Jordan,

Nice job, this looks good to me.

Thanks,
Jerry


Thanks Jerry!
___
zones-discuss mailing list
zones-discuss@opensolaris.org