From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Dmitry Gutov Newsgroups: gmane.emacs.bugs Subject: bug#60100: 29.0.60; project-current is not resilient Date: Sun, 18 Dec 2022 02:41:05 +0200 Message-ID: <525c62db-7562-46e3-b58c-92d97e0a4322@yandex.ru> References: <86mt7o4hmx.fsf@mail.linkov.net> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="3214"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2 To: Juri Linkov , 60100@debbugs.gnu.org Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Sun Dec 18 01:42:23 2022 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1p6hl1-0000hV-6V for geb-bug-gnu-emacs@m.gmane-mx.org; Sun, 18 Dec 2022 01:42:23 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1p6hkj-0000S1-2q; Sat, 17 Dec 2022 19:42:05 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1p6hkh-0000Ql-IU for bug-gnu-emacs@gnu.org; Sat, 17 Dec 2022 19:42:03 -0500 Original-Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1p6hkg-0007CO-Lf for bug-gnu-emacs@gnu.org; Sat, 17 Dec 2022 19:42:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1p6hkg-0006My-Eb for bug-gnu-emacs@gnu.org; Sat, 17 Dec 2022 19:42:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Dmitry Gutov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 18 Dec 2022 00:42:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 60100 X-GNU-PR-Package: emacs Original-Received: via spool by 60100-submit@debbugs.gnu.org id=B60100.167132407424476 (code B ref 60100); Sun, 18 Dec 2022 00:42:02 +0000 Original-Received: (at 60100) by debbugs.gnu.org; 18 Dec 2022 00:41:14 +0000 Original-Received: from localhost ([127.0.0.1]:58907 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1p6hju-0006Mi-Hv for submit@debbugs.gnu.org; Sat, 17 Dec 2022 19:41:14 -0500 Original-Received: from mail-ed1-f48.google.com ([209.85.208.48]:46994) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1p6hjt-0006MY-3W for 60100@debbugs.gnu.org; Sat, 17 Dec 2022 19:41:13 -0500 Original-Received: by mail-ed1-f48.google.com with SMTP id c17so8382382edj.13 for <60100@debbugs.gnu.org>; Sat, 17 Dec 2022 16:41:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :sender:from:to:cc:subject:date:message-id:reply-to; bh=L8FfQ/OM+suRAmWHItOdGnekJR3i94asOQyvFkDrGd0=; b=GheJ93jUl9ZpuJR4LwWSKpftlMxSHZx6Otihnmqr8gExOkjd9D/erg5CFm5n6KbniE BZqZ+3fiBXFIadoQKTA4TvhDuBouQO02d1qsHGx7MKY36qA41eW4WZGDAdY55LielRaQ 4g6pH54CiEPlaXLpU6WG+3izzdvRmwUjAABjvqBvPG3sAweL2aZgsT98NjNtfkkO6VsA 6sZ/A8H0RjJ7ER1HpUw51fmTR35QeoumaSBNT8LBf1iVO2re2CHgl+pT9dnuFUrGebQU fVvsJUMCkRGvF/NktdtrKp98z630391J1NF2zLBGKKi9AtYaMPQJRI12ofv6kXwc0/yx k2tQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :sender:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=L8FfQ/OM+suRAmWHItOdGnekJR3i94asOQyvFkDrGd0=; b=tQKx5LBgwCOjXD5vEy+VCPFLxGDhWcSLCbuQBcXr+f+WTF4wipwumANY4ybQNZdKrq Pewy1dG0HchBd/WdStjMxNCslouA9YZzJlGCwc8/5Tt/Tt4X+vn+UOD+UxYE3m6BKGd3 ht1E8pQ4rUjREKQlDYl8Dj3Ei0kRFc1OY1cjBaN5jpzkz7zt4HUXO18u55lm9cq6ys7N WMHcVBH+/ojPN1fn+JgN7VOGEFd0vz6d9OQePXjnaCM9qDREMuMIsRAqzPubwgPwueQM hG/Uln3TOTZTgS49avL2TrDCKX3YkJwKtWWVsiuxfYH2R2QIWpWtGIeDuwNZ6ezIwSII BLJQ== X-Gm-Message-State: ANoB5pkB8a7J62e5IHs6bPNGJMrRw2kmmmmvUNh3StrKcPylmBeMfq2O 2EfGHS3nIZc0KfWY+aCoJbk= X-Google-Smtp-Source: AA0mqf54+6EuFCkX1377tlH9pffJ2ZHJkUAHI9DUOjl2TcsEJe79qxhSu2k69/x3xSc6yeXPkxQdig== X-Received: by 2002:a05:6402:24a3:b0:461:ed76:cb45 with SMTP id q35-20020a05640224a300b00461ed76cb45mr33426078eda.24.1671324067340; Sat, 17 Dec 2022 16:41:07 -0800 (PST) Original-Received: from [192.168.0.2] ([46.251.119.176]) by smtp.googlemail.com with ESMTPSA id k4-20020aa7c044000000b0046856c307d0sm2584433edo.5.2022.12.17.16.41.06 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 17 Dec 2022 16:41:06 -0800 (PST) Content-Language: en-US In-Reply-To: <86mt7o4hmx.fsf@mail.linkov.net> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list 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-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:251326 Archived-At: On 15/12/2022 21:00, Juri Linkov wrote: > Is this intentional that > > ``` > (project-current nil "/tmp/nonexistent/") > ``` > > raises the error > > Debugger entered--Lisp error: (file-missing "Opening directory" "No such file or directory" "/tmp/nonexistent") > > instead of silently returning nil. I would say this is expected. How could we avoid this? By adding a file-exists-p check somewhere inside, right? And failing silently if it returns nil. Right now the caller code needs to use that check. Or, more likely, figure out why they are passing in a directory that does not exist. > Because when called with > an existing non-project dir, it just returns nil: > > ``` > (project-current nil "/tmp/") => nil > ``` nil is a normal return value in this case: it means "no project here", that's a regular occasion. The callers who don't want to deal with nil should pass t as the first argument.