GNU bug report logs - #75954
[PATCH] eglot: Allow omnisharp binary to be capitalized "OmniSharp"

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 75954 in the body.
You can then email your comments to 75954 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Damien Cassou <damien <at> cassou.me>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] eglot: Allow omnisharp binary to be capitalized "OmniSharp"
Date: Thu, 30 Jan 2025 22:15:21 +0100

[Message part 1 (text/plain, inline)]

Tags: patch

Hi,

some packages provide a binary "OmniSharp" instead of "omnisharp". In
a case-sensitive file-systems this means eglot will not find the binary
file and won't work.

Best

-- 
Damien Cassou

"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill

[0001-eglot-Allow-omnisharp-binary-to-be-capitalized-OmniS.patch (text/patch, attachment)]

Message #8 received at 75954 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Damien Cassou <damien <at> cassou.me>, 75954 <at> debbugs.gnu.org
Subject: Re: bug#75954: [PATCH] eglot: Allow omnisharp binary to be
 capitalized "OmniSharp"
Date: Thu, 30 Jan 2025 21:26:53 -0500

Damien Cassou <damien <at> cassou.me> writes:

> some packages provide a binary "OmniSharp" instead of "omnisharp". In
> a case-sensitive file-systems this means eglot will not find the binary
> file and won't work.

Thanks.

Is this on MS-Windows?  Isn't that supposed to be case-insensitive?

I think we should document in this bug report at least, what are the
"some packages" above, and which systems are affected.

If the workaround is only helpful for MS-Windows, I think we should add
a comment to that effect.

Message #11 received at 75954 <at> debbugs.gnu.org (full text, mbox):

From: Damien Cassou <damien <at> cassou.me>
To: Stefan Kangas <stefankangas <at> gmail.com>, 75954 <at> debbugs.gnu.org
Subject: Re: bug#75954: [PATCH] eglot: Allow omnisharp binary to be
 capitalized "OmniSharp"
Date: Fri, 31 Jan 2025 14:26:56 +0100

Hi Stefan,

Stefan Kangas <stefankangas <at> gmail.com> writes:
> Damien Cassou <damien <at> cassou.me> writes:
>
>> some packages provide a binary "OmniSharp" instead of "omnisharp". In
>> a case-sensitive file-systems this means eglot will not find the binary
>> file and won't work.
>
> Is this on MS-Windows?  Isn't that supposed to be case-insensitive?


nixpkgs for example provides "OmniSharp"

https://search.nixos.org/packages?channel=unstable&show=omnisharp-roslyn&from=0&size=50&sort=relevance&type=packages&query=omnisharp-roslyn


-- 
Damien Cassou

"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill

Message #16 received at 75954-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Damien Cassou <damien <at> cassou.me>, 75954-done <at> debbugs.gnu.org
Subject: Re: bug#75954: [PATCH] eglot: Allow omnisharp binary to be
 capitalized "OmniSharp"
Date: Sat, 1 Feb 2025 03:29:40 +0100

Version: 30.1

Damien Cassou <damien <at> cassou.me> writes:

> Hi Stefan,
>
> Stefan Kangas <stefankangas <at> gmail.com> writes:
>> Damien Cassou <damien <at> cassou.me> writes:
>>
>>> some packages provide a binary "OmniSharp" instead of "omnisharp". In
>>> a case-sensitive file-systems this means eglot will not find the binary
>>> file and won't work.
>>
>> Is this on MS-Windows?  Isn't that supposed to be case-insensitive?
>
>
> nixpkgs for example provides "OmniSharp"
>
> https://search.nixos.org/packages?channel=unstable&show=omnisharp-roslyn&from=0&size=50&sort=relevance&type=packages&query=omnisharp-roslyn

OK, thanks.  I installed the patch on emacs-30 as 87be3aa1491.  Closing.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.