Rafael Vanoni wrote:

> Li, Aubrey wrote:
>> Rafael.Vanoni wrote:
>> 
>>> Hey everyone
>>> 
>>> This is the same patch I posted here last time, but without the -i
>>> option, as I'm still working on that. This one allows non-priv users
>>> to use the tool (if they have dtrace privileges) and some more
>>> code cleanup.
>>> 
>>> Thought it would be best to get these changes out of the way
>>> and have a
>>> cleaner patch once I fix the -i option. Hope you agree.
>>> 
>>> thanks
>>> Rafael
>> 
>> Yeah, this one looks good, please feel free to commit into repo.
>> 
>> Since I can't commit code directly into any tesla repo any more,
>> the battery patch I submitted is also based on rev5931, which
>> would fail to import after you commit your "non-priv user" patch.
>> You might take care of it or just let me know I have to re-submit
>> the patch against your 5932.
> 
> Hi Aubrey
> 
> I imported your patch on top of my repo (which contained my 5932) and
> it worked fine :)
> 
> $ hg tip
> changeset:   5933:0a4a60b78db5
> tag:         tip
> user:        Aubrey Li <aubrey.li at intel.com>
> date:        Mon Apr 21 02:09:18 2008 +0800
> summary:     add battery consumption information
> 
> I'll push my changes and then yours, if that's alright with you.

That would be great, thanks.

> How come you can't commit code to tesla anymore ?
> 

There is a story about the code license between Sun and Intel.
Although it doesn't make any sense of open source project,
I have to defer to it.

Thanks,
-Aubrey

Reply via email to