unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Arash Esbati <arash@gnu.org>
To: Andrew Cohen <cohen@bu.edu>
Cc: Andrew G Cohen <cohen@andy.bu.edu>,
	62815@debbugs.gnu.org, Andreas Schwab <schwab@linux-m68k.org>,
	bronger@physik.rwth-aachen.de, larsi@gnus.org,
	Eli Zaretskii <eliz@gnu.org>
Subject: bug#62815: Gnus: MIME types of attachments contain regexp quotation
Date: Mon, 24 Apr 2023 12:29:38 +0200	[thread overview]
Message-ID: <86bkjd3759.fsf@gnu.org> (raw)
In-Reply-To: <87bkjf8gj4.fsf@ust.hk> (Andrew Cohen's message of "Sun, 23 Apr 2023 10:43:27 +0800")

Andrew Cohen <cohen@bu.edu> writes:

> If all this is right, the simple fix is to revert the part of
> 3faa508eba84a1983732099cbd3cc1eaad404158 that treated the extension data
> as regexps.  I have attached a diff below that does this. If Torsten
> and Arash can check if this continues to provide a fix for bug#52038 and
> bug#62815 that would be great!

Thanks for looking at this.  I built the latest repo version
(65735cee71) of Emacs and tried it with and w/o your patch.  It seems to
fix the issues Torsten and I have reported, but ...

> There is one more place I think needs fixing (and maybe others I haven't
> spotted yet):

I see a strange side effect when I use 'e' in article buffer to view a
attachment.  Without your change, Emacs starts Excel like this (this it
at least I see in *Messages* buffer) and it works:

  Displaying start excel.exe "`cygpath -w -a c\:/msys64/tmp/emm.fMzcnA/file_name.xlsx`"

With your change, I get:

  Displaying start excel.exe "`cygpath -w -a "c:/msys64/tmp/emm.QDUAFP/file_name.xlsx"`"

which doesn't work due to extra pair of quotes.  This is the entry in my
.mailcap:

  application/vnd.openxmlformats-officedocument.spreadsheetml.sheet; \
                       start excel.exe "`cygpath -w -a %s`"

This on Win10.

Best, Arash





  reply	other threads:[~2023-04-24 10:29 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-13 11:46 bug#62815: Gnus: MIME types of attachments contain regexp quotation Torsten Bronger
2023-04-13 14:48 ` Eli Zaretskii
2023-04-22  8:42   ` Eli Zaretskii
2023-04-23  2:43     ` Andrew Cohen
2023-04-24 10:29       ` Arash Esbati [this message]
2023-04-30  1:26     ` Andrew Cohen
2023-05-07  0:21 ` Andrew Cohen
2023-05-07  5:17   ` Eli Zaretskii
2023-09-14  0:42     ` Stefan Kangas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=86bkjd3759.fsf@gnu.org \
    --to=arash@gnu.org \
    --cc=62815@debbugs.gnu.org \
    --cc=bronger@physik.rwth-aachen.de \
    --cc=cohen@andy.bu.edu \
    --cc=cohen@bu.edu \
    --cc=eliz@gnu.org \
    --cc=larsi@gnus.org \
    --cc=schwab@linux-m68k.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).