From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Nikolay Kudryavtsev Newsgroups: gmane.emacs.bugs Subject: bug#19452: 24.4; sql-connect fails in first invocation: "Attempt to set a constant symbol: nil" Date: Mon, 25 Jul 2016 23:12:47 +0300 Message-ID: References: NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------88BC4C2E2937FF7BF3E62BB3" X-Trace: ger.gmane.org 1469477667 16266 80.91.229.3 (25 Jul 2016 20:14:27 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 25 Jul 2016 20:14:27 +0000 (UTC) To: 19452@debbugs.gnu.org Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Jul 25 22:14:15 2016 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1bRmGT-0007E1-3v for geb-bug-gnu-emacs@m.gmane.org; Mon, 25 Jul 2016 22:14:13 +0200 Original-Received: from localhost ([::1]:34855 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bRmGS-0005tc-7Q for geb-bug-gnu-emacs@m.gmane.org; Mon, 25 Jul 2016 16:14:12 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:52786) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bRmGM-0005sU-M3 for bug-gnu-emacs@gnu.org; Mon, 25 Jul 2016 16:14:07 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bRmGI-000706-EO for bug-gnu-emacs@gnu.org; Mon, 25 Jul 2016 16:14:05 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:53110) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bRmGI-000702-9p for bug-gnu-emacs@gnu.org; Mon, 25 Jul 2016 16:14:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1bRmGH-0001Af-Ui for bug-gnu-emacs@gnu.org; Mon, 25 Jul 2016 16:14:01 -0400 X-Loop: help-debbugs@gnu.org In-Reply-To: Resent-From: Nikolay Kudryavtsev Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 25 Jul 2016 20:14:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 19452 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 19452-submit@debbugs.gnu.org id=B19452.14694775884425 (code B ref 19452); Mon, 25 Jul 2016 20:14:01 +0000 Original-Received: (at 19452) by debbugs.gnu.org; 25 Jul 2016 20:13:08 +0000 Original-Received: from localhost ([127.0.0.1]:37214 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bRmFQ-00019I-J7 for submit@debbugs.gnu.org; Mon, 25 Jul 2016 16:13:08 -0400 Original-Received: from mail-lf0-f45.google.com ([209.85.215.45]:34106) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bRmFO-00018l-Nh for 19452@debbugs.gnu.org; Mon, 25 Jul 2016 16:13:07 -0400 Original-Received: by mail-lf0-f45.google.com with SMTP id l69so136074557lfg.1 for <19452@debbugs.gnu.org>; Mon, 25 Jul 2016 13:13:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:subject:message-id:date:user-agent:mime-version; bh=WqvMPR8eEU+3zXqo93OIT7pP8IIziaZ093zaZEvoXzc=; b=Wps1g6nJtTEgx9fMNCi2e6wN3vNr65OK6XUEz5Y0CP99FPcn1svmQg5hGWZijX1A0Y uTITq2XSyPyxFt0lMDdue1yP7nOmmDaWyRGBzBGMW9QUDfkl1R1jT3IMkVB599WyTVa6 jMwyECr2nqZK+VIqeiaqUsVXVZpbkVeq/bByn2kLbdYxZsr2LUDK/PnGvA9rfWNDq41W 4cr9hVKQT3Ty9DwNCa7kpi7nHCUESfMNO7WPHUTk/WxZQ+xQgr+UFKJrfVcYmhx1zjMx wqzZkiR/UtCMAA/yKdZJE5ISrmkyJCro/rLfLDqkXwVI60Jxwbrd8huhh89goLL9HC4i 4hNA== 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:message-id:date:user-agent :mime-version; bh=WqvMPR8eEU+3zXqo93OIT7pP8IIziaZ093zaZEvoXzc=; b=OUM+aYRWuKL4l6KaIXoDcvzU7RMpNht5X6B4l2zp7wfeLKPwLdaorfF9E/Ot9O0Cjl 6D7ZDYpg2VVt0Wy9aqNY58qYzaZ9hEOrw6o3qbUG1Sc1DtNrxJnjL6t9BqtUXzsZBx+U NedDFJ3mavB7gREXqtCtfn7aiHK9fa/TxSFyjZ+bsvXlRwMHN2LByb7ZZqsThvIAmFiy xCzD0WoQr0U2beq3H1IyrB2tuon+3UVCiJxzHP5OaaORporyp2Og68+fdpqUtCSLisVj CxH2awwWXwnXCZ4L74FjnxjRN2BI1EyMXzbDLURAqWJc9CbvzY90A1x/E59dK3jEndFh X54Q== X-Gm-Message-State: AEkoouvc+SEjGA/2O15WyIMCaAIs+maMbucWYfIHBA7zMw1AhYN97y7/tWkke7bNjpTkGw== X-Received: by 10.25.213.20 with SMTP id m20mr9012166lfg.61.1469477580384; Mon, 25 Jul 2016 13:13:00 -0700 (PDT) Original-Received: from [192.168.10.82] (mail.mcruises.ru. [212.248.66.250]) by smtp.gmail.com with ESMTPSA id r190sm5841866lfg.49.2016.07.25.13.12.59 for <19452@debbugs.gnu.org> (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 25 Jul 2016 13:12:59 -0700 (PDT) X-Google-Original-From: Nikolay Kudryavtsev User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:121532 Archived-At: This is a multi-part message in MIME format. --------------88BC4C2E2937FF7BF3E62BB3 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Ok, this bug has been bugging me long enough. What happens is that there's a dynamic let block that looks like this: (eval `(let ((,param-var ',rem-params)) (sql-product-interactive ',sql-product ',new-name)))) Here's the code that sets param-var: (setq param-var (sql-get-product-feature sql-product :sqli-login nil t)) It uses global variable sql-product. The default sql-product is ansi and running the above for it results in nil. Running "(let ((nil ..." leads to this error. In the same function there is a block that sets it to a proper value based on the current connection, but it is executed after param-var is already set. That's why the second time works. I've attached a simple patch that just reorders code within sql-connect. -- Best Regards, Nikolay Kudryavtsev --------------88BC4C2E2937FF7BF3E62BB3 Content-Type: text/plain; charset=UTF-8; name="19452.patch" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="19452.patch" ZGlmZiAtLWdpdCBsaXNwL3Byb2dtb2Rlcy9zcWwuZWwgbGlzcC9wcm9nbW9kZXMvc3FsLmVs DQppbmRleCBmZDU5ZjQ2Li5hNmU3ZWM0IDEwMDY0NA0KLS0tIGxpc3AvcHJvZ21vZGVzL3Nx bC5lbA0KKysrIGxpc3AvcHJvZ21vZGVzL3NxbC5lbA0KQEAgLTQwNDAsNiArNDA0MCwxMiBA QCBzcWwtY29ubmVjdA0KICAgICAgICAgICAoaWYgY29ubmVjdC1zZXQNCiAgICAgICAgICAg ICAgIDs7IFNldCB0aGUgZGVzaXJlZCBwYXJhbWV0ZXJzDQogICAgICAgICAgICAgICAobGV0 IChwYXJhbS12YXIgbG9naW4tcGFyYW1zIHNldC1wYXJhbXMgcmVtLXBhcmFtcykNCisgICAg ICAgICAgICAgICAgOzsgU2V0IHRoZSBwYXJhbWV0ZXJzIGFuZCBzdGFydCB0aGUgaW50ZXJh Y3RpdmUgc2Vzc2lvbg0KKyAgICAgICAgICAgICAgICAobWFwYw0KKyAgICAgICAgICAgICAg ICAgIycobGFtYmRhICh2dikNCisgICAgICAgICAgICAgICAgICAgICAoc2V0LWRlZmF1bHQg KGNhciB2dikgKGV2YWwgKGNhZHIgdnYpKSkpDQorICAgICAgICAgICAgICAgICAoY2RyIGNv bm5lY3Qtc2V0KSkNCisgICAgICAgICAgICAgICAgKHNldHEtZGVmYXVsdCBzcWwtY29ubmVj dGlvbiBjb25uZWN0aW9uKQ0KDQogICAgICAgICAgICAgICAgIDs7IDpzcWxpLWxvZ2luIHBh cmFtcyB2YXJpYWJsZQ0KICAgICAgICAgICAgICAgICAoc2V0cSBwYXJhbS12YXINCkBAIC00 MDY5LDEzICs0MDc1LDYgQEAgc3FsLWNvbm5lY3QNCiAgICAgICAgICAgICAgICAgICAgICAg ICAgICAodW5sZXNzIChtZW1iZXIgdG9rZW4gc2V0LXBhcmFtcykNCiAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIChpZiBwbGlzdCAoY29ucyB0b2tlbiBwbGlzdCkgdG9rZW4pKSkp KQ0KDQotICAgICAgICAgICAgICAgIDs7IFNldCB0aGUgcGFyYW1ldGVycyBhbmQgc3RhcnQg dGhlIGludGVyYWN0aXZlIHNlc3Npb24NCi0gICAgICAgICAgICAgICAgKG1hcGMNCi0gICAg ICAgICAgICAgICAgICMnKGxhbWJkYSAodnYpDQotICAgICAgICAgICAgICAgICAgICAgKHNl dC1kZWZhdWx0IChjYXIgdnYpIChldmFsIChjYWRyIHZ2KSkpKQ0KLSAgICAgICAgICAgICAg ICAgKGNkciBjb25uZWN0LXNldCkpDQotICAgICAgICAgICAgICAgIChzZXRxLWRlZmF1bHQg c3FsLWNvbm5lY3Rpb24gY29ubmVjdGlvbikNCi0NCiAgICAgICAgICAgICAgICAgOzsgU3Rh cnQgdGhlIFNRTGkgc2Vzc2lvbiB3aXRoIHJldmlzZWQgbGlzdCBvZiBsb2dpbiBwYXJhbWV0 ZXJzDQogICAgICAgICAgICAgICAgIChldmFsIGAobGV0ICgoLHBhcmFtLXZhciAnLHJlbS1w YXJhbXMpKQ0KICAgICAgICAgICAgICAgICAgICAgICAgICAoc3FsLXByb2R1Y3QtaW50ZXJh Y3RpdmUgJyxzcWwtcHJvZHVjdCAnLG5ldy1uYW1lKSkpKQ0K --------------88BC4C2E2937FF7BF3E62BB3--