missing "/" in value of user-emacs-directory when using --init-directory flag

missing "/" in value of user-emacs-directory when using --init-directory flag

[Yilkal Argaw]

In my configs I use (concat user-emacs-directory "somedirectory or
file") to refer to files with respect to emacs. When loading emacs
with the option --init-directory this
this string is not separated by a backslash since the value
user-emacs-directory does not have a backslash on the end of it. This
might break previous configs so it should be standardized to contain
the same string value in both cases.

Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Yilkal Argaw]

> In my configs I use (concat user-emacs-directory "somedirectory or
> file") to refer to files with respect to emacs. When loading emacs
> with the option --init-directory this
> this string is not separated by a backslash since the value
> user-emacs-directory does not have a backslash on the end of it. This
> might break previous configs so it should be standardized to contain
> the same string value in both cases.

I meant forward-slash when I said back slash sorry generally
"user-emacs-directory" sould contain something like ".emacs.d/"
instead of ".emacs.d".

On Tue, Nov 29, 2022 at 5:55 PM Yilkal Argaw
<yilkalargawworkneh@gmail.com> wrote:
>
> In my configs I use (concat user-emacs-directory "somedirectory or
> file") to refer to files with respect to emacs. When loading emacs
> with the option --init-directory this
> this string is not separated by a backslash since the value
> user-emacs-directory does not have a backslash on the end of it. This
> might break previous configs so it should be standardized to contain
> the same string value in both cases.

Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Eli Zaretskii]

> From: Yilkal Argaw <yilkalargawworkneh@gmail.com>
> Date: Tue, 29 Nov 2022 17:55:45 +0000
> 
> In my configs I use (concat user-emacs-directory "somedirectory or
> file") to refer to files with respect to emacs. When loading emacs
> with the option --init-directory this
> this string is not separated by a backslash since the value
> user-emacs-directory does not have a backslash on the end of it. This
> might break previous configs so it should be standardized to contain
> the same string value in both cases.

Don't use 'concat' for file names; use expand-file-name instead.

RE: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Drew Adams]

> Don't use 'concat' for file names; use expand-file-name instead.

This seems to be a fairly common question/misstep.
I wonder if it might be worthwhile to mention this
in the doc string of `concat'.  That's the most
likely string-manipulation function to be (mis)used
for file-name manipulation.

Re: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Greg Farough]

[-- Attachment #1: Type: text/plain, Size: 637 bytes --]

On Tue, Nov 29 2022, Drew Adams <drew.adams@oracle.com> wrote:

>> Don't use 'concat' for file names; use expand-file-name instead.
>
> This seems to be a fairly common question/misstep.
> I wonder if it might be worthwhile to mention this
> in the doc string of `concat'.  That's the most
> likely string-manipulation function to be (mis)used
> for file-name manipulation.

I've been using Emacs for years and just learned this today from Eli's
message, so, probably.

No one ever said that I was using Emacs _well_, though.

-g

-- 
"We carry a new world here, in our hearts. That world is growing in
this minute."

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

Re: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Stefan Monnier]

> No one ever said that I was using Emacs _well_, though.

Maybe we could tweak the byte-compiler so it emits congratulatory
messages for every 10-or-so uses of `expand-file-name`?


        Stefan

Re: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Greg Farough]

On Tue, Nov 29 2022, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

>> No one ever said that I was using Emacs _well_, though.
>
> Maybe we could tweak the byte-compiler so it emits congratulatory
> messages for every 10-or-so uses of `expand-file-name`?

Sounds good to me! : )

-g

-- 
"We carry a new world here, in our hearts. That world is growing in
this minute."

Re: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Juanma Barranquero]

[-- Attachment #1: Type: text/plain, Size: 131 bytes --]

No, we need a new emacs-feedback library, with an user option to select
'positive-reinforcement or 'passive-aggresive minor modes.

[-- Attachment #2: Type: text/html, Size: 240 bytes --]

RE: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Drew Adams]

> No, we need a new emacs-feedback library, with
> an user option to select 'positive-reinforcement
> or 'passive-aggresive minor modes.

Our mails overlapped.  Clearly we sometimes think
alike.  But I vote for a single option, with
multiple possibilities.

RE: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Drew Adams]

> > No one ever said that I was using Emacs _well_, though.
> 
> Maybe we could tweak the byte-compiler so it emits congratulatory
> messages for every 10-or-so uses of `expand-file-name`?

That's if option `reinforcement' is contains
symbol `positive'.  If the list value contains
`negative' then it can emit warnings to impose
appropriate penalties (TBD).  If both then both.

Re: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > > Maybe we could tweak the byte-compiler so it emits congratulatory
  > > messages for every 10-or-so uses of `expand-file-name`?

  > That's if option `reinforcement' is contains
  > symbol `positive'.  If the list value contains
  > `negative' then it can emit warnings to impose
  > appropriate penalties (TBD).  If both then both.

How about a flag to make `concat' detect that you're doing
file name concatenation and enter the debugger?

THat sounds like an absurd exaggeration, but it
might be a useful debugging feature, for real.
We would not ever want it on by default.

-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)

Re: [External] : Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Yuri Khan]

On Sat, 3 Dec 2022 at 05:49, Richard Stallman <rms@gnu.org> wrote:

>   > > Maybe we could tweak the byte-compiler so it emits congratulatory
>   > > messages for every 10-or-so uses of `expand-file-name`?
>
>   > That's if option `reinforcement' is contains
>   > symbol `positive'.  If the list value contains
>   > `negative' then it can emit warnings to impose
>   > appropriate penalties (TBD).  If both then both.
>
> How about a flag to make `concat' detect that you're doing
> file name concatenation and enter the debugger?

You’re dangerously close to inventing a type system ;)

Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Robin Tarsiger]

Eli Zaretskii wrote:
>> From: Yilkal Argaw <yilkalargawworkneh@gmail.com>
>>
>> In my configs I use (concat user-emacs-directory "somedirectory or
>> file") to refer to files with respect to emacs. When loading emacs
>> with the option --init-directory this
>> this string is not separated by a backslash since the value
>> user-emacs-directory does not have a backslash on the end of it. This
>> might break previous configs so it should be standardized to contain
>> the same string value in both cases.
> 
> Don't use 'concat' for file names; use expand-file-name instead.

While that's as may be in general, the docstring for user-emacs-directory
(in my Emacs 28.2) states "Note that this should end with a directory
separator." specifically, which I take to be part of the variable contract.
Doesn't that imply that it is Emacs's responsibility to handle that aspect
if setting user-emacs-directory based on a file name that may not satisfy
that predicate?

-RTT

Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Yilkal Argaw]

[-- Attachment #1: Type: text/plain, Size: 507 bytes --]

>
> While that's as may be in general, the docstring for user-emacs-directory
> (in my Emacs 28.2) states "Note that this should end with a directory
> separator." specifically, which I take to be part of the variable contract.
> Doesn't that imply that it is Emacs's responsibility to handle that aspect
> if setting user-emacs-directory based on a file name that may not satisfy
> that predicate?
>
> -RTT


Correct.  consistency and standardization on such matters is better for
overall user experience.

[-- Attachment #2: Type: text/html, Size: 731 bytes --]

Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Eli Zaretskii]

> From: Yilkal Argaw <yilkalargawworkneh@gmail.com>
> Date: Mon, 12 Dec 2022 03:41:17 +0000
> Cc: Eli Zaretskii <eliz@gnu.org>, emacs-devel@gnu.org
> 
>  While that's as may be in general, the docstring for user-emacs-directory
>  (in my Emacs 28.2) states "Note that this should end with a directory
>  separator." specifically, which I take to be part of the variable contract.
>  Doesn't that imply that it is Emacs's responsibility to handle that aspect
>  if setting user-emacs-directory based on a file name that may not satisfy
>  that predicate?
> 
>  -RTT
> 
> Correct.  consistency and standardization on such matters is better for overall user experience.

The issue was fixed more than a week ago, so I'm unsure what is the
purpose of this post.  We are consistent.

Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Yilkal Argaw]

[-- Attachment #1: Type: text/plain, Size: 1117 bytes --]

>
> The issue was fixed more than a week ago, so I'm unsure what is the
> purpose of this post.  We are consistent.


Sorry I didn't see anything regarding a patch or a commit to fix the issue
in the thread. Sorry about that.

On Mon, Dec 12, 2022 at 12:23 PM Eli Zaretskii <eliz@gnu.org> wrote:

> > From: Yilkal Argaw <yilkalargawworkneh@gmail.com>
> > Date: Mon, 12 Dec 2022 03:41:17 +0000
> > Cc: Eli Zaretskii <eliz@gnu.org>, emacs-devel@gnu.org
> >
> >  While that's as may be in general, the docstring for
> user-emacs-directory
> >  (in my Emacs 28.2) states "Note that this should end with a directory
> >  separator." specifically, which I take to be part of the variable
> contract.
> >  Doesn't that imply that it is Emacs's responsibility to handle that
> aspect
> >  if setting user-emacs-directory based on a file name that may not
> satisfy
> >  that predicate?
> >
> >  -RTT
> >
> > Correct.  consistency and standardization on such matters is better for
> overall user experience.
>
> The issue was fixed more than a week ago, so I'm unsure what is the
> purpose of this post.  We are consistent.
>

[-- Attachment #2: Type: text/html, Size: 1863 bytes --]

Re: missing "/" in value of user-emacs-directory when using --init-directory flag

[Eli Zaretskii]

> From: Yilkal Argaw <yilkalargawworkneh@gmail.com>
> Date: Mon, 12 Dec 2022 18:04:10 +0000
> Cc: rtt@dasyatidae.com, emacs-devel@gnu.org
> 
>  The issue was fixed more than a week ago, so I'm unsure what is the
>  purpose of this post.  We are consistent.
> 
> Sorry I didn't see anything regarding a patch or a commit to fix the issue in the thread. Sorry about that.

See commit 401f76cc3d.