* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
[not found] <s0dwmqaiwsq.fsf.ref@yahoo.com>
@ 2024-03-10 3:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-10 6:26 ` Eli Zaretskii
2024-03-15 2:06 ` Dmitry Gutov
0 siblings, 2 replies; 10+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2024-03-10 3:35 UTC (permalink / raw)
To: 69691
New VCS output inserted after the compilation buffer created by a
`vc-pull' command is refreshed (by selecting "refresh" from tool bar,
for instance) is not fontified as such but as ordinary compiler output,
and file names inside are not clickable. This is a bug, since no
compelling reason exists to treat VCS commands as compilers once they
are executed a second time.
In GNU Emacs 30.0.50 (build 1, aarch64-unknown-linux-android22) of
2024-03-10 built on trinity
Repository revision: d7071359522f60d332320cf63700faa58d237d0b
Repository branch: master-android-1
Windowing system distributor 'Google', version 30.0.0
System Description: google/marlin/marlin:10/QP1A.191005.007.A3/5972272:user/release-keys
Configured using:
'configure --with-png=yes --with-webp=yes --with-gif=yes
--with-json=ifavailable --with-jpeg=yes --with-xml2=yes
--with-sqlite3=yes --with-gnutls=yes --with-tiff=yes --with-selinux=yes
--with-modules=yes --with-tree-sitter=ifavailable --with-imagemagick=no
--with-lcms2=yes --with-mailutils= --with-pop=no-by-default
--with-harfbuzz=yes --with-threads=yes --enable-check-lisp-object-type=
'CFLAGS=-O2 -g3 -Werror=implicit-function-declaration''
Configured features:
GIF GMP GNUTLS HARFBUZZ JPEG JSON LIBSELINUX LIBXML2 MODULES NOTIFY
INOTIFY PDUMPER PNG SECCOMP SQLITE3 THREADS TIFF TREE_SITTER WEBP ZLIB
Important settings:
value of $LANG: en_US.utf8
locale-coding-system: utf-8-unix
Major mode: Compilation
Minor modes in effect:
save-place-mode: t
kill-ring-deindent-mode: t
electric-pair-mode: t
server-mode: t
tooltip-mode: t
global-eldoc-mode: t
show-paren-mode: t
electric-indent-mode: t
mouse-wheel-mode: t
modifier-bar-mode: t
tool-bar-mode: t
tab-bar-mode: t
menu-bar-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
window-divider-mode: t
minibuffer-regexp-mode: t
buffer-read-only: 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
Features:
(compile vc-hg vc-git diff-mode vc-bzr vc-src vc-sccs vc-svn vc-cvs
vc-rcs log-view easy-mmode pcvs-util vc-dir ewoc vc vc-dispatcher
dired-aux misearch multi-isearch pp shadow sort mail-extr emacsbug
message yank-media puny dired dired-loaddefs rfc822 mml mml-sec epa
derived epg rfc6068 epg-config mm-decode mm-bodies mm-encode mail-parse
rfc2231 mailabbrev gmm-utils mailheader sendmail rfc2047 rfc2045
ietf-drums org-mouse org ob ob-tangle ob-ref ob-lob ob-table ob-exp
org-macro org-src ob-comint org-pcomplete org-list org-footnote
org-faces org-entities noutline outline org-version ob-emacs-lisp
ob-core ob-eval org-cycle org-table ol org-fold org-fold-core org-keys
oc org-loaddefs find-func cal-menu calendar cal-loaddefs org-compat
org-macs cl-extra icons saveplace tramp-sh tramp rx trampver
tramp-integration files-x tramp-message help-mode tramp-compat xdg shell
pcomplete comint ansi-osc ring parse-time iso8601 format-spec ansi-color
tramp-loaddefs indent-aux gnus nnheader gnus-util text-property-search
time-date mail-utils range mm-util mail-prsvr wid-edit elec-pair
cus-load server aircon-theme-autoloads bash-completion-autoloads
bison-mode-autoloads corfu-autoloads coterm-autoloads debbugs-autoloads
ement-autoloads jinx-autoloads jit-spell-autoloads magit-autoloads
git-commit-autoloads markdown-mode-autoloads org-drill-autoloads
persist-autoloads plz-autoloads svg-lib-autoloads
taxy-magit-section-autoloads taxy-autoloads magit-section-autoloads
dash-autoloads transient-autoloads visual-fill-autoloads
visual-fill-column-autoloads info with-editor-autoloads package
browse-url url url-proxy url-privacy url-expand url-methods url-history
url-cookie generate-lisp-file url-domsuf url-util mailcap url-handlers
url-parse auth-source cl-seq eieio eieio-core cl-macs password-cache
json subr-x map byte-opt gv bytecomp byte-compile url-vars cl-loaddefs
cl-lib rmc iso-transl tooltip cus-start cconv eldoc paren electric
uniquify ediff-hook vc-hooks lisp-float-type elisp-mode mwheel
android-win term/common-win touch-screen ls-lisp 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 inotify android multi-tty
move-toolbar make-network-process emacs)
Memory information:
((conses 16 338563 122792) (symbols 48 35588 32)
(strings 32 102829 3470) (string-bytes 1 2541090) (vectors 16 57987)
(vector-slots 8 1477083 25171) (floats 8 325 231)
(intervals 56 1220 595) (buffers 984 17))
^ permalink raw reply [flat|nested] 10+ messages in thread* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-10 3:35 ` bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2024-03-10 6:26 ` Eli Zaretskii
2024-03-10 6:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-15 2:06 ` Dmitry Gutov
1 sibling, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2024-03-10 6:26 UTC (permalink / raw)
To: Po Lu; +Cc: 69691
> Date: Sun, 10 Mar 2024 11:35:17 +0800
> From: Po Lu via "Bug reports for GNU Emacs,
> the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
>
> New VCS output inserted after the compilation buffer created by a
> `vc-pull' command is refreshed (by selecting "refresh" from tool bar,
> for instance) is not fontified as such but as ordinary compiler output,
> and file names inside are not clickable. This is a bug, since no
> compelling reason exists to treat VCS commands as compilers once they
> are executed a second time.
Please provide a recipe for reproducing the problem starting from
"emacs -Q". The above description is too terse and lacks crucial
details, like the stuff you see in the buffer, which parts are not
fontified and/or not clickable, etc.
Thanks.
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-10 6:26 ` Eli Zaretskii
@ 2024-03-10 6:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
0 siblings, 0 replies; 10+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2024-03-10 6:35 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: 69691
Eli Zaretskii <eliz@gnu.org> writes:
>> Date: Sun, 10 Mar 2024 11:35:17 +0800
>> From: Po Lu via "Bug reports for GNU Emacs,
>> the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
>>
>> New VCS output inserted after the compilation buffer created by a
>> `vc-pull' command is refreshed (by selecting "refresh" from tool bar,
>> for instance) is not fontified as such but as ordinary compiler output,
>> and file names inside are not clickable. This is a bug, since no
>> compelling reason exists to treat VCS commands as compilers once they
>> are executed a second time.
>
> Please provide a recipe for reproducing the problem starting from
> "emacs -Q". The above description is too terse and lacks crucial
> details, like the stuff you see in the buffer, which parts are not
> fontified and/or not clickable, etc.
I'm not sure what I omitted, but it should be reproducible by typing "g"
in a compilation buffer created by vc-pull. If any new upstream changes
are then detected and retrieved, the file names the resulting VCS
process prints will not be fontified or made clickable.
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-10 3:35 ` bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-10 6:26 ` Eli Zaretskii
@ 2024-03-15 2:06 ` Dmitry Gutov
2024-03-15 2:49 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-15 8:08 ` Eli Zaretskii
1 sibling, 2 replies; 10+ messages in thread
From: Dmitry Gutov @ 2024-03-15 2:06 UTC (permalink / raw)
To: Po Lu, 69691
On 10/03/2024 05:35, Po Lu via Bug reports for GNU Emacs, the Swiss army
knife of text editors wrote:
> New VCS output inserted after the compilation buffer created by a
> `vc-pull' command is refreshed (by selecting "refresh" from tool bar,
You probably mean "Recompile".
> for instance) is not fontified as such but as ordinary compiler output,
> and file names inside are not clickable. This is a bug, since no
> compelling reason exists to treat VCS commands as compilers once they
> are executed a second time.
I guess some specific provision for restoring more of the variables
after "recompile" could be made inside vc-compilation-mode's definition.
Maybe it'll have to go as far as defining a separate major mode and its
own "recompile" command with a dedicated binding, since it doesn't seem
like 'M-x recompile' runs anything generic that would help us here.
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-15 2:06 ` Dmitry Gutov
@ 2024-03-15 2:49 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-15 3:00 ` Dmitry Gutov
2024-03-15 8:08 ` Eli Zaretskii
1 sibling, 1 reply; 10+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2024-03-15 2:49 UTC (permalink / raw)
To: Dmitry Gutov; +Cc: 69691
Dmitry Gutov <dmitry@gutov.dev> writes:
> You probably mean "Recompile".
That's the name of the command, yes. I was going off the name of the
tool bar icon.
> I guess some specific provision for restoring more of the variables
> after "recompile" could be made inside vc-compilation-mode's
> definition.
>
> Maybe it'll have to go as far as defining a separate major mode and
> its own "recompile" command with a dedicated binding, since it doesn't
> seem like 'M-x recompile' runs anything generic that would help us
> here.
Sure, I'm only concerned with the functioning of the tool bar item.
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-15 2:49 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2024-03-15 3:00 ` Dmitry Gutov
0 siblings, 0 replies; 10+ messages in thread
From: Dmitry Gutov @ 2024-03-15 3:00 UTC (permalink / raw)
To: Po Lu; +Cc: 69691
On 15/03/2024 04:49, Po Lu wrote:
> Dmitry Gutov <dmitry@gutov.dev> writes:
>
>> You probably mean "Recompile".
>
> That's the name of the command, yes. I was going off the name of the
> tool bar icon.
In compilation-mode the button has a text as well. Anyway, not important.
>> I guess some specific provision for restoring more of the variables
>> after "recompile" could be made inside vc-compilation-mode's
>> definition.
>>
>> Maybe it'll have to go as far as defining a separate major mode and
>> its own "recompile" command with a dedicated binding, since it doesn't
>> seem like 'M-x recompile' runs anything generic that would help us
>> here.
>
> Sure, I'm only concerned with the functioning of the tool bar item.
Right, and the above is the way to reach that, if I haven't missed
something.
I don't have the time to get to this now, so patches welcome.
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-15 2:06 ` Dmitry Gutov
2024-03-15 2:49 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2024-03-15 8:08 ` Eli Zaretskii
2024-03-15 12:10 ` Dmitry Gutov
1 sibling, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2024-03-15 8:08 UTC (permalink / raw)
To: Dmitry Gutov; +Cc: luangruo, 69691
> Date: Fri, 15 Mar 2024 04:06:23 +0200
> From: Dmitry Gutov <dmitry@gutov.dev>
>
> > for instance) is not fontified as such but as ordinary compiler output,
> > and file names inside are not clickable. This is a bug, since no
> > compelling reason exists to treat VCS commands as compilers once they
> > are executed a second time.
>
> I guess some specific provision for restoring more of the variables
> after "recompile" could be made inside vc-compilation-mode's definition.
>
> Maybe it'll have to go as far as defining a separate major mode and its
> own "recompile" command with a dedicated binding, since it doesn't seem
> like 'M-x recompile' runs anything generic that would help us here.
Maybe just customize revert-buffer-function in vc-compilation-mode?
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-15 8:08 ` Eli Zaretskii
@ 2024-03-15 12:10 ` Dmitry Gutov
2024-03-15 12:41 ` Eli Zaretskii
0 siblings, 1 reply; 10+ messages in thread
From: Dmitry Gutov @ 2024-03-15 12:10 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: luangruo, 69691
On 15/03/2024 10:08, Eli Zaretskii wrote:
>> Date: Fri, 15 Mar 2024 04:06:23 +0200
>> From: Dmitry Gutov<dmitry@gutov.dev>
>>
>>> for instance) is not fontified as such but as ordinary compiler output,
>>> and file names inside are not clickable. This is a bug, since no
>>> compelling reason exists to treat VCS commands as compilers once they
>>> are executed a second time.
>> I guess some specific provision for restoring more of the variables
>> after "recompile" could be made inside vc-compilation-mode's definition.
>>
>> Maybe it'll have to go as far as defining a separate major mode and its
>> own "recompile" command with a dedicated binding, since it doesn't seem
>> like 'M-x recompile' runs anything generic that would help us here.
> Maybe just customize revert-buffer-function in vc-compilation-mode?
'M-x recompile' doesn't call it.
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-15 12:10 ` Dmitry Gutov
@ 2024-03-15 12:41 ` Eli Zaretskii
2024-03-15 12:59 ` Dmitry Gutov
0 siblings, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2024-03-15 12:41 UTC (permalink / raw)
To: Dmitry Gutov; +Cc: luangruo, 69691
> Date: Fri, 15 Mar 2024 14:10:39 +0200
> Cc: luangruo@yahoo.com, 69691@debbugs.gnu.org
> From: Dmitry Gutov <dmitry@gutov.dev>
>
> On 15/03/2024 10:08, Eli Zaretskii wrote:
> >> Date: Fri, 15 Mar 2024 04:06:23 +0200
> >> From: Dmitry Gutov<dmitry@gutov.dev>
> >>
> >>> for instance) is not fontified as such but as ordinary compiler output,
> >>> and file names inside are not clickable. This is a bug, since no
> >>> compelling reason exists to treat VCS commands as compilers once they
> >>> are executed a second time.
> >> I guess some specific provision for restoring more of the variables
> >> after "recompile" could be made inside vc-compilation-mode's definition.
> >>
> >> Maybe it'll have to go as far as defining a separate major mode and its
> >> own "recompile" command with a dedicated binding, since it doesn't seem
> >> like 'M-x recompile' runs anything generic that would help us here.
> > Maybe just customize revert-buffer-function in vc-compilation-mode?
>
> 'M-x recompile' doesn't call it.
Po Lu said to press 'g', which AFAIU does?
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh
2024-03-15 12:41 ` Eli Zaretskii
@ 2024-03-15 12:59 ` Dmitry Gutov
0 siblings, 0 replies; 10+ messages in thread
From: Dmitry Gutov @ 2024-03-15 12:59 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: luangruo, 69691
On 15/03/2024 14:41, Eli Zaretskii wrote:
>> Date: Fri, 15 Mar 2024 14:10:39 +0200
>> Cc:luangruo@yahoo.com,69691@debbugs.gnu.org
>> From: Dmitry Gutov<dmitry@gutov.dev>
>>
>> On 15/03/2024 10:08, Eli Zaretskii wrote:
>>>> Date: Fri, 15 Mar 2024 04:06:23 +0200
>>>> From: Dmitry Gutov<dmitry@gutov.dev>
>>>>
>>>>> for instance) is not fontified as such but as ordinary compiler output,
>>>>> and file names inside are not clickable. This is a bug, since no
>>>>> compelling reason exists to treat VCS commands as compilers once they
>>>>> are executed a second time.
>>>> I guess some specific provision for restoring more of the variables
>>>> after "recompile" could be made inside vc-compilation-mode's definition.
>>>>
>>>> Maybe it'll have to go as far as defining a separate major mode and its
>>>> own "recompile" command with a dedicated binding, since it doesn't seem
>>>> like 'M-x recompile' runs anything generic that would help us here.
>>> Maybe just customize revert-buffer-function in vc-compilation-mode?
>> 'M-x recompile' doesn't call it.
> Po Lu said to press 'g', which AFAIU does?
It calls 'recompile'.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2024-03-15 12:59 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <s0dwmqaiwsq.fsf.ref@yahoo.com>
2024-03-10 3:35 ` bug#69691: 30.0.50; vc-pull buffer doesn't fontify links after refresh Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-10 6:26 ` Eli Zaretskii
2024-03-10 6:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-15 2:06 ` Dmitry Gutov
2024-03-15 2:49 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-03-15 3:00 ` Dmitry Gutov
2024-03-15 8:08 ` Eli Zaretskii
2024-03-15 12:10 ` Dmitry Gutov
2024-03-15 12:41 ` Eli Zaretskii
2024-03-15 12:59 ` Dmitry Gutov
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).