Hi Octave,

Are you asking about the new code for the ManifestParser and ManifestWriter,
or is your comment in relation to one of the other "Code review request..."
emails that are all flying around together?

MP/MW have unit test code supplied along with the module code and these
tests all pass.  Also, it is currently being used in the new DC app which
will provide integration testing of code.

- Dermot


On 10/11/10 16:53, Octave Orgeron wrote:
What is the best way for someone to test out these code changes? I have a number of bugs filed against AI and I'd like to get a snapshot and see how things are progressing.

 *-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*
Octave J. Orgeron
Solaris Virtualization Architect and Consultant
Web: http://unixconsole.blogspot.com
E-Mail: [email protected]
*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*



----- Original Message ----
From: Evan Layton <[email protected]>
To: Dermot McCluskey <[email protected]>
Cc: caiman-discuss <[email protected]>
Sent: Mon, October 11, 2010 10:14:38 AM
Subject: Re: [caiman-discuss] Code review request: ManifestParser

On 10/8/10 7:36 AM, Dermot McCluskey wrote:
Hi,

Anyone else got time to review this?

I've already had one review for this and one more promised. If I could
get a third person to look over that would be great.

Thanks,
- Dermot


I'm looking at it.

-evan

On 09/30/10 15:40, Dermot McCluskey wrote:
Hi All,

I'd like a code review for the ManifestParser/ManifestWriter
checkpoints for the CUD project. The changes are here:
http://cr.opensolaris.org/~dermot/webrev_mp_mw_01/

Please sent comments by 10/12/2010.


Note: this webrev just includes the new files under usr/src/lib/install_manifest.
There are also a very small number of changes to higher-level Makefiles, etc
which have been omitted here for convenience.


Thanks,
- Dermot

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss




_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to