Karen:

slim unit tests are broken due to the /system/violatile corruption issue and we are still waiting for the fix to put back.

I did ran the lib/install_target/test and cmd/text-install/test and there were no failures.


/net/osol-bldx/datapool/ding/7174597/slim_source/usr/src/tools/tests/test_result and test_result_1




On 06/25/12 04:05 PM, Karen Tung wrote:
Hi Mary,

The changes look good to me.  I assume you ran slim unit test as well?

--Karen

On 06/25/12 15:34, Mary Ding wrote:
Hi:


Can I get two code reviewers to look at the following simple pylint bug fix relating to 7174597:

CR: http://monaco.sfbay/detail.jsf?cr=7174597
pylint errors in usr/src/cmd/text-install



Web rev is here:

https://cr.opensolaris.org/action/browse/caiman/md5645/7174597

The fix is basically removed the unused imports

1. Run pylint after the fix and confirm it no longer complains about unused imports

2. I used DC to build the x86 text install iso with my fix and can do text install without any problem.



_______________________________________________
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