unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp
@ 2023-01-02 11:26 Mohammed Sadiq
  2023-01-02 12:43 ` Eli Zaretskii
  0 siblings, 1 reply; 6+ messages in thread
From: Mohammed Sadiq @ 2023-01-02 11:26 UTC (permalink / raw)
  To: 60490

Opening sqlite db files from remote system over tramp (with M-x
sqlite-mode-open-file) fails with the following error in *Messages*:

sqlite-mode-list-tables: Wrong type argument: sqlitep, nil


In GNU Emacs 29.0.60 (build 5, x86_64-pc-linux-gnu, GTK+ Version
  3.24.35, cairo version 1.16.0) of 2023-01-02 built on purism
Repository revision: 2569ede9c496bb060e0b88428cb541088aaba1f9
Repository branch: emacs-29
Windowing system distributor 'The X.Org Foundation', version 
11.0.12101005
System Description: Debian GNU/Linux bookworm/sid

Configured using:
  'configure --prefix=/usr'

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG
JSON LCMS2 LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NOTIFY
INOTIFY PDUMPER PNG RSVG SECCOMP SOUND SQLITE3 THREADS TIFF
TOOLKIT_SCROLL_BARS TREE_SITTER WEBP X11 XDBE XIM XINPUT2 XPM GTK3 ZLIB

Important settings:
   value of $LANG: en_IN.UTF-8
   value of $XMODIFIERS: @im=ibus
   locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
   tooltip-mode: t
   global-eldoc-mode: t
   eldoc-mode: t
   show-paren-mode: t
   electric-indent-mode: t
   mouse-wheel-mode: t
   tool-bar-mode: t
   menu-bar-mode: t
   file-name-shadow-mode: t
   global-font-lock-mode: t
   font-lock-mode: t
   blink-cursor-mode: t
   line-number-mode: t
   indent-tabs-mode: t
   transient-mark-mode: t
   auto-composition-mode: t
   auto-encryption-mode: t
   auto-compression-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message mailcap yank-media puny dired
dired-loaddefs rfc822 mml mml-sec password-cache epa derived epg rfc6068
epg-config gnus-util text-property-search time-date subr-x mm-decode
mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader
cl-loaddefs cl-lib sendmail rfc2047 rfc2045 ietf-drums mm-util
mail-prsvr mail-utils rmc iso-transl tooltip cconv eldoc paren electric
uniquify ediff-hook vc-hooks lisp-float-type elisp-mode mwheel
term/x-win x-win term/common-win x-dnd tool-bar dnd fontset image
regexp-opt fringe tabulated-list replace newcomment text-mode lisp-mode
prog-mode register page tab-bar menu-bar rfn-eshadow isearch easymenu
timer select scroll-bar mouse jit-lock font-lock syntax font-core
term/tty-colors frame minibuffer nadvice seq simple cl-generic
indonesian philippine cham georgian utf-8-lang misc-lang vietnamese
tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek
romanian slovak czech european ethiopic indian cyrillic chinese
composite emoji-zwj charscript charprop case-table epa-hook
jka-cmpr-hook help abbrev obarray oclosure cl-preloaded button loaddefs
theme-loaddefs faces cus-face macroexp files window text-properties
overlay sha1 md5 base64 format env code-pages mule custom widget keymap
hashtable-print-readable backquote threads dbusbind inotify lcms2
dynamic-setting system-font-setting font-render-setting cairo
move-toolbar gtk x-toolkit xinput2 x multi-tty make-network-process
emacs)

Memory information:
((conses 16 36062 9075)
  (symbols 48 5148 0)
  (strings 32 13055 1639)
  (string-bytes 1 367474)
  (vectors 16 9288)
  (vector-slots 8 147524 12377)
  (floats 8 21 22)
  (intervals 56 233 0)
  (buffers 984 11))





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp
  2023-01-02 11:26 bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp Mohammed Sadiq
@ 2023-01-02 12:43 ` Eli Zaretskii
  2023-01-15  5:41   ` Mohammed Sadiq
  0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2023-01-02 12:43 UTC (permalink / raw)
  To: Mohammed Sadiq; +Cc: 60490

severity 60490 wishlist
thanks

> Date: Mon, 02 Jan 2023 16:56:24 +0530
> From: Mohammed Sadiq <sadiq@sadiqpk.org>
> 
> Opening sqlite db files from remote system over tramp (with M-x
> sqlite-mode-open-file) fails with the following error in *Messages*:
> 
> sqlite-mode-list-tables: Wrong type argument: sqlitep, nil

The built-in SQLite support doesn't (yet) support remote DB access.
So I'm converting this report to a feature request.  Patches welcome.

Thanks.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp
  2023-01-02 12:43 ` Eli Zaretskii
@ 2023-01-15  5:41   ` Mohammed Sadiq
  2023-01-15  7:27     ` Eli Zaretskii
  0 siblings, 1 reply; 6+ messages in thread
From: Mohammed Sadiq @ 2023-01-15  5:41 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 60490

On 2023-01-02 18:13, Eli Zaretskii wrote:
> severity 60490 wishlist
> thanks
> 
>> Date: Mon, 02 Jan 2023 16:56:24 +0530
>> From: Mohammed Sadiq <sadiq@sadiqpk.org>
>> 
>> Opening sqlite db files from remote system over tramp (with M-x
>> sqlite-mode-open-file) fails with the following error in *Messages*:
>> 
>> sqlite-mode-list-tables: Wrong type argument: sqlitep, nil
> 
> The built-in SQLite support doesn't (yet) support remote DB access.
> So I'm converting this report to a feature request.  Patches welcome.
> 

In that case, it would be better to show a warning message that
it's not supported instead of failing with errors that gives no
hint to the user.

Also, Emacs may also show the related bug number regarding such
issues so that interested users can subscribe to know updates.


thanks,
Mohammed Sadiq





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp
  2023-01-15  5:41   ` Mohammed Sadiq
@ 2023-01-15  7:27     ` Eli Zaretskii
  2023-01-15  8:19       ` Mohammed Sadiq
  0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2023-01-15  7:27 UTC (permalink / raw)
  To: Mohammed Sadiq; +Cc: 60490

> Date: Sun, 15 Jan 2023 11:11:49 +0530
> From: Mohammed Sadiq <sadiq@sadiqpk.org>
> Cc: 60490@debbugs.gnu.org
> 
> On 2023-01-02 18:13, Eli Zaretskii wrote:
> > severity 60490 wishlist
> > thanks
> > 
> >> Date: Mon, 02 Jan 2023 16:56:24 +0530
> >> From: Mohammed Sadiq <sadiq@sadiqpk.org>
> >> 
> >> Opening sqlite db files from remote system over tramp (with M-x
> >> sqlite-mode-open-file) fails with the following error in *Messages*:
> >> 
> >> sqlite-mode-list-tables: Wrong type argument: sqlitep, nil
> > 
> > The built-in SQLite support doesn't (yet) support remote DB access.
> > So I'm converting this report to a feature request.  Patches welcome.
> > 
> 
> In that case, it would be better to show a warning message that
> it's not supported instead of failing with errors that gives no
> hint to the user.

I disagree that we should show a warning in such cases.  The user
asked Emacs to do something, and Emacs couldn't; the appropriate
response is to signal an error, not to show a warning.  Showing a
warning would mean we don't abort whatever Lisp program that was
trying to do something we cannot do.

I agree that a more targeted error message would be beneficial;
patches for that are welcome as well.

> Also, Emacs may also show the related bug number regarding such
> issues so that interested users can subscribe to know updates.

We don't have any infrastructure in Emacs for doing stuff like that.
It would be a useful feature, I think (although not easy to
implement), but please submit a separate feature request for it.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp
  2023-01-15  7:27     ` Eli Zaretskii
@ 2023-01-15  8:19       ` Mohammed Sadiq
  2023-01-15 10:10         ` Eli Zaretskii
  0 siblings, 1 reply; 6+ messages in thread
From: Mohammed Sadiq @ 2023-01-15  8:19 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 60490

On 2023-01-15 12:57, Eli Zaretskii wrote:
>> Date: Sun, 15 Jan 2023 11:11:49 +0530
>> From: Mohammed Sadiq <sadiq@sadiqpk.org>
>> Cc: 60490@debbugs.gnu.org
>> 
>> On 2023-01-02 18:13, Eli Zaretskii wrote:
>> > severity 60490 wishlist
>> > thanks
>> >
>> >> Date: Mon, 02 Jan 2023 16:56:24 +0530
>> >> From: Mohammed Sadiq <sadiq@sadiqpk.org>
>> >>
>> >> Opening sqlite db files from remote system over tramp (with M-x
>> >> sqlite-mode-open-file) fails with the following error in *Messages*:
>> >>
>> >> sqlite-mode-list-tables: Wrong type argument: sqlitep, nil
>> >
>> > The built-in SQLite support doesn't (yet) support remote DB access.
>> > So I'm converting this report to a feature request.  Patches welcome.
>> >
>> 
>> In that case, it would be better to show a warning message that
>> it's not supported instead of failing with errors that gives no
>> hint to the user.
> 
> I disagree that we should show a warning in such cases.  The user
> asked Emacs to do something, and Emacs couldn't; the appropriate
> response is to signal an error, not to show a warning.  Showing a
> warning would mean we don't abort whatever Lisp program that was
> trying to do something we cannot do.
> 
> I agree that a more targeted error message would be beneficial;
> patches for that are welcome as well.

An error message was what I meant.

> 
>> Also, Emacs may also show the related bug number regarding such
>> issues so that interested users can subscribe to know updates.
> 
> We don't have any infrastructure in Emacs for doing stuff like that.
> It would be a useful feature, I think (although not easy to
> implement), but please submit a separate feature request for it.

I was suggesting to show an error message like:
"sqlite-xxx is not supported over tramp, see bug#xxx"





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp
  2023-01-15  8:19       ` Mohammed Sadiq
@ 2023-01-15 10:10         ` Eli Zaretskii
  0 siblings, 0 replies; 6+ messages in thread
From: Eli Zaretskii @ 2023-01-15 10:10 UTC (permalink / raw)
  To: Mohammed Sadiq; +Cc: 60490

> Date: Sun, 15 Jan 2023 13:49:49 +0530
> From: Mohammed Sadiq <sadiq@sadiqpk.org>
> Cc: 60490@debbugs.gnu.org
> 
> >> In that case, it would be better to show a warning message that
> >> it's not supported instead of failing with errors that gives no
> >> hint to the user.
> > 
> > I disagree that we should show a warning in such cases.  The user
> > asked Emacs to do something, and Emacs couldn't; the appropriate
> > response is to signal an error, not to show a warning.  Showing a
> > warning would mean we don't abort whatever Lisp program that was
> > trying to do something we cannot do.
> > 
> > I agree that a more targeted error message would be beneficial;
> > patches for that are welcome as well.
> 
> An error message was what I meant.

I've now added a more meaningful error message to sqlite-mode.el.

> >> Also, Emacs may also show the related bug number regarding such
> >> issues so that interested users can subscribe to know updates.
> > 
> > We don't have any infrastructure in Emacs for doing stuff like that.
> > It would be a useful feature, I think (although not easy to
> > implement), but please submit a separate feature request for it.
> 
> I was suggesting to show an error message like:
> "sqlite-xxx is not supported over tramp, see bug#xxx"

Sorry, no: I don't think it's right for us to have references to bug
reports in our sources, let alone in error messages.  It's a long-term
maintenance problem, especially when we change the issue tracker.





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-01-15 10:10 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-02 11:26 bug#60490: 29.0.60; sqlite-mode-open-file fails to work over tramp Mohammed Sadiq
2023-01-02 12:43 ` Eli Zaretskii
2023-01-15  5:41   ` Mohammed Sadiq
2023-01-15  7:27     ` Eli Zaretskii
2023-01-15  8:19       ` Mohammed Sadiq
2023-01-15 10:10         ` Eli Zaretskii

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).