From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jeff Garzik <jeff@garzik.org>
Subject: Re: CLD doesn't build on db-4.3
Date: Thu, 01 Apr 2010 11:03:48 -0400
Message-ID: <4BB4B5D4.2000300@garzik.org>
References: <4BB47D27.6080405@samba.org>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Return-path: <hail-devel-owner@vger.kernel.org>
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=gamma;
        h=domainkey-signature:received:received:sender:message-id:date:from
         :user-agent:mime-version:to:cc:subject:references:in-reply-to
         :content-type:content-transfer-encoding;
        bh=+uYQKUq98OrE6pBZmqZp6SnLdTEpmys5gjH9KEHz00U=;
        b=ZJMYu84f3BKrOne96MrV6WdWXVxTYZ1fEUqRxyG2tVS5jhTsJG1obS/EszengGiIZm
         +qVD/Xo5VJMzl/7u75JsxbFizYlXMRycHSJBPEa2WeRlIYhwWOOMgVJazlpQna51HECS
         CLw4BVrzrQHqjVFugefWVhYppaxk62NBtxGTQ=
In-Reply-To: <4BB47D27.6080405@samba.org>
Sender: hail-devel-owner@vger.kernel.org
List-ID: <hail-devel.vger.kernel.org>
Content-Type: text/plain; charset="us-ascii"; format="flowed"
To: Samba - BoYang <boyang@samba.org>
Cc: hail-devel@vger.kernel.org

On 04/01/2010 07:01 AM, Samba - BoYang wrote:
> hi, *
>      CLD doesn't build on db-4.3 on suse 11, since db-4.3 uses deprecated
> structure members DBC->c_xxx(c_close(), etc) instead of DBC->xxx. :-)
>
>      It won't build on db-4.4, either. probably won't build on db-4.5, as
> db-5.0 says DBC->xxx was introduced in db-4.6. :-) Should we disable
> support for 4.3 - 4.5 and add 4.9 - 5.0?

I'd answer "yes", by a circuitous route:  if I understand things 
correctly, the replicated PAXOS db4 backend that we are heading towards 
(see the 'replica' branch of 
git://git.kernel.org/pub/scm/daemon/cld/cld.git) was buggy in early db4 
releases.

Therefore, it sounds like we could eliminate two issues with a single 
change, by removing support for db 4.3 - 4.5, the DBC issue and the 
PAXOS issue.

I'm fine with adding support for 4.9+ as long as the APIs function in a 
compatible manner.

Want to create the simple patch for this?  :)

Thanks,

	Jeff