Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-08 Thread Baptiste Daroussin
I've developed similar API during HAST work, maybe it is a good starting point? src/sbin/hastd/nv.{c,h}. What about pushing this into some public libraries, I think this could be usefull in libc ou libutil or something like that? regards, Bapt ___

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Pawel Jakub Dawidek
On Fri, Feb 04, 2011 at 11:03:53AM -0700, Justin T. Gibbs wrote: The attached patch is sufficient to allow a C++ program to use libzfs. The motivation for these changes is work I'm doing on a ZFS fault handling daemon that is written in C++. SpectraLogic's intention is to return this work to

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Justin T. Gibbs
On 2/5/2011 8:39 AM, Pawel Jakub Dawidek wrote: On Fri, Feb 04, 2011 at 11:03:53AM -0700, Justin T. Gibbs wrote: The attached patch is sufficient to allow a C++ program to use libzfs. The motivation for these changes is work I'm doing on a ZFS fault handling daemon that is written in C++.

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Pawel Jakub Dawidek
On Sat, Feb 05, 2011 at 02:36:40PM -0700, Justin T. Gibbs wrote: On 2/5/2011 8:39 AM, Pawel Jakub Dawidek wrote: On Fri, Feb 04, 2011 at 11:03:53AM -0700, Justin T. Gibbs wrote: The attached patch is sufficient to allow a C++ program to use libzfs. The motivation for these changes is work

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Justin T. Gibbs
On 2/5/2011 3:06 PM, Pawel Jakub Dawidek wrote: On Sat, Feb 05, 2011 at 02:36:40PM -0700, Justin T. Gibbs wrote: Perhaps IllumOS will accept these changes back? As I mentioned in the change descriptions included with the patch, the header files already show the intention of providing C++

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Garrett Cooper
On Sat, Feb 5, 2011 at 7:29 PM, Justin T. Gibbs gi...@scsiguy.com wrote: On 2/5/2011 3:06 PM, Pawel Jakub Dawidek wrote: On Sat, Feb 05, 2011 at 02:36:40PM -0700, Justin T. Gibbs wrote: Perhaps IllumOS will accept these changes back?  As I mentioned in the change descriptions included with

[PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-04 Thread Justin T. Gibbs
The attached patch is sufficient to allow a C++ program to use libzfs. The motivation for these changes is work I'm doing on a ZFS fault handling daemon that is written in C++. SpectraLogic's intention is to return this work to the FreeBSD project once it is a bit more complete. Since these