From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Noam Postavsky Newsgroups: gmane.emacs.bugs Subject: bug#32129: 26.1; Bug in add-dir-local-variable when used with eval, progn and setenv Date: Fri, 13 Jul 2018 05:26:23 -0400 Message-ID: <87tvp3a3ls.fsf@gmail.com> References: <31BA42EC-58BC-4C69-9E04-E464AF41540D@luisgerhorst.de> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: blaine.gmane.org 1531473911 6386 195.159.176.226 (13 Jul 2018 09:25:11 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Fri, 13 Jul 2018 09:25:11 +0000 (UTC) User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) Cc: 32129@debbugs.gnu.org To: Luis Gerhorst Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Fri Jul 13 11:25:07 2018 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1fduK2-0001WC-OA for geb-bug-gnu-emacs@m.gmane.org; Fri, 13 Jul 2018 11:25:06 +0200 Original-Received: from localhost ([::1]:36201 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fduM8-0008I4-4C for geb-bug-gnu-emacs@m.gmane.org; Fri, 13 Jul 2018 05:27:16 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:53059) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fduLz-0008GY-OR for bug-gnu-emacs@gnu.org; Fri, 13 Jul 2018 05:27:08 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fduLu-00019U-Q7 for bug-gnu-emacs@gnu.org; Fri, 13 Jul 2018 05:27:07 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:49804) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fduLu-00019K-Ky for bug-gnu-emacs@gnu.org; Fri, 13 Jul 2018 05:27:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1fduLu-0002fe-En for bug-gnu-emacs@gnu.org; Fri, 13 Jul 2018 05:27:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Noam Postavsky Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 13 Jul 2018 09:27:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 32129 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 32129-submit@debbugs.gnu.org id=B32129.153147399210211 (code B ref 32129); Fri, 13 Jul 2018 09:27:02 +0000 Original-Received: (at 32129) by debbugs.gnu.org; 13 Jul 2018 09:26:32 +0000 Original-Received: from localhost ([127.0.0.1]:57695 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1fduLQ-0002eX-LJ for submit@debbugs.gnu.org; Fri, 13 Jul 2018 05:26:32 -0400 Original-Received: from mail-it0-f47.google.com ([209.85.214.47]:36716) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1fduLO-0002eG-KH; Fri, 13 Jul 2018 05:26:31 -0400 Original-Received: by mail-it0-f47.google.com with SMTP id j185-v6so10645430ite.1; Fri, 13 Jul 2018 02:26:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:references:date:in-reply-to:message-id :user-agent:mime-version; bh=jzEvHy3bHinBJkHNGPqCoJp1wO0obK9IOnImmgjfKxg=; b=PGeKsjkDj/hgVQngHYUdfXy6yd2JrgrPzyCe94mYNJVo6oDS9Cl0hzCCq7ZxjN960v JNG6OoeCikTkDUEDaaoKEhCY7sTmz8JErlDiz1YryZ97SiAfcYHdT2dDZvKhmZHlgIgf XyKI2xvi20HxSbIL/+5sU+iBA8dIt6Bxtce42N12lwA4+z0ucyd/uD2YpyFjI+0lzYI3 W9NxkBN6b979+e5WbH9offHuB4nXPWyvPIMcu8TuDmw7pMjRGYKv/cVnlLXwONZ9U+WW dZtHSWshn08MmOOPCbw9/ZyBmSD0u2W1mWz0runGjKMjRkzOZMMmB1LUKmpsAags9YFh yUgQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:references:date:in-reply-to :message-id:user-agent:mime-version; bh=jzEvHy3bHinBJkHNGPqCoJp1wO0obK9IOnImmgjfKxg=; b=OSOfvyttwuqd9AW4rWYJU2w61gCleEhbg6CvMRclLmjiifjIxDX48+kmCjOgvFA5TC dF0F+a0rcD6GFhlqkozDOSg8g7O9KpKecwkNG8GSj1VlTYOZfwRNOKwiq/VmqefhcYbS iF1okt5q0cgLCEj08Hu7wkaIcnhNmhytfDE7jx26nwAEcz3Nf47r5dXQQjyV3+0In8J7 rMHWxDJumkNfLamTAlLPq91IYRkQYNNrgbyle1pRSKoS5i2dnkys0Jcww+52QAXcNqg7 FV9HkMpCv6gMGurIYjVwCNgNasAC4DXs1pJGc7XBVjJPrioXC7e9cyDQL5IkxyzNY7IA 2APg== X-Gm-Message-State: AOUpUlGzXUgWLfKsE1gIOA7UYbXiqTBuKZ4SHI2u82W04SJfM3B1k153 TcYPyURz0zT+k9jBcIn2avCkaQ== X-Google-Smtp-Source: AAOMgpfW7ILzP+fdCx/EZVD8vUi16P7QEYXXdImQQZ6ABQRveePQ0qf3e5CuBdl09Gk1VI7jLOaoQg== X-Received: by 2002:a24:73c1:: with SMTP id y184-v6mr4286107itb.31.1531473984990; Fri, 13 Jul 2018 02:26:24 -0700 (PDT) Original-Received: from zebian (cbl-45-2-119-34.yyz.frontiernetworks.ca. [45.2.119.34]) by smtp.googlemail.com with ESMTPSA id t67-v6sm2369657ita.3.2018.07.13.02.26.23 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 13 Jul 2018 02:26:24 -0700 (PDT) In-Reply-To: <31BA42EC-58BC-4C69-9E04-E464AF41540D@luisgerhorst.de> (Luis Gerhorst's message of "Thu, 12 Jul 2018 01:18:19 +0200") 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:148484 Archived-At: tags 32129 notabug close 32129 quit Luis Gerhorst writes: > I'm sorry for the noise, I just retried it and it worked, TEST was > set. So the generated .dir-locals.el does the right thing. Right, I'm pretty sure you really do want setq-local though. > Anyway: Shouldn't the .dir-locals.el look different? This is of course > just a question of preference, if it makes the code more elegant or > something I would accept that it's better to leave it as it is. Yeah, it currently just does `pp' for the whole structure. We would need to special case the top level printing of values I think. It might be interesting to have a pretty printer that's able to take a sort of "descriptor" of printing structure, to handle this sort of thing more generally. But I'm going to go ahead and close this report as notabug, since special casing the dir-local printing is not worth the trouble.