GNU bug report logs - #59471
[PATCH] sh-mode: Run treesit-ready-p only when sh-shell is 'bash

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Brian Leung <leungbk <at> posteo.net>
Subject: bug#59471: closed (Re: bug#59471: [PATCH] sh-mode: Run
 treesit-ready-p only when  sh-shell is 'bash)
Date: Wed, 23 Nov 2022 01:40:02 +0000

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

Your bug report

#59471: [PATCH] sh-mode: Run treesit-ready-p only when sh-shell is 'bash

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 59471 <at> debbugs.gnu.org.

-- 
59471: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=59471
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Yuan Fu <casouri <at> gmail.com>
To: leungbk <at> posteo.net
Cc: 59471-done <at> debbugs.gnu.org
Subject: Re: bug#59471: [PATCH] sh-mode: Run treesit-ready-p only when 
 sh-shell is 'bash
Date: Tue, 22 Nov 2022 17:39:08 -0800

Brian Leung <leungbk <at> posteo.net> writes:

> Tags: patch
>
> This patch guards the treesit-ready-p check to run only when sh-shell is
> 'bash, in order to prevent an annoying warning when viewing (say) sh or
> zsh files. But maybe we also want to allow treesitter when sh-shell is
> 'sh?

Oh, thanks! Though I did it a bit differently, by separating
tree-sitter-based mode into a independent major mode (sh-ts-mode), which
is in line with other tree-sitter-based modes (python-ts-mode,
js-ts-mode, etc).

How to make bash-ts-mode fallback to sh-mode when it encounters a shell
script other than bash is still in discussion.

Yuan

[Message part 3 (message/rfc822, inline)]

From: Brian Leung <leungbk <at> posteo.net>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] sh-mode: Run treesit-ready-p only when sh-shell is 'bash
Date: Tue, 22 Nov 2022 02:54:55 +0000

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

Tags: patch

This patch guards the treesit-ready-p check to run only when sh-shell is
'bash, in order to prevent an annoying warning when viewing (say) sh or
zsh files. But maybe we also want to allow treesitter when sh-shell is
'sh?


In GNU Emacs 29.0.50 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo
version 1.16.0, Xaw3d scroll bars)
Repository revision: aeadba1418d8fc18f17b4ae415cde35e9e272e7a
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12014000
System Description: NixOS 22.11 (Raccoon)

Configured using:
 'configure
 --prefix=/nix/store/yf7bpm9w6nqwil5xfxa9iz22m57wz4l1-emacs-git-20221121.0
 --disable-build-details --with-modules --with-x-toolkit=lucid
 --with-xft --with-cairo --with-native-compilation'

[0001-sh-mode-Run-treesit-ready-p-only-when-sh-shell-is-ba.patch (text/patch, attachment)]

Previous Next

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