From debbugs-submit-bounces@debbugs.gnu.org Sat Jan 24 18:38:17 2015 Received: (at submit) by debbugs.gnu.org; 24 Jan 2015 23:38:17 +0000 Received: from localhost ([127.0.0.1]:55453 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFAHQ-0004zP-Fl for submit@debbugs.gnu.org; Sat, 24 Jan 2015 18:38:17 -0500 Received: from eggs.gnu.org ([208.118.235.92]:45876) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFAHN-0004z9-TN for submit@debbugs.gnu.org; Sat, 24 Jan 2015 18:38:14 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YFAHH-0007Zw-Bx for submit@debbugs.gnu.org; Sat, 24 Jan 2015 18:38:08 -0500 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on eggs.gnu.org X-Spam-Level: X-Spam-Status: No, score=0.8 required=5.0 tests=BAYES_50 autolearn=disabled version=3.3.2 Received: from lists.gnu.org ([2001:4830:134:3::11]:59863) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YFAHH-0007Zs-92 for submit@debbugs.gnu.org; Sat, 24 Jan 2015 18:38:07 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47598) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YFAHF-0002AS-Ff for bug-gnu-emacs@gnu.org; Sat, 24 Jan 2015 18:38:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YFAHC-0007Yj-8z for bug-gnu-emacs@gnu.org; Sat, 24 Jan 2015 18:38:05 -0500 Received: from kwanyin.sergiodj.net ([176.31.208.32]:49312) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YFAHC-0007Ya-2M for bug-gnu-emacs@gnu.org; Sat, 24 Jan 2015 18:38:02 -0500 From: Sergio Durigan Junior To: bug-gnu-emacs@gnu.org Subject: [PATCH] EUDC does not support BBDB 3.x X-URL: http://blog.sergiodj.net Date: Sat, 24 Jan 2015 18:37:58 -0500 Message-ID: <87mw57hhrd.fsf@sergiodj.net> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2001:4830:134:3::11 X-Spam-Score: -5.0 (-----) X-Debbugs-Envelope-To: submit X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -5.0 (-----) This bug was opened due to a request made at: As explained at: With the release of BBDB 3.x, EUDC needs adjustments to properly work with it. Actually, after some investigation, I noticed that the only adjustment was to replace the occurrences of 'net' by 'mail' in the code. The second link above contains a patch that does that, but Thomas Fitzsimmons correctly noted (in the first link) that the patch breaks compatibility with BBDB 2.x. The following patch is a first attempt to implement the support for both BBDB 2.x and 3.x. -- Sergio GPG key ID: 0x65FC5E36 Please send encrypted e-mail if possible http://sergiodj.net/ diff --git a/lisp/ChangeLog b/lisp/ChangeLog index 15518a7..0ed1d67 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog @@ -1,3 +1,15 @@ +2015-01-24 Sergio Durigan Junior + + Adapt EUDC to BBDB 3.x + * net/eudcb-bbdb.el (eudc-bbdb-get-major-version-number): New + function. + (eudc-bbdb-attributes-translation-alist): Use + eudc-bbdb-get-major-version-number to decide whether to use 'net' + (BBDB 2.x) or 'mail' (BBDB 3.x). + (eudc-bbdb-format-query): Likewise. + (eudc-bbdb-filter-non-matching-record): Likewise. + (eudc-bbdb-format-record-as-result): Likewise. + 2015-01-23 Thomas Fitzsimmons * net/ldap.el (ldap-search-internal): Mention binddn in invalid diff --git a/lisp/net/eudcb-bbdb.el b/lisp/net/eudcb-bbdb.el index 0400e5b..d400871 100644 --- a/lisp/net/eudcb-bbdb.el +++ b/lisp/net/eudcb-bbdb.el @@ -41,9 +41,18 @@ (defvar eudc-bbdb-current-query nil) (defvar eudc-bbdb-current-return-attributes nil) +(defun eudc-bbdb-get-major-version-number () + "Return the major version number of the BBDB version we are + currently using. Return nil if we could not load BBDB." + (if (require 'bbdb nil t) + (string-to-number (substring bbdb-version 0 1)) + 0)) + (defvar eudc-bbdb-attributes-translation-alist '((name . lastname) - (email . net) + (email . (if (= (eudc-bbdb-get-major-version-number) 3) + mail + net)) (phone . phones)) "Alist mapping EUDC attribute names to BBDB names.") @@ -63,10 +72,13 @@ firstname lastname)) (company (cdr (assq 'company query))) - (net (cdr (assq 'net query))) + (mail (cdr (assq (if (= (eudc-bbdb-get-major-version-number) 3) + 'mail + 'net) + query))) (notes (cdr (assq 'notes query))) (phone (cdr (assq 'phone query)))) - (list name company net notes phone))) + (list name company mail notes phone))) (defun eudc-bbdb-filter-non-matching-record (record) @@ -80,7 +92,10 @@ (case-fold-search t) bbdb-val) (or (and (memq attr '(firstname lastname aka company phones - addresses net)) + addresses + (if (= (eudc-bbdb-get-major-version-number) 3) + mail + net))) (progn (setq bbdb-val (eval (list (intern (concat "bbdb-record-" @@ -151,7 +166,11 @@ The record is filtered according to `eudc-bbdb-current-return-attributes'" (require 'bbdb) (let ((attrs (or eudc-bbdb-current-return-attributes - '(firstname lastname aka company phones addresses net notes))) + '(firstname lastname aka company phones addresses + (if (= (eudc-bbdb-get-major-version-number) 3) + mail + net) + notes))) attr eudc-rec val) @@ -163,7 +182,11 @@ The record is filtered according to `eudc-bbdb-current-return-attributes'" (setq val (eudc-bbdb-extract-phones record))) ((eq attr 'addresses) (setq val (eudc-bbdb-extract-addresses record))) - ((memq attr '(firstname lastname aka company net notes)) + ((memq attr '(firstname lastname aka company + (if (= (eudc-bbdb-get-major-version-number) 3) + mail + net) + notes)) (setq val (eval (list (intern (concat "bbdb-record-" From debbugs-submit-bounces@debbugs.gnu.org Sun Jan 25 18:50:09 2015 Received: (at 19678) by debbugs.gnu.org; 25 Jan 2015 23:50:09 +0000 Received: from localhost ([127.0.0.1]:56327 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFWwS-0004gY-9V for submit@debbugs.gnu.org; Sun, 25 Jan 2015 18:50:09 -0500 Received: from mail-ie0-f178.google.com ([209.85.223.178]:53318) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFWwO-0004fu-1o for 19678@debbugs.gnu.org; Sun, 25 Jan 2015 18:50:05 -0500 Received: by mail-ie0-f178.google.com with SMTP id rp18so5952376iec.9 for <19678@debbugs.gnu.org>; Sun, 25 Jan 2015 15:49:58 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:references:date:in-reply-to :message-id:user-agent:mime-version:content-type; bh=j2hQq1CvoauBAVSevvdDPbArXMp2TRhhoyI2UsENC+8=; b=XkfjAaZqt4VyG5VHz0bFfJJmQ3XWrfiIbtonCzqeWurduj9VurrzuPwNa/AvK5u/30 uN20biadYuYfBv9jtidxGxUYONC8btWtvTLGcAdLlXdh2dbDyChTTefBbaapbfP2R8ws E6oT0SqTRaI0bLLlEv37RvqVDEB6vioDUgJyPc6rt/WQTM7PtJJZKr2WcRw5g6D8l6Nw /fAprlvysrjPSgPRswjcmeevlgSb06I6iQQMwmKK0t9hn0f2RfgVUELS3K+DGi30njvu dWZyXeKOFp/0FaDU0N45LEC55p2FhrGUGCP4AP5E5S+z0n1v3oBYdUdLw4Lhfj2fhNaE bm7w== X-Gm-Message-State: ALoCoQmiBorKwAmSBqqSFIWm5vgixp84zdcz4GUAxFoEoiCQGl4WCsb9x9+bPRhZc6pxPNGq/suh X-Received: by 10.50.25.166 with SMTP id d6mr13553896igg.41.1422229798331; Sun, 25 Jan 2015 15:49:58 -0800 (PST) Received: from hp-dv5t (69-165-165-189.dsl.teksavvy.com. [69.165.165.189]) by mx.google.com with ESMTPSA id v39sm5036701iov.17.2015.01.25.15.49.57 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Sun, 25 Jan 2015 15:49:57 -0800 (PST) From: Thomas Fitzsimmons To: Sergio Durigan Junior Subject: Re: bug#19678: [PATCH] EUDC does not support BBDB 3.x References: <87mw57hhrd.fsf@sergiodj.net> Date: Sun, 25 Jan 2015 18:49:56 -0500 In-Reply-To: <87mw57hhrd.fsf@sergiodj.net> (Sergio Durigan Junior's message of "Sat, 24 Jan 2015 18:37:58 -0500") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-Spam-Score: -0.7 (/) X-Debbugs-Envelope-To: 19678 Cc: 19678@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.7 (/) --=-=-= Content-Type: text/plain Sergio Durigan Junior writes: > This bug was opened due to a request made at: > > > > As explained at: > > > > With the release of BBDB 3.x, EUDC needs adjustments to properly work > with it. Actually, after some investigation, I noticed that the only > adjustment was to replace the occurrences of 'net' by 'mail' in the > code. The second link above contains a patch that does that, but Thomas > Fitzsimmons correctly noted (in the first link) that the patch breaks > compatibility with BBDB 2.x. > > The following patch is a first attempt to implement the support for both > BBDB 2.x and 3.x. Can you see if the attached patch works for you? It works for me, introduces minimal changes and allows for upgrading BBDB 2 to BBDB 3 without restarting Emacs. The defvar change in your patch concerns me because it will miss if the BBDB version changes after the variable is first assigned. I'm also going to experiment with eudc-export which also may need to be updated for BBDB 3 handling. Longer term I hope to work toward bundling BBDB in Emacs which would eliminate the need to support multiple versions. Thomas --=-=-= Content-Type: text/x-patch Content-Disposition: attachment; filename=0001-EUDC-Support-BBDB-3.patch >From 34dbaf966d2875c1f129f35c9866122f88e0ac16 Mon Sep 17 00:00:00 2001 From: Thomas Fitzsimmons Date: Sun, 25 Jan 2015 01:32:52 -0500 Subject: [PATCH 1/2] EUDC: Support BBDB >= 3 * net/eudcb-bbdb.el (eudc-bbdb-field): New function. (eudc-bbdb-filter-non-matching-record): Call eudc-bbdb-field. (eudc-bbdb-format-record-as-result): Likewise. --- lisp/ChangeLog | 6 ++++++ lisp/net/eudcb-bbdb.el | 16 ++++++++++++++-- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/lisp/ChangeLog b/lisp/ChangeLog index 70293af..e26e028 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog @@ -1,3 +1,9 @@ +2015-01-25 Thomas Fitzsimmons + + * net/eudcb-bbdb.el (eudc-bbdb-field): New function. + (eudc-bbdb-filter-non-matching-record): Call eudc-bbdb-field. + (eudc-bbdb-format-record-as-result): Likewise. + 2015-01-25 Stefan Monnier * emacs-lisp/cl-generic.el (cl--generic-no-next-method-function): New fun. diff --git a/lisp/net/eudcb-bbdb.el b/lisp/net/eudcb-bbdb.el index 0400e5b..7aed542 100644 --- a/lisp/net/eudcb-bbdb.el +++ b/lisp/net/eudcb-bbdb.el @@ -41,6 +41,16 @@ (defvar eudc-bbdb-current-query nil) (defvar eudc-bbdb-current-return-attributes nil) +(defun eudc-bbdb-field (field-symbol) + "Convert FIELD-SYMBOL so that it is recognized by the current BBDB version. +BBDB < 3 used `net'; BBDB >= 3 uses `mail'." + ;; This just-in-time translation permits upgrading from BBDB 2 to + ;; BBDB 3 without restarting Emacs. + (if (and (eq field-symbol 'net) + (version<= "3" bbdb-version)) + 'mail + field-symbol)) + (defvar eudc-bbdb-attributes-translation-alist '((name . lastname) (email . net) @@ -84,7 +94,9 @@ (progn (setq bbdb-val (eval (list (intern (concat "bbdb-record-" - (symbol-name attr))) + (symbol-name + (eudc-bbdb-field + attr)))) 'record))) (if (listp bbdb-val) (if eudc-bbdb-enable-substring-matches @@ -167,7 +179,7 @@ The record is filtered according to `eudc-bbdb-current-return-attributes'" (setq val (eval (list (intern (concat "bbdb-record-" - (symbol-name attr))) + (symbol-name (eudc-bbdb-field attr)))) 'record)))) (t (error "Unknown BBDB attribute"))) -- 1.8.1.4 --=-=-=-- From debbugs-submit-bounces@debbugs.gnu.org Mon Jan 26 14:02:44 2015 Received: (at 19678) by debbugs.gnu.org; 26 Jan 2015 19:02:44 +0000 Received: from localhost ([127.0.0.1]:57325 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFovr-0003cv-P7 for submit@debbugs.gnu.org; Mon, 26 Jan 2015 14:02:44 -0500 Received: from kwanyin.sergiodj.net ([176.31.208.32]:41887) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFovo-0003cd-Iy for 19678@debbugs.gnu.org; Mon, 26 Jan 2015 14:02:42 -0500 From: Sergio Durigan Junior To: Thomas Fitzsimmons Subject: Re: bug#19678: [PATCH] EUDC does not support BBDB 3.x References: <87mw57hhrd.fsf@sergiodj.net> X-URL: http://blog.sergiodj.net Date: Mon, 26 Jan 2015 14:02:36 -0500 In-Reply-To: (Thomas Fitzsimmons's message of "Sun, 25 Jan 2015 18:49:56 -0500") Message-ID: <87lhkpjrg3.fsf@sergiodj.net> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: -0.0 (/) X-Debbugs-Envelope-To: 19678 Cc: 19678@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.0 (/) On Sunday, January 25 2015, Thomas Fitzsimmons wrote: > Can you see if the attached patch works for you? It works for me, > introduces minimal changes and allows for upgrading BBDB 2 to BBDB 3 > without restarting Emacs. The defvar change in your patch concerns me > because it will miss if the BBDB version changes after the variable is > first assigned. Almost there... The patch doesn't work as-is because 'eudc-bbdb-current-return-attributes' is set to '(firstname lastname mail) for me when eudc-bbdb-format-record-as-result is called. It means that, in the while loop, it will try to call eq/memq against 'mail, and it fails. What I've done now is to call eudc-bbdb-field when we are composing the variables that will hold the BBDB attributes to match against. > I'm also going to experiment with eudc-export which also may need to be > updated for BBDB 3 handling. Nice. Let me know if you need some help. > Longer term I hope to work toward bundling BBDB in Emacs which would > eliminate the need to support multiple versions. That would be awesome. Again, I could help with the task. So, the following patch is the last version of what works for my case. I hope it is OK to apply. Cheers, -- Sergio GPG key ID: 0x65FC5E36 Please send encrypted e-mail if possible http://sergiodj.net/ diff --git a/lisp/ChangeLog b/lisp/ChangeLog index ff352a2..4bee79b 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog @@ -1,3 +1,11 @@ +2015-01-25 Thomas Fitzsimmons + Sergio Durigan Junior + + * net/eudcb-bbdb.el (eudc-bbdb-field): New function. + (eudc-bbdb-format-query): Call eudc-bbdb-field. + (eudc-bbdb-filter-non-matching-record): Likewise. + (eudc-bbdb-format-record-as-result): Likewise. + 2015-01-26 Stefan Monnier diff --git a/lisp/net/eudcb-bbdb.el b/lisp/net/eudcb-bbdb.el index 0400e5b..10c2c86 100644 --- a/lisp/net/eudcb-bbdb.el +++ b/lisp/net/eudcb-bbdb.el @@ -41,6 +41,16 @@ (defvar eudc-bbdb-current-query nil) (defvar eudc-bbdb-current-return-attributes nil) +(defun eudc-bbdb-field (field-symbol) + "Convert FIELD-SYMBOL so that it is recognized by the current BBDB version. +BBDB < 3 used `net'; BBDB >= 3 uses `mail'." + ;; This just-in-time translation permits upgrading from BBDB 2 to + ;; BBDB 3 without restarting Emacs. + (if (and (eq field-symbol 'net) + (version<= "3" bbdb-version)) + 'mail + field-symbol)) + (defvar eudc-bbdb-attributes-translation-alist '((name . lastname) (email . net) @@ -63,7 +73,7 @@ firstname lastname)) (company (cdr (assq 'company query))) - (net (cdr (assq 'net query))) + (net (cdr (assq (eudc-bbdb-field 'net) query))) (notes (cdr (assq 'notes query))) (phone (cdr (assq 'phone query)))) (list name company net notes phone))) @@ -79,12 +89,14 @@ (val (cdr condition)) (case-fold-search t) bbdb-val) - (or (and (memq attr '(firstname lastname aka company phones - addresses net)) + (or (and (memq attr `(firstname lastname aka company phones + addresses ,(eudc-bbdb-field 'net))) (progn (setq bbdb-val (eval (list (intern (concat "bbdb-record-" - (symbol-name attr))) + (symbol-name + (eudc-bbdb-field + attr)))) 'record))) (if (listp bbdb-val) (if eudc-bbdb-enable-substring-matches @@ -151,7 +163,8 @@ The record is filtered according to `eudc-bbdb-current-return-attributes'" (require 'bbdb) (let ((attrs (or eudc-bbdb-current-return-attributes - '(firstname lastname aka company phones addresses net notes))) + `(firstname lastname aka company phones addresses + ,(eudc-bbdb-field 'net) notes))) attr eudc-rec val) @@ -163,11 +176,12 @@ The record is filtered according to `eudc-bbdb-current-return-attributes'" (setq val (eudc-bbdb-extract-phones record))) ((eq attr 'addresses) (setq val (eudc-bbdb-extract-addresses record))) - ((memq attr '(firstname lastname aka company net notes)) + ((memq attr `(firstname lastname aka company + ,(eudc-bbdb-field 'net) notes)) (setq val (eval (list (intern (concat "bbdb-record-" - (symbol-name attr))) + (symbol-name (eudc-bbdb-field attr)))) 'record)))) (t (error "Unknown BBDB attribute"))) From debbugs-submit-bounces@debbugs.gnu.org Mon Jan 26 17:21:33 2015 Received: (at 19678) by debbugs.gnu.org; 26 Jan 2015 22:21:33 +0000 Received: from localhost ([127.0.0.1]:57361 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFs2H-0000PA-5P for submit@debbugs.gnu.org; Mon, 26 Jan 2015 17:21:33 -0500 Received: from mail-ig0-f177.google.com ([209.85.213.177]:36196) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFs2E-0000Os-IN for 19678@debbugs.gnu.org; Mon, 26 Jan 2015 17:21:31 -0500 Received: by mail-ig0-f177.google.com with SMTP id z20so1290056igj.4 for <19678@debbugs.gnu.org>; Mon, 26 Jan 2015 14:21:25 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:references:date:in-reply-to :message-id:user-agent:mime-version:content-type; bh=sCeDBeStt/Ztq/W0d/KSJoeflkIbImAOmFSFODJ7Rgw=; b=Vn58YqYOytOi0QQsoIrm/DFHZmx87DJLVac6nRDslEQAe++5Zp7JIozSIh0gUJgEO2 OCabO67M2PbgWeACYJ9q7cx1PflxAzgyCpZXqjkG23QjmUOhKEVgGfAeiUZblBT31Fye qsD20IJ9r+Urq5/GTMeJMdb7RTIENOjnI21cvIm2QHuiqthVSw4T2UafuC+INBwISmGP 8SgD1XyaqyK0tuDzUhK18VPDpnskkPRaFHWI5W6XI0m7QLiYX1u40uCCReYQ/nD/x6Kb WTEtbRXluYDUZkwWpTVambPJ+fNGI6DipPa5wr3VmxHKQ2aSIVya5XLHKtNRiYovTj3V MHMg== X-Gm-Message-State: ALoCoQnXG32zFRV9NDSXN2YAtrAfnj1dM66X0kJ9Hi3fXvIYiO13qtzG1RvahTfysWFz8s83sB1x X-Received: by 10.107.5.85 with SMTP id 82mr21516892iof.41.1422310884817; Mon, 26 Jan 2015 14:21:24 -0800 (PST) Received: from hp-dv5t (69-165-165-189.dsl.teksavvy.com. [69.165.165.189]) by mx.google.com with ESMTPSA id nv8sm6505828igb.6.2015.01.26.14.21.23 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Mon, 26 Jan 2015 14:21:24 -0800 (PST) From: Thomas Fitzsimmons To: Sergio Durigan Junior Subject: Re: bug#19678: [PATCH] EUDC does not support BBDB 3.x References: <87mw57hhrd.fsf@sergiodj.net> <87lhkpjrg3.fsf@sergiodj.net> Date: Mon, 26 Jan 2015 17:21:22 -0500 In-Reply-To: <87lhkpjrg3.fsf@sergiodj.net> (Sergio Durigan Junior's message of "Mon, 26 Jan 2015 14:02:36 -0500") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: -0.7 (/) X-Debbugs-Envelope-To: 19678 Cc: 19678@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.7 (/) Sergio Durigan Junior writes: > On Sunday, January 25 2015, Thomas Fitzsimmons wrote: > >> Can you see if the attached patch works for you? It works for me, >> introduces minimal changes and allows for upgrading BBDB 2 to BBDB 3 >> without restarting Emacs. The defvar change in your patch concerns me >> because it will miss if the BBDB version changes after the variable is >> first assigned. > > Almost there... The patch doesn't work as-is because > 'eudc-bbdb-current-return-attributes' is set to '(firstname lastname > mail) for me when eudc-bbdb-format-record-as-result is called. It means > that, in the while loop, it will try to call eq/memq against 'mail, and > it fails. Hmm, I thought those get converted; in any case, I wasn't seeing that problem. Do you have any customizations of the relevant variables? Do you have my latest EUDC/LDAP changes from master tip? Can we step back a bit and make sure we're doing the same tests? BBDB 2.x is tricky because it is provided by the distro, in my case Fedora. Let's focus on testing BBDB 3.x so that I can replicate the exact same issue that you're seeing. Can you revert our patch, then: 1) Checkout and build Emacs revision 03a20dc9519616359bfa1b77fd4b31e1963c8bd4 from git://git.savannah.gnu.org/emacs.git This revision has a bunch of my EUDC/LDAP updates in it. 2) Download http://download.savannah.gnu.org/releases/bbdb/bbdb-3.1.2.tar.gz 3) Untar and build the ELPA package: export EMACS=/src/emacs ./configure && make elpa unset EMACS 4) In the emacs src directory: mkdir test-home 5) HOME=`pwd`/test-home ./emacs -Q 6) M-x package-install-file bbdb-3.1.2.tar (the one built in step 3) 7) M-x bbdb-create Name: Test User Network Address: test@gnu.org 8) C-x s to save .bbdb 9) M-: (eval-after-load "message" '(define-key message-mode-map (kbd "TAB") 'eudc-expand-inline)) 10) M-: (setq debug-on-error 't) 11) C-x m 12) Tes[TAB] (no server, bbdb protocol) Without my patch, I get: Debugger entered--Lisp error: (void-function bbdb-record-net) (bbdb-record-net record) eval((bbdb-record-net record)) eudc-bbdb-format-record-as-result(["Test" "User" nil nil nil nil nil ("test@gnu.org") ((creation-date . "2015-01-26 21:56:26 +0000") (timestamp . "2015-01-26 21:56:26 +0000")) ["Test User" "User, Test" nil ("test@gnu.org") nil #]]) mapcar(eudc-bbdb-format-record-as-result (["Test" "User" nil nil nil nil nil ("test@gnu.org") ((creation-date . "2015-01-26 21:56:26 +0000") (timestamp . "2015-01-26 21:56:26 +0000")) ["Test User" "User, Test" nil ("test@gnu.org") nil #]])) eudc-bbdb-query-internal(((firstname . "Tes")) (firstname lastname net)) eudc-query(((firstname . "Tes")) (firstname lastname net)) eudc-expand-inline() funcall-interactively(eudc-expand-inline) call-interactively(eudc-expand-inline nil nil) command-execute(eudc-expand-inline) With my patch (0001-EUDC-Support-BBDB-3.patch), it works. eudc-query gets called with 'net, not 'mail. If that works for you, can you try to replicate the other error you're seeing when my patch is applied, in this same minimal environment, and paste the testing steps and the backtrace you get? Thanks, Thomas From debbugs-submit-bounces@debbugs.gnu.org Mon Jan 26 17:47:22 2015 Received: (at 19678) by debbugs.gnu.org; 26 Jan 2015 22:47:22 +0000 Received: from localhost ([127.0.0.1]:57365 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFsRF-00012R-Mx for submit@debbugs.gnu.org; Mon, 26 Jan 2015 17:47:22 -0500 Received: from kwanyin.sergiodj.net ([176.31.208.32]:42074) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFsRD-00012J-Og for 19678@debbugs.gnu.org; Mon, 26 Jan 2015 17:47:21 -0500 From: Sergio Durigan Junior To: Thomas Fitzsimmons Subject: Re: bug#19678: [PATCH] EUDC does not support BBDB 3.x References: <87mw57hhrd.fsf@sergiodj.net> <87lhkpjrg3.fsf@sergiodj.net> X-URL: http://blog.sergiodj.net Date: Mon, 26 Jan 2015 17:47:17 -0500 In-Reply-To: (Thomas Fitzsimmons's message of "Mon, 26 Jan 2015 17:21:22 -0500") Message-ID: <8761btjh1m.fsf@sergiodj.net> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: -0.0 (/) X-Debbugs-Envelope-To: 19678 Cc: 19678@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.0 (/) On Monday, January 26 2015, Thomas Fitzsimmons wrote: >> Almost there... The patch doesn't work as-is because >> 'eudc-bbdb-current-return-attributes' is set to '(firstname lastname >> mail) for me when eudc-bbdb-format-record-as-result is called. It means >> that, in the while loop, it will try to call eq/memq against 'mail, and >> it fails. > > Hmm, I thought those get converted; in any case, I wasn't seeing that > problem. Do you have any customizations of the relevant variables? Do > you have my latest EUDC/LDAP changes from master tip? I am using git HEAD to test and develop, so I think I do have your changes. As for my customizations, I'm almost sure they're the reason for me seeing the errors. What I have here is: (eudc-protocol-set 'eudc-inline-expansion-format '("%s %s <%s>" firstname name mail) 'bbdb) (eudc-protocol-set 'eudc-inline-query-format '((mail) (firstname) (lastname) (firstname lastname) (aka)) 'bbdb) > Can we step back a bit and make sure we're doing the same tests? BBDB > 2.x is tricky because it is provided by the distro, in my case Fedora. > Let's focus on testing BBDB 3.x so that I can replicate the exact same > issue that you're seeing. > > Can you revert our patch, then: > > 1) Checkout and build Emacs revision > 03a20dc9519616359bfa1b77fd4b31e1963c8bd4 from > git://git.savannah.gnu.org/emacs.git > > This revision has a bunch of my EUDC/LDAP updates in it. I'm buying an even newer revision from git, FWIW. > 2) Download > http://download.savannah.gnu.org/releases/bbdb/bbdb-3.1.2.tar.gz > > 3) Untar and build the ELPA package: > export EMACS=/src/emacs > ./configure && make elpa > unset EMACS > > 4) In the emacs src directory: > mkdir test-home > > 5) HOME=`pwd`/test-home ./emacs -Q > > 6) M-x package-install-file > > bbdb-3.1.2.tar (the one built in step 3) > > 7) M-x bbdb-create > > Name: Test User > Network Address: test@gnu.org > > 8) C-x s > > to save .bbdb > > 9) M-: (eval-after-load "message" > '(define-key message-mode-map (kbd "TAB") 'eudc-expand-inline)) > > 10) M-: (setq debug-on-error 't) > > 11) C-x m > > 12) Tes[TAB] > > (no server, bbdb protocol) > > Without my patch, I get: > > Debugger entered--Lisp error: (void-function bbdb-record-net) > (bbdb-record-net record) > eval((bbdb-record-net record)) > > eudc-bbdb-format-record-as-result(["Test" "User" nil nil nil nil nil > ("test@gnu.org") ((creation-date . "2015-01-26 21:56:26 +0000") > (timestamp . "2015-01-26 21:56:26 +0000")) ["Test User" "User, Test" > nil ("test@gnu.org") nil #]]) > > mapcar(eudc-bbdb-format-record-as-result (["Test" "User" nil nil nil > nil nil ("test@gnu.org") ((creation-date . "2015-01-26 21:56:26 > +0000") (timestamp . "2015-01-26 21:56:26 +0000")) ["Test User" "User, > Test" nil ("test@gnu.org") nil #]])) > > eudc-bbdb-query-internal(((firstname . "Tes")) (firstname lastname > net)) > > eudc-query(((firstname . "Tes")) (firstname lastname net)) > eudc-expand-inline() > funcall-interactively(eudc-expand-inline) > call-interactively(eudc-expand-inline nil nil) > command-execute(eudc-expand-inline) > > With my patch (0001-EUDC-Support-BBDB-3.patch), it works. eudc-query > gets called with 'net, not 'mail. > > If that works for you, can you try to replicate the other error you're > seeing when my patch is applied, in this same minimal environment, and > paste the testing steps and the backtrace you get? If you insist, I can do this test later (unfortunately I will be very busy these next days). Meanwhile, if you could check that my configuration is what triggers the failure, I'd appreciate. Also, IMHO, the final patch makes sense to me, even if there are still other issues to be fix on EUDC. Thank you, -- Sergio GPG key ID: 0x65FC5E36 Please send encrypted e-mail if possible http://sergiodj.net/ From debbugs-submit-bounces@debbugs.gnu.org Mon Jan 26 21:10:37 2015 Received: (at 19678) by debbugs.gnu.org; 27 Jan 2015 02:10:37 +0000 Received: from localhost ([127.0.0.1]:57454 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFvbw-00067h-B1 for submit@debbugs.gnu.org; Mon, 26 Jan 2015 21:10:36 -0500 Received: from mail-ie0-f177.google.com ([209.85.223.177]:47259) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YFvbr-00067L-52 for 19678@debbugs.gnu.org; Mon, 26 Jan 2015 21:10:32 -0500 Received: by mail-ie0-f177.google.com with SMTP id vy18so12569213iec.8 for <19678@debbugs.gnu.org>; Mon, 26 Jan 2015 18:10:25 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:references:date:in-reply-to :message-id:user-agent:mime-version:content-type; bh=/pc2e078xZfZSEVEa7qg0Ef6ILeqjiq1AOCpujZJbqc=; b=TlO/CsOdbyZEJeA6YtOGTSAp/vx9EWZfy+HVOGa4UlfVIxdn0yt4RYMF3TCsP6Dpvj 2oLgMAXbMVsXVaxJCDGaJkkl2nFtc9FwhKxYxXyLxHWE09pRBSfFET1CiSaBBRVP1ikI 2/uGOPY2ph+sooxXT4DTlOZaaYylilNA/NX1WuyymcsjgXsTrSWYvYTP2l7YoHLYPMu1 QPTkB3vbSOdBQ0rGzMuXnGzaz8eYC/qTBKJvFXYu3wkSogwJeielGJYN7GSYTJtbMcQR aPbMp1O4uZTlaUKZROfnj+WSsphQbhTWy6OhPFJ7qkmoROCA/K2O5/QK8NADWajXb/CP EJSw== X-Gm-Message-State: ALoCoQleiJAARgXC3c8gJ/Ww8KT5UDB6B34RMLqqMJRGqvS0PaSuJ6VfJHIJFoK2B7C2XrGJhlj6 X-Received: by 10.42.169.197 with SMTP id c5mr501720icz.72.1422324625368; Mon, 26 Jan 2015 18:10:25 -0800 (PST) Received: from hp-dv5t (69-165-165-189.dsl.teksavvy.com. [69.165.165.189]) by mx.google.com with ESMTPSA id 15sm6935478ioq.8.2015.01.26.18.10.24 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Mon, 26 Jan 2015 18:10:24 -0800 (PST) From: Thomas Fitzsimmons To: Sergio Durigan Junior Subject: Re: bug#19678: [PATCH] EUDC does not support BBDB 3.x References: <87mw57hhrd.fsf@sergiodj.net> <87lhkpjrg3.fsf@sergiodj.net> <8761btjh1m.fsf@sergiodj.net> Date: Mon, 26 Jan 2015 21:10:23 -0500 In-Reply-To: <8761btjh1m.fsf@sergiodj.net> (Sergio Durigan Junior's message of "Mon, 26 Jan 2015 17:47:17 -0500") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: -0.7 (/) X-Debbugs-Envelope-To: 19678 Cc: 19678@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.7 (/) Sergio Durigan Junior writes: > On Monday, January 26 2015, Thomas Fitzsimmons wrote: > >>> Almost there... The patch doesn't work as-is because >>> 'eudc-bbdb-current-return-attributes' is set to '(firstname lastname >>> mail) for me when eudc-bbdb-format-record-as-result is called. It means >>> that, in the while loop, it will try to call eq/memq against 'mail, and >>> it fails. >> >> Hmm, I thought those get converted; in any case, I wasn't seeing that >> problem. Do you have any customizations of the relevant variables? Do >> you have my latest EUDC/LDAP changes from master tip? > > I am using git HEAD to test and develop, so I think I do have your > changes. > > As for my customizations, I'm almost sure they're the reason for me > seeing the errors. What I have here is: > > (eudc-protocol-set 'eudc-inline-expansion-format '("%s %s <%s>" firstname name mail) > 'bbdb) OK, yes, that explains why you're seeing an error and I'm not. Though I can understand why you're doing it this way (see below), I think calling eudc-protocol-set directly in this case is a misconfiguration; it bypasses the "defcustom" way of doing things. Prior to my patches, the way this was supposed to work is that the user customized eudc-inline-expansion-format using generic EUDC symbols for the fields. In your case, this would have been: (customize-set-variable 'eudc-inline-expansion-format '("%s %s <%s>" firstname name email)) i.e., email not mail, and with no protocol specified. Then EUDC would translate those to the fields used by the specific backend. Then if you were using an LDAP server and a BBDB "server", you'd get the same results from both without having to figure out the "email" field for each of them. All that said, my patches fixed the default to be exactly what you're configuring. Now you can just remove that line and it'll do exactly what you want. > (eudc-protocol-set 'eudc-inline-query-format '((mail) > (firstname) > (lastname) > (firstname lastname) > (aka)) > 'bbdb) I also changed this default to be: '((email) (firstname) (firstname name)) The aka is where eudc-protocol-set seems to make most sense; after all, aka is BBDB-specific. But EUDC backends will just return no results for fields they don't know about, so it's actually safe to just put aka in eudc-inline-query-format via customization, without resorting to eudc-protocol-set. The LDAP backend will ignore it, the BBDB backend will interpret and attempt it: (customize-set-variable 'eudc-inline-query-format '((email) (firstname) (name) (firstname name) (aka)) The manual should maybe explicitly mention this, but then we'd have to expose all fields from all backends directly to the user for configuration, which may confuse things even more. In general I don't like this aspect of EUDC: how confusing it is to configure. It's made even worse by the fact that even when configured properly it can still fail because of bugs. I've tried to simplify the configuration by providing better defaults, and I'm also trying to fix the bugs. >> Can we step back a bit and make sure we're doing the same tests? BBDB >> 2.x is tricky because it is provided by the distro, in my case Fedora. >> Let's focus on testing BBDB 3.x so that I can replicate the exact same >> issue that you're seeing. >> >> Can you revert our patch, then: >> >> 1) Checkout and build Emacs revision >> 03a20dc9519616359bfa1b77fd4b31e1963c8bd4 from >> git://git.savannah.gnu.org/emacs.git >> >> This revision has a bunch of my EUDC/LDAP updates in it. > > I'm buying an even newer revision from git, FWIW. > >> 2) Download >> http://download.savannah.gnu.org/releases/bbdb/bbdb-3.1.2.tar.gz >> >> 3) Untar and build the ELPA package: >> export EMACS=/src/emacs >> ./configure && make elpa >> unset EMACS >> >> 4) In the emacs src directory: >> mkdir test-home >> >> 5) HOME=`pwd`/test-home ./emacs -Q >> >> 6) M-x package-install-file >> >> bbdb-3.1.2.tar (the one built in step 3) >> >> 7) M-x bbdb-create >> >> Name: Test User >> Network Address: test@gnu.org >> >> 8) C-x s >> >> to save .bbdb >> >> 9) M-: (eval-after-load "message" >> '(define-key message-mode-map (kbd "TAB") 'eudc-expand-inline)) >> >> 10) M-: (setq debug-on-error 't) >> >> 11) C-x m >> >> 12) Tes[TAB] >> >> (no server, bbdb protocol) >> >> Without my patch, I get: >> >> Debugger entered--Lisp error: (void-function bbdb-record-net) >> (bbdb-record-net record) >> eval((bbdb-record-net record)) >> >> eudc-bbdb-format-record-as-result(["Test" "User" nil nil nil nil nil >> ("test@gnu.org") ((creation-date . "2015-01-26 21:56:26 +0000") >> (timestamp . "2015-01-26 21:56:26 +0000")) ["Test User" "User, Test" >> nil ("test@gnu.org") nil #]]) >> >> mapcar(eudc-bbdb-format-record-as-result (["Test" "User" nil nil nil >> nil nil ("test@gnu.org") ((creation-date . "2015-01-26 21:56:26 >> +0000") (timestamp . "2015-01-26 21:56:26 +0000")) ["Test User" "User, >> Test" nil ("test@gnu.org") nil #]])) >> >> eudc-bbdb-query-internal(((firstname . "Tes")) (firstname lastname >> net)) >> >> eudc-query(((firstname . "Tes")) (firstname lastname net)) >> eudc-expand-inline() >> funcall-interactively(eudc-expand-inline) >> call-interactively(eudc-expand-inline nil nil) >> command-execute(eudc-expand-inline) >> >> With my patch (0001-EUDC-Support-BBDB-3.patch), it works. eudc-query >> gets called with 'net, not 'mail. >> >> If that works for you, can you try to replicate the other error you're >> seeing when my patch is applied, in this same minimal environment, and >> paste the testing steps and the backtrace you get? > > If you insist, I can do this test later (unfortunately I will be very > busy these next days). No need. > Meanwhile, if you could check that my configuration is what triggers > the failure, I'd appreciate. Done. > Also, IMHO, the final patch makes sense to me, even if there are still > other issues to be fix on EUDC. I'd like to keep the attribute symbols "private" in the backends, and encourage people to use the generic EUDC symbol names instead (plus hidden extras like aka, if they want) using the standard customization procedures. In which case the 'net stuff can stay as is in the BBDB backend. Can you try the configuration changes I've outlined above, along with 0001-EUDC-Support-BBDB-3.patch, and confirm that they work for you? I'll credit you in the ChangeLog too since we both worked on the patch. Let me know when your copyright assignment goes through, and I'll push it after that. Thanks, Thomas From debbugs-submit-bounces@debbugs.gnu.org Fri Feb 06 20:43:41 2015 Received: (at 19678) by debbugs.gnu.org; 7 Feb 2015 01:43:41 +0000 Received: from localhost ([127.0.0.1]:35962 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YJuQu-0000K0-Jo for submit@debbugs.gnu.org; Fri, 06 Feb 2015 20:43:41 -0500 Received: from kwanyin.sergiodj.net ([176.31.208.32]:56989) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YJuQq-0000Jm-Tg for 19678@debbugs.gnu.org; Fri, 06 Feb 2015 20:43:38 -0500 From: Sergio Durigan Junior To: Thomas Fitzsimmons Subject: Re: bug#19678: [PATCH] EUDC does not support BBDB 3.x References: <87mw57hhrd.fsf@sergiodj.net> <87lhkpjrg3.fsf@sergiodj.net> <8761btjh1m.fsf@sergiodj.net> X-URL: http://blog.sergiodj.net Date: Fri, 06 Feb 2015 20:43:32 -0500 In-Reply-To: (Thomas Fitzsimmons's message of "Mon, 26 Jan 2015 21:10:23 -0500") Message-ID: <87d25mcx8b.fsf@sergiodj.net> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: 0.0 (/) X-Debbugs-Envelope-To: 19678 Cc: 19678@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.0 (/) On Monday, January 26 2015, Thomas Fitzsimmons wrote: > Sergio Durigan Junior writes: > >> On Monday, January 26 2015, Thomas Fitzsimmons wrote: >> >>>> Almost there... The patch doesn't work as-is because >>>> 'eudc-bbdb-current-return-attributes' is set to '(firstname lastname >>>> mail) for me when eudc-bbdb-format-record-as-result is called. It means >>>> that, in the while loop, it will try to call eq/memq against 'mail, and >>>> it fails. >>> >>> Hmm, I thought those get converted; in any case, I wasn't seeing that >>> problem. Do you have any customizations of the relevant variables? Do >>> you have my latest EUDC/LDAP changes from master tip? >> >> I am using git HEAD to test and develop, so I think I do have your >> changes. >> >> As for my customizations, I'm almost sure they're the reason for me >> seeing the errors. What I have here is: >> >> (eudc-protocol-set 'eudc-inline-expansion-format '("%s %s <%s>" firstname name mail) >> 'bbdb) > > OK, yes, that explains why you're seeing an error and I'm not. > > Though I can understand why you're doing it this way (see below), I > think calling eudc-protocol-set directly in this case is a > misconfiguration; it bypasses the "defcustom" way of doing things. > Prior to my patches, the way this was supposed to work is that the user > customized eudc-inline-expansion-format using generic EUDC symbols for > the fields. In your case, this would have been: > > (customize-set-variable 'eudc-inline-expansion-format > '("%s %s <%s>" firstname name email)) > > i.e., email not mail, and with no protocol specified. Then EUDC would > translate those to the fields used by the specific backend. Then if you > were using an LDAP server and a BBDB "server", you'd get the same > results from both without having to figure out the "email" field for > each of them. > > All that said, my patches fixed the default to be exactly what you're > configuring. Now you can just remove that line and it'll do exactly > what you want. [ I'm back from the trip. Sorry about the long delay. I confess I had some things to mention/discuss about your reply before leaving, but now I forgot almost all of them. Oh, well... ] Right, thanks for explaining. It indeed makes sense to use EUDC's specific names, instead of BBDB/LDAP/whatever. I will change my configuration to reflect that. >> (eudc-protocol-set 'eudc-inline-query-format '((mail) >> (firstname) >> (lastname) >> (firstname lastname) >> (aka)) >> 'bbdb) > > I also changed this default to be: > > '((email) > (firstname) > (firstname name)) > > The aka is where eudc-protocol-set seems to make most sense; after all, > aka is BBDB-specific. But EUDC backends will just return no results for > fields they don't know about, so it's actually safe to just put aka in > eudc-inline-query-format via customization, without resorting to > eudc-protocol-set. The LDAP backend will ignore it, the BBDB backend > will interpret and attempt it: > > (customize-set-variable 'eudc-inline-query-format '((email) > (firstname) > (name) > (firstname name) > (aka)) > > The manual should maybe explicitly mention this, but then we'd have to > expose all fields from all backends directly to the user for > configuration, which may confuse things even more. > > In general I don't like this aspect of EUDC: how confusing it is to > configure. It's made even worse by the fact that even when configured > properly it can still fail because of bugs. I've tried to simplify the > configuration by providing better defaults, and I'm also trying to fix > the bugs. Thanks a lot for working on EUDC. I agree that it is frustrating to stumble upon bugs every time. >> Also, IMHO, the final patch makes sense to me, even if there are still >> other issues to be fix on EUDC. > > I'd like to keep the attribute symbols "private" in the backends, and > encourage people to use the generic EUDC symbol names instead (plus > hidden extras like aka, if they want) using the standard customization > procedures. In which case the 'net stuff can stay as is in the BBDB > backend. Can you try the configuration changes I've outlined above, > along with 0001-EUDC-Support-BBDB-3.patch, and confirm that they work > for you? > > I'll credit you in the ChangeLog too since we both worked on the patch. > Let me know when your copyright assignment goes through, and I'll push > it after that. All right, thanks a lot. I still haven't received feedback from the FSF, but I believe things are mostly OK since this is just an extension of my existing copyright assignment for GDB/binutils. That being said, I think that, if you want to go ahead and push the patch, it's fine. Otherwise, I will get back to you when I receive FSF's reply. Cheers, -- Sergio GPG key ID: 0x65FC5E36 Please send encrypted e-mail if possible http://sergiodj.net/ From debbugs-submit-bounces@debbugs.gnu.org Sun Mar 08 21:20:09 2015 Received: (at control) by debbugs.gnu.org; 9 Mar 2015 01:20:09 +0000 Received: from localhost ([127.0.0.1]:39903 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YUmMb-00044s-54 for submit@debbugs.gnu.org; Sun, 08 Mar 2015 21:20:09 -0400 Received: from mail-ie0-f175.google.com ([209.85.223.175]:36794) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YUmMY-00044H-Ng for control@debbugs.gnu.org; Sun, 08 Mar 2015 21:20:07 -0400 Received: by iecar1 with SMTP id ar1so3627726iec.3 for ; Sun, 08 Mar 2015 18:20:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:date:message-id:user-agent:mime-version :content-type; bh=yi/b5vl+L6xNjtm0ty+JnORUVJQMnbRz8hSvJ7lktHc=; b=d3fljbHvIqW6pp/MbrfIno/HjsCoB4LZhPU5sYhcXvKRqJZ5eby//fIDywiSpgIVfk zjdfbUHZEvyAgXrf4qE3tG8kkJtox3iAAX5d8phljX1BKvXCge4+xQLfYmmILi7MoVaK tRv2J3v6YY0GCcpEa1W1wAeuCauA+hLxqm2+dUXZBqR8KoJnV8rqCRJahWhBrFMj/MOQ pq29muU2vugPlsK0WW2JTyYI+fARR9V7ZR/xzUcfuOYcHdLyoriE63eAZ6uTlOPydXTG Xl78n92zeVXjK3Fql5ZQ1XJ4v+8zihXbMo6MNOE7XkUwAhCqsoYCMZejGH9vg+1wQjz6 gC9Q== X-Gm-Message-State: ALoCoQmbpCLWploOpe6U89jzlRYMhulSmm+QvcmCD/IFqezECqzdAAqu8l1y8O33AMq83tQrS9f8 X-Received: by 10.42.240.4 with SMTP id ky4mr20660668icb.70.1425864000780; Sun, 08 Mar 2015 18:20:00 -0700 (PDT) Received: from hp-dv5t (69-165-165-189.dsl.teksavvy.com. [69.165.165.189]) by mx.google.com with ESMTPSA id h7sm5114629ioi.5.2015.03.08.18.19.59 for (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Sun, 08 Mar 2015 18:19:59 -0700 (PDT) From: Thomas Fitzsimmons To: control@debbugs.gnu.org Date: Sun, 08 Mar 2015 21:19:58 -0400 Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: 1.3 (+) X-Spam-Report: Spam detection software, running on the system "debbugs.gnu.org", has identified this incoming email as possible spam. The original message has been attached to this so you can view it (if it isn't spam) or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: fixed 19678 25.1 thanks [...] Content analysis details: (1.3 points, 10.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low trust [209.85.223.175 listed in list.dnswl.org] 1.8 MISSING_SUBJECT Missing Subject: header 0.2 NO_SUBJECT Extra score for no subject X-Debbugs-Envelope-To: control X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: 1.3 (+) X-Spam-Report: Spam detection software, running on the system "debbugs.gnu.org", has identified this incoming email as possible spam. The original message has been attached to this so you can view it (if it isn't spam) or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: fixed 19678 25.1 thanks [...] Content analysis details: (1.3 points, 10.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low trust [209.85.223.175 listed in list.dnswl.org] 1.8 MISSING_SUBJECT Missing Subject: header 0.2 NO_SUBJECT Extra score for no subject fixed 19678 25.1 thanks From debbugs-submit-bounces@debbugs.gnu.org Sun Mar 08 21:22:30 2015 Received: (at 19678-done) by debbugs.gnu.org; 9 Mar 2015 01:22:30 +0000 Received: from localhost ([127.0.0.1]:39908 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YUmOr-00049C-ON for submit@debbugs.gnu.org; Sun, 08 Mar 2015 21:22:29 -0400 Received: from mail-ie0-f178.google.com ([209.85.223.178]:38530) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1YUmOq-00048y-I7 for 19678-done@debbugs.gnu.org; Sun, 08 Mar 2015 21:22:28 -0400 Received: by iecrl12 with SMTP id rl12so16280601iec.5 for <19678-done@debbugs.gnu.org>; Sun, 08 Mar 2015 18:22:23 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:subject:references:date:in-reply-to :message-id:user-agent:mime-version:content-type; bh=xtNZSRmT0AN26OFkpoZRv+AkVCXNcvOr7wmdnFM/6jk=; b=aHEhKo3REvNlPz5X2Vw4sj4bipgA9MjxPHwQQQU3oHPh2ztGxuFy9666Ump7odkhhm eMuH/CBNM/3wa0ea4L7jisy0etgZB/Lu6U4jiIfs4zYMWIZI76YcUB1ihNu5OHD3hB0N lR/rdAnX1fPZps9IOiWtt49YzQ94Pghk3OUnt1agDI8r5IYzU1AnuZgEbTvAYMZZmD52 RxNhEXj3h5Olj4Z4/4B/4rFSLwT68f6db+D+Uf5ljdJBTkYYTb0sA6WK8rqkCzNnBLZv FHkn6gWy5fwvFy39Wm/3ho/1BHejhLcWv9E+ZUapkWaujh1GxA+J09LOthhSfWSIUe6d jnyA== X-Gm-Message-State: ALoCoQmFtB4APE7IzBHiuTgF9CE4OLj6D4rREpSICNnzFNQKOxi5ZwapgEwKqqyPwUomb770rlYO X-Received: by 10.51.17.7 with SMTP id ga7mr27070923igd.42.1425864143074; Sun, 08 Mar 2015 18:22:23 -0700 (PDT) Received: from hp-dv5t (69-165-165-189.dsl.teksavvy.com. [69.165.165.189]) by mx.google.com with ESMTPSA id x10sm5338585igl.13.2015.03.08.18.22.22 for <19678-done@debbugs.gnu.org> (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Sun, 08 Mar 2015 18:22:22 -0700 (PDT) From: Thomas Fitzsimmons To: 19678-done@debbugs.gnu.org Subject: Re: bug#19678: [PATCH] EUDC does not support BBDB 3.x References: <87mw57hhrd.fsf@sergiodj.net> <87lhkpjrg3.fsf@sergiodj.net> <8761btjh1m.fsf@sergiodj.net> <87d25mcx8b.fsf@sergiodj.net> Date: Sun, 08 Mar 2015 21:22:21 -0400 In-Reply-To: <87d25mcx8b.fsf@sergiodj.net> (Sergio Durigan Junior's message of "Fri, 06 Feb 2015 20:43:32 -0500") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: -0.7 (/) X-Debbugs-Envelope-To: 19678-done X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -0.7 (/) Hi, I pushed our patch, with some extra version checks I discovered were needed during testing. Thanks for reporting and providing a patch, Thomas From unknown Mon Aug 18 19:32:10 2025 Received: (at fakecontrol) by fakecontrolmessage; To: internal_control@debbugs.gnu.org From: Debbugs Internal Request Subject: Internal Control Message-Id: bug archived. Date: Mon, 06 Apr 2015 11:24:04 +0000 User-Agent: Fakemail v42.6.9 # This is a fake control message. # # The action: # bug archived. thanks # This fakemail brought to you by your local debbugs # administrator