[illumos-Developer] Illumos issue 278: (eliminate python dependency from zfs command)

[Alexander Stetsenko]

Hi Gordon

On 10/21/2010 06:22 AM, Gordon Ross wrote:
> see below
>
> On Wed, Oct 20, 2010 at 3:31 PM, Alexander Stetsenko<ams at nexenta.com>  wrote:
>    
>>> On 10/20/2010 08:07 AM, Gordon Ross wrote:
>>> Could you please provide a pointer to the (python?) code this replaces?
>>>        
>> It should replace all cmd/pyzfs lib/pyzfs stuff.
>> I still have to provide replacement for allow/unallow, so python code is
>> still not removed  - and there is no webrev available for it.
>>
>> You can look into lib/pyzfs/holds.py (for holds subcommand),
>> lib/pyzfs/common/userspace.py and lib/pyzfs/common/groupspace.py
>> (for userspace/groupspace one) and lib/pyzfs/common/ioctl.c for both.
>>
>> cmd/pyzfs calls pyzfs module.
>>      
> Thanks, that's what I wanted to review for context.
>
> After looking at lib/pyzfs, I have a suggestion:
> Write a C++ library of classes following the design of
> the classes provided in the python ZFS library code.
>
> I realize the "C++ is Evil!" bigots may jump on my case
> over this suggestion, but for this particular usage, I'd
> suggest this is a better approach than converting that
> code into a language with no classes or inheritance.
>
> My humble opinion.
>
> (And if you haven't guessed, I don't subscribe to the
> "C++ is Evil!" fan club.  My view is that no language
> prevents people from writing unmaintainable code:)
>    
I really like C++ and I guess it is the best for application development,
but for system stuff C is better (IMHO :)
If you want C++ for zfs we have to rewrite zfs command completely.
I really do not want to be drown in zfs improving :)
> Gordon
>