From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Tim Cross Newsgroups: gmane.emacs.devel Subject: Re: [External] : Re: Default custom file was: Re: Propose to add setup-wizard.el to ELPA Date: Sat, 08 Jan 2022 11:29:06 +1100 Message-ID: <871r1jm4hf.fsf@gmail.com> References: <740A136F-8710-4F4C-BFC1-A3DB418447F4@gmail.com> <87r19nxx7x.fsf@gmail.com> <878rvv9esx.fsf@yahoo.com> <87fsq28x4l.fsf@yahoo.com> <87bl0q8vfa.fsf@yahoo.com> <83pmp69vsu.fsf@gnu.org> <8735m17l8c.fsf@yahoo.com> <875yqx5nub.fsf@yahoo.com> <83lezt8cm6.fsf@gnu.org> <871r1k38ym.fsf@gmail.com> <87ee5kmm6t.fsf@gmail.com> <87a6g8m1n1.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="21121"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: mu4e 1.7.5; emacs 28.0.90 Cc: "emacs-devel@gnu.org" To: Drew Adams Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Sat Jan 08 01:56:30 2022 Return-path: Envelope-to: ged-emacs-devel@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 1n6021-0005EE-E0 for ged-emacs-devel@m.gmane-mx.org; Sat, 08 Jan 2022 01:56:29 +0100 Original-Received: from localhost ([::1]:39130 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1n6020-0000MH-39 for ged-emacs-devel@m.gmane-mx.org; Fri, 07 Jan 2022 19:56:28 -0500 Original-Received: from eggs.gnu.org ([209.51.188.92]:42146) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1n600R-0007Qw-C6 for emacs-devel@gnu.org; Fri, 07 Jan 2022 19:54:51 -0500 Original-Received: from [2607:f8b0:4864:20::630] (port=35516 helo=mail-pl1-x630.google.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1n600L-0000po-Km for emacs-devel@gnu.org; Fri, 07 Jan 2022 19:54:51 -0500 Original-Received: by mail-pl1-x630.google.com with SMTP id n16so6322702plc.2 for ; Fri, 07 Jan 2022 16:54:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=references:user-agent:from:to:cc:subject:date:in-reply-to :message-id:mime-version; bh=l43VuYeScGawcvKALPPrS7EOjfcw2hARIqglyYYDzMw=; b=SPUEjr77bWeIdrMrsNAR0p28DoCP1SjUKE3OPjjcGQTcAUxpdmniVE5P3iOTd2c8+U 5/gPN3VmvcS+S86Py/3Q9SvNDnP4Qpf6IXHgFbh8uWQYSP7SMncydXFcdg3SQPsomDMZ oCq3g6G68VLn4kRDYs/DVLmVr5E8GV4TeyZpUOi2vibqXM0nBqrXTsL43wxsdg/sq4br Qy9gx4+Vt8R3HMiLtz1fqc8GFdIOqxzywOYt6Asf+JdUY5UaomcZPmIoQ82cCFGQHwv/ krajuofmYRbCjL22L/cQdGUluds0296vzyqdE8lEMUYcuc0Y+OPhdmkaBgb33uhhePFK zJBw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:references:user-agent:from:to:cc:subject:date :in-reply-to:message-id:mime-version; bh=l43VuYeScGawcvKALPPrS7EOjfcw2hARIqglyYYDzMw=; b=wZVERRZRBGkVXhTkqKXwb6VuHQMqdtxUNLPYnOcoEQAs6po1u5bAXUpi+FbxU2KKtS jOC0Ca652kHn/napergxxAGGMacmve0JZfoc9Uv/290eYkfEKrcBTtGBnG4G5dAmg2JS Zs0Cfco2nBvkQvViZ39AiIaDyMRoI/j9fm8Cynp1V4ExVUoD1CHIChA0sYLgiveLYU8f Bv60YxZwnmUhEGXfjCIoc7hce4xjPHOVtwbZyf4HFFdXkwZqpyH5+TrnIf/J1URcpoK1 XpsLCHbL3lXGcnj8YU2sQD7fmiCCA27yF8Ya5PiwThGpFx6lRd83CI+twK+UJzQoySys 63Bg== X-Gm-Message-State: AOAM531e6/1cw+4hPkxhPkpo+q/O8e9FncGbIX7gw3sdCLBgCI02mLwX Wc6K8nKmoNDM3Lw36EAc1p9148gItT0= X-Google-Smtp-Source: ABdhPJx+XQvZwkgJE5PTfOh8A7elZBg+aCv6S2a4NL5IuAPJdSh824CKn0mRL34GP9aq2nY2E6dmlQ== X-Received: by 2002:a17:90a:77c6:: with SMTP id e6mr18204290pjs.41.1641603280753; Fri, 07 Jan 2022 16:54:40 -0800 (PST) Original-Received: from dingbat ([124.149.107.194]) by smtp.gmail.com with ESMTPSA id hk8sm148161pjb.52.2022.01.07.16.54.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 07 Jan 2022 16:54:40 -0800 (PST) In-reply-to: X-Host-Lookup-Failed: Reverse DNS lookup failed for 2607:f8b0:4864:20::630 (failed) Received-SPF: pass client-ip=2607:f8b0:4864:20::630; envelope-from=theophilusx@gmail.com; helo=mail-pl1-x630.google.com X-Spam_score_int: -12 X-Spam_score: -1.3 X-Spam_bar: - X-Spam_report: (-1.3 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RDNS_NONE=0.793, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:284436 Archived-At: Drew Adams writes: >> From your response, there are a couple of additional >> points I'd like to make to clarify some things. >> >> The reason I don't think just setting the custom file >> to some value really covers the full scope of the >> change is because in addition to that change, it will >> also be necessary to add code to make emacs load that file. > > I assume you mean load that file _automatically_, > since users can (and do) already load it in their > init files. > > Automatically loading `custom-file', if it isn't > loaded by the init file (and any code that file > loads etc.), is not a hard requirement. > > It's something that could easily be done, but it's > not _required_ to advance the aim of getting more > users aware of and using `custom-file'. > >> This means either the timing of loading that file would then >> be up to Emacs or we would have to add some other switch to disable >> automatic loading to restore user control over loading that file. > > Yes. But I think that can be as simple as what > I suggested: > > 1. Automatically load `custom-file' (provided it's > not the same as the init file) immediately after > the init file is loaded. > > (We already automatically load other files if > they exist - site-lisp.el etc.) > > 2. Provide users with a way to inhibit automatic > loading if they need/want to do that. > > (Those users who want to do everything in their > init file are not involved in this - they'd > just set `custom-file' to their init file.) > >> So already the 'simple' change proposal has added additional complexity >> (albeit small). > > As I say, automatic loading isn't a requirement. > It's a feature. And with #2 it's optional. > > (One way to realize #2 would be with a user > option. We could discuss its default value.) > >> There could be other corner cases I've not thought of as >> well, especially once we add a new 'toggle' for the loading behaviour. > > If we're really worried about that, then we > just don't provide any such automatic loading. > > I don't expect problems, but automatic loading > isn't a requirement. Nothing in the general > aim and proposed solutions requires it. > > Without it, users would just be responsible for > loading `custom-file' - like now. Not a big > deal, but I think it might help users to provide > it (new users especially). > >> The change management aspects I referred to are perhaps a little subtle >> and are certainly hard to quantify. However, it is often way too easy to >> underestimate the impact of such change and identify what needs to be >> done to mitigate it. This impact can be especially hard to recognise >> when you are invested in the change. > > I don't disagree with that general point. > > I don't foresee any complications, but I'm > _not_ really invested in Emacs providing the > ability to automatically load `custom-file'. > >> Things which need to be considered >> (some of which have been mentioned) include >> >> - dealing with impact on existing users >> - updating documentation, including manuals, howtos, faqs etc >> - managing the confusion that will arise due to the amount of existing >> and easily found information out there (stack overflow, reddit, wikki, >> blogs, books etc) which will be out of date and will likely cause more >> confusion. > > Sure. > >> Just dealing with the first one will likely result in the final solution >> being more complex than simply setting a default custom file value, >> which in turn will make the other points more substantial to deal with. > > I don't think so, but I can't prove you're wrong. > > Suppose we don't offer automatic loading. > If you don't load `custom-file' (from your > init file or in some other way) then it doesn't > get loaded. End of story. > > Or suppose we offer it, but by way of a user > option that by default is off (no autoload). > > IOW, no change from what we do now, in this > respect (no autoloading of `custom-file'). > > In that case, the change is just to default > `custom-file' to a standard location, not to > nil. Now reread your paragraph of things that > need to be considered. Not a big deal in this > case, right? > > I wouldn't be completely happy with that > solution, but it would still be an improvement. > > As I said, it would even be a (small) > improvement if Emacs would just come out and > recommend to users to use `custom-file', > instead of just warning them, in the init-file > template-comment, not to edit the inserted > generated code. > > I'd hope that we go further than that, but > even that would help. > >> The above are some of the reasons I think it may be misleading to >> characterise the proposal as something simple. > > "The proposal" is really a set/hierarchy of > proposals, from trivially simple, with little > effect (and I hope little controversy), to > something that includes possibly automatically > loading `custom-file'. > > I hope you'll agree that the mere change to > defaulting `custom-file' to a file name isn't > complicated in its implications, and it should > not be controversial. > > All of what would happen in that case already > happens, if a user sets `custom-file' to a > file name. If no one loads that file, or if > that file remains nonexistent or empty, we're > in no-op land. > > Even users who only want to use their init > file for customizations wouldn't be impacted. > No-op. > >> However, I would be in support if I thought >> this was an actual problem needing to be addressed. > > Maybe think of it as what we have now: offering > `custom-file', but just making use of it a bit > more visible and likely. Instead of thinking > "problem" and "solution", maybe just think that > it might help more users to take advantage of > `custom-file'. > >> TO me, it really does feel more like a solution in search of a problem >> or at the very least, a change which will result in non-trivial effort >> (at various levels) when there is little evidence it is really required. > > I understand that you feel that. I think your > fears are unnecessary. Take out the "automatic > loading" feature from your consideration, and > see how fearful you still are. Hi Drew, we seem to be mis-communicating here or I'm just not being clear enough. One last attempt and then I'll leave it alone. My understanding of your suggestion is to set custom-file to some default location rather than nil e.g. .emacs.d/custom.el. You suggest that is all which is required and that no automatic loading would need to be added. I reject this as automatic loading would be absolutely necessary to retain existing behaviour. If all we do is set custom-file to a default location, custom will stop working between sessions because nothing will load those settings. This significantly changes behaviour. To keep the same behaviour, either all users who now just use custom with custom-file set to nil would need to add a load statement OR emacs will have to automatically load that file. Requiring users add a load statement to their initialisation file to ensure custom settings work across sessions is a bad design and would impact too many existing users. If we change the default setting of custom-file from nil to a default file location, there will need to be code added to Emacs to load that file at some point in the initialisation step. If we then also want to retain the user's ability to control when this file is loaded, we will also require a toggle to turn off that loading and allow the user to do it. Claiming that all you are proposing is a change to the default value for custom-file is inaccurate if you also want to claim no change in behaviour.