GNU logs - #76417, boring messages

Message sent to bug-gnu-emacs@HIDDEN:

Subject: bug#76417: [PATCH] Add tramp-cleanup-bufferless-connections
Resent-From: Ship Mints <shipmints@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Wed, 19 Feb 2025 17:45:02 +0000
Resent-Message-ID: <handler.76417.B.1739987084386 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: 76417 <at> debbugs.gnu.org
MIME-Version: 1.0
From: Ship Mints <shipmints@HIDDEN>
Date: Wed, 19 Feb 2025 12:44:19 -0500
Message-ID: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
Content-Type: multipart/mixed; boundary="0000000000004e4b23062e824ff1"
Received-SPF: pass client-ip=2607:f8b0:4864:20::e2a;
 envelope-from=shipmints@HIDDEN; helo=mail-vs1-xe2a.google.com
X-Spam_score_int: -20
X-Spam_score: -2.1
X-Spam_bar: --
X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,
 DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001,
 HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001,
 SPF_PASS=-0.001 autolearn=ham autolearn_force=no
X-Spam_action: no action
X-Spam-Score: 1.0 (+)
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -0.0 (/)

--0000000000004e4b23062e824ff1
Content-Type: multipart/alternative; boundary="0000000000004e4b21062e824fef"

--0000000000004e4b21062e824fef
Content-Type: text/plain; charset="UTF-8"

'tramp-cleanup-bufferless-connections' cleans up connection-related objects
for which no associated buffers exist.  This is helpful to prune
connections after you close remote-file buffers without having to either
cherry pick via 'tramp-cleanup-connection' or clear them all via
'tramp-cleanup-all-connections'.

I looked for an equivalent but didn't see one so I've been using this
command, and I think it's ready for contribution.

-Stephane

--0000000000004e4b21062e824fef
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_default" style=3D"font-family:monospac=
e">&#39;tramp-cleanup-bufferless-connections&#39; cleans up connection-rela=
ted objects for which no associated buffers exist.=C2=A0 This is helpful to=
 prune connections after you close remote-file buffers without having to ei=
ther cherry pick via &#39;tramp-cleanup-connection&#39; or clear them all v=
ia &#39;tramp-cleanup-all-connections&#39;.</div><div class=3D"gmail_defaul=
t" style=3D"font-family:monospace"><br></div><div class=3D"gmail_default" s=
tyle=3D"font-family:monospace">I looked for an equivalent but didn&#39;t se=
e one so I&#39;ve been using this command, and I think it&#39;s ready for c=
ontribution.</div><div class=3D"gmail_default" style=3D"font-family:monospa=
ce"><br></div><div class=3D"gmail_default" style=3D"font-family:monospace">=
-Stephane</div></div>

--0000000000004e4b21062e824fef--

--0000000000004e4b23062e824ff1
Content-Type: application/octet-stream; 
	name="0001-Add-tramp-cleanup-bufferless-connections.patch"
Content-Disposition: attachment; 
	filename="0001-Add-tramp-cleanup-bufferless-connections.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_m7c79dhu0>
X-Attachment-Id: f_m7c79dhu0

RnJvbSBjMzc0YmUwYTU3YzU5MzU5ODgxZjcyMTgzZTBiZjI0MWVhOTQ0YThjIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBzaGlwbWludHMgPHNoaXBtaW50c0BnbWFpbC5jb20+CkRhdGU6
IFdlZCwgMTkgRmViIDIwMjUgMTI6Mzk6MDkgLTA1MDAKU3ViamVjdDogW1BBVENIXSBBZGQgdHJh
bXAtY2xlYW51cC1idWZmZXJsZXNzLWNvbm5lY3Rpb25zLgoKKiBsaXNwL25ldC90cmFtcC1jbWRz
LmVsOgoodHJhbXAtY2xlYW51cC1idWZmZXJsZXNzLWNvbm5lY3Rpb25zKTogTmV3IGNvbW1hbmQu
Ci0tLQogZXRjL05FV1MgICAgICAgICAgICAgICB8ICA4ICsrKysrKysrCiBsaXNwL25ldC90cmFt
cC1jbWRzLmVsIHwgMzMgKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrCiAyIGZpbGVz
IGNoYW5nZWQsIDQxIGluc2VydGlvbnMoKykKCmRpZmYgLS1naXQgYS9ldGMvTkVXUyBiL2V0Yy9O
RVdTCmluZGV4IDc5YTQ4MjgwMTkxLi42NDA5YjcyMTEyNyAxMDA2NDQKLS0tIGEvZXRjL05FV1MK
KysrIGIvZXRjL05FV1MKQEAgLTg4OCw2ICs4ODgsMTQgQEAgd2UgaW52aXRlIEZseXNwZWxsIHVz
ZXJzIHRvIGVuYWJsZSB0aGlzIG5ldyBvcHRpb24gYW5kIHJlcG9ydCBpc3N1ZXMuCiAKICoqIFRy
YW1wCiAKKysrKworKioqIE5ldyBjb21tYW5kIHRyYW1wLWNsZWFudXAtYnVmZmVybGVzcy1jb25u
ZWN0aW9ucy4KK0Nvbm5lY3Rpb24tcmVsYXRlZCBvYmplY3RzIGZvciB3aGljaCBubyBhc3NvY2lh
dGVkIGJ1ZmZlcnMgZXhpc3QgYXJlCitmbHVzaGVkLiAgVGhpcyBpcyBoZWxwZnVsIHRvIHBydW5l
IGNvbm5lY3Rpb25zIGFmdGVyIHlvdSBjbG9zZQorcmVtb3RlLWZpbGUgYnVmZmVycyB3aXRob3V0
IGhhdmluZyB0byBlaXRoZXIgY2hlcnJ5IHBpY2sgdmlhCisndHJhbXAtY2xlYW51cC1jb25uZWN0
aW9uJyBvciBjbGVhciB0aGVtIGFsbCB2aWEKKyd0cmFtcC1jbGVhbnVwLWFsbC1jb25uZWN0aW9u
cycuCisKICsrKwogKioqIENvbm5lY3Rpb24gbWV0aG9kICJrdWJlcm5ldGVzIiBzdXBwb3J0cyBu
b3cgb3B0aW9uYWwgbmFtZXNwYWNlLgogVGhlIGhvc3QgbmFtZSBmb3IgS3ViZXJuZXRlcyBjb25u
ZWN0aW9ucyBjYW4gYmUgb2Yga2luZApkaWZmIC0tZ2l0IGEvbGlzcC9uZXQvdHJhbXAtY21kcy5l
bCBiL2xpc3AvbmV0L3RyYW1wLWNtZHMuZWwKaW5kZXggNzE4MjllODEwOTMuLjRhZDU0YmZlOTc3
IDEwMDY0NAotLS0gYS9saXNwL25ldC90cmFtcC1jbWRzLmVsCisrKyBiL2xpc3AvbmV0L3RyYW1w
LWNtZHMuZWwKQEAgLTIzOSw2ICsyMzksMzkgQEAgdHJhbXAtY2xlYW51cC1hbGwtY29ubmVjdGlv
bnMKICAgOzsgVGhlIGVuZC4KICAgKHJ1bi1ob29rcyAndHJhbXAtY2xlYW51cC1hbGwtY29ubmVj
dGlvbnMtaG9vaykpCiAKKzs7OyMjI3RyYW1wLWF1dG9sb2FkCisoZGVmdW4gdHJhbXAtY2xlYW51
cC1idWZmZXJsZXNzLWNvbm5lY3Rpb25zICgpCisgICJGbHVzaCBjb25uZWN0aW9uLXJlbGF0ZWQg
b2JqZWN0cyBmb3Igd2hpY2ggbm8gYnVmZmVyIGV4aXN0cy4KK0EgYnVmZmVybGVzcyBjb25uZWN0
aW9uIGlzIG9uZSBmb3Igd2hpY2ggbm8gbGl2ZSBidWZmZXIncworYGJ1ZmZlci1maWxlLW5hbWUn
IG9yIGBkZWZhdWx0LWRpcmVjdG9yeScgaXMgYXNzb2NpYXRlZCB3aXRoIHRoYXQKK2Nvbm5lY3Rp
b24uCitEaXNwbGF5IGEgbWVzc2FnZSBvZiBjbGVhbmVkLXVwIGNvbm5lY3Rpb25zLiIKKyAgKGlu
dGVyYWN0aXZlKQorICAobGV0KiAoKG5vbi10cmFtcC1idWZmZXJzIChzZXEtcmVtb3ZlCisgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgIChsYW1iZGEgKGIpCisgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgKG1lbWJlciAoYnVmZmVyLW5hbWUgYikKKyAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICh0cmFtcC1saXN0LXRyYW1wLWJ1ZmZlcnMpKSkKKyAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgKGJ1ZmZlci1saXN0KSkpCisgICAgICAgICBidWZmZXItY29u
bmVjdGlvbnMKKyAgICAgICAgIChwdXNoLWlmLXJlbW90ZSAobGFtYmRhIChmaWxlLW5hbWUpCisg
ICAgICAgICAgICAgICAgICAgICAgICAgICAod2hlbi1sZXQqICgoZmlsZS1yZW1vdGUKKyAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAoYW5kIGZpbGUtbmFtZQorICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgKGZpbGUtcmVtb3RlLXAgZmls
ZS1uYW1lKSkpKQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAoY2wtcHVzaG5ldyBmaWxl
LXJlbW90ZQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBidWZmZXIt
Y29ubmVjdGlvbnMgOnRlc3QgIydlcXVhbCkpKSkpCisgICAgKGRvbGlzdCAoYnVmZmVyIG5vbi10
cmFtcC1idWZmZXJzKQorICAgICAgKGZ1bmNhbGwgcHVzaC1pZi1yZW1vdGUgKGJ1ZmZlci1maWxl
LW5hbWUgYnVmZmVyKSkKKyAgICAgIDs7IEZvciBjb21pbnQgYnVmZmVycywgYW5kIHRob3NlIHdp
dGggYSBjaGFuZ2VkIGRlZmF1bHQtZGlyZWN0b3J5LgorICAgICAgKHdpdGgtY3VycmVudC1idWZm
ZXIgYnVmZmVyCisgICAgICAgIChmdW5jYWxsIHB1c2gtaWYtcmVtb3RlIGRlZmF1bHQtZGlyZWN0
b3J5KSkpCisgICAgKHdoZW4tbGV0KiAoKGJ1ZmZlcmxlc3MtY29ubmVjdGlvbnMKKyAgICAgICAg
ICAgICAgICAgKHNlcS1kaWZmZXJlbmNlCisgICAgICAgICAgICAgICAgICAobWFwY2FyICMndHJh
bXAtbWFrZS10cmFtcC1maWxlLW5hbWUgKHRyYW1wLWxpc3QtY29ubmVjdGlvbnMpKQorICAgICAg
ICAgICAgICAgICAgYnVmZmVyLWNvbm5lY3Rpb25zKSkpCisgICAgICAobWVzc2FnZSAiQ2xlYW5p
bmcgdXAgJXMiIChtYXBjb25jYXQgIydpZGVudGl0eSBidWZmZXJsZXNzLWNvbm5lY3Rpb25zICIs
ICIpKQorICAgICAgKGRvbGlzdCAoY29ubmVjdGlvbiBidWZmZXJsZXNzLWNvbm5lY3Rpb25zKQor
ICAgICAgICAodHJhbXAtY2xlYW51cC1jb25uZWN0aW9uICh0cmFtcC1kaXNzZWN0LWZpbGUtbmFt
ZSBjb25uZWN0aW9uICdub2V4cGFuZCkpKSkpKQorCiAoZGVmY3VzdG9tIHRyYW1wLWNsZWFudXAt
c29tZS1idWZmZXJzLWhvb2sgbmlsCiAgICJIb29rIGZvciBgdHJhbXAtY2xlYW51cC1zb21lLWJ1
ZmZlcnMnLgogVGhlIGZ1bmN0aW9ucyBkZXRlcm1pbmUgd2hpY2ggYnVmZmVycyBzaGFsbCBiZSBr
aWxsZWQuICBUaGlzCi0tIAoyLjQ3LjEKCg==
--0000000000004e4b23062e824ff1--

Message sent:

Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Mailer: MIME-tools 5.505 (Entity 5.505)
Content-Type: text/plain; charset=utf-8
X-Loop: help-debbugs@HIDDEN
From: help-debbugs@HIDDEN (GNU bug Tracking System)
To: Ship Mints <shipmints@HIDDEN>
Subject: bug#76417: Acknowledgement ([PATCH] Add tramp-cleanup-bufferless-connections)
Message-ID: <handler.76417.B.1739987084386.ack <at> debbugs.gnu.org>
References: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
X-Gnu-PR-Message: ack 76417
X-Gnu-PR-Package: emacs
X-Gnu-PR-Keywords: patch
Reply-To: 76417 <at> debbugs.gnu.org
Date: Wed, 19 Feb 2025 17:45:02 +0000

Thank you for filing a new bug report with debbugs.gnu.org.

This is an automatically generated reply to let you know your message
has been received.

Your message is being forwarded to the package maintainers and other
interested parties for their attention; they will reply in due course.

Your message has been sent to the package maintainer(s):
 bug-gnu-emacs@HIDDEN

If you wish to submit further information on this problem, please
send it to 76417 <at> debbugs.gnu.org.

Please do not send mail to help-debbugs@HIDDEN unless you wish
to report a problem with the Bug-tracking system.

--=20
76417: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=3D76417
GNU Bug Tracking System
Contact help-debbugs@HIDDEN with problems

Message sent to bug-gnu-emacs@HIDDEN:

Subject: bug#76417: [PATCH] Add tramp-cleanup-bufferless-connections
Resent-From: Michael Albinus <michael.albinus@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Thu, 20 Feb 2025 12:08:02 +0000
Resent-Message-ID: <handler.76417.B76417.17400532683113 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: Ship Mints <shipmints@HIDDEN>
Cc: 76417 <at> debbugs.gnu.org
From: Michael Albinus <michael.albinus@HIDDEN>
In-Reply-To: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
References: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
Date: Thu, 20 Feb 2025 13:07:36 +0100
Message-ID: <871pvsn7jr.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: text/plain
UI-OutboundReport: notjunk:1;M01:P0:cZy0QuutdGw=;GkCocOr7pJYMpx5hpBCt9AsZqRV
 2l/3CQhPinYm9O4PgMaj3ejstHYzWsmC84ZRt+WvqAWPcNgGVjfwAPRC8V+fvNHNHXHM7ZMGs
 /LOAQ2aOJW3key55U8Qh2Aejfo7q9MWQN0z41D1EfbB3sKMPp3NnIN6GXtqje+A/k/2mKzMDC
 wgyPlgdBc8LiJbg+kA3QhUUaqnmGAh2DO7uMjHrNN2J17S9yq1sl5hog3yk2gvpWy/xmRE5LK
 G2i1j/MzlUtLq8hsaTgE6P02A4sfPSIuZ9WznIm2Ve7c0mB0ac6FE33jTuk/h6ipjOyj8QBJf
 y3RO1pclzVyScYVXTtL0yC4HoRBJ6r8judF6gTu76SHlD05vmNIHy8YpILZFLRdEsz5D8w5Ly
 6wxYwLILq5oZYoGm797lTC6Ke5JLMcfu9StDqPS2CY0p/Jkeigae4BbjpoEtAQNxuXTzIj3WV
 M6DNot90nO+UA1KOL3O6qKENe9XCxfxpzMvBvjCOIYMKLxwFd/YCJE35EDakD6tMth5rBLsbv
 na2SKj0T2xY21o///oV9PFRB9hHr/VReTQhO0LSzUPZziXkZCFzanYwUqF1h1iBmUE6cflEau
 9a3TaaVFVoRsY4v3Kd4OyGOVhjmfUSqM7vQ0wxK71DSglsMptSbPNgXc1zkKkGhWTgE1U1nMz
 sHO9tmIeiV/ZZd/4Hybau2yqVBY4PgA+/BH3dJbQ7fOdEKGPset6m0jfMeMmmDHUcEe2DiKhM
 l5bS/TCMVmysXrEh4DpUWfudsQ8+p0lkOONPRQ8HOIIXPyQsxGROWQkeqwznB8Sjsl5rivifA
 e4su1g/pcs1icJcUibVD9VN/w2Jmr+6K+potjE5zvQvt9k6584VawTxfBJGcCq7Cw+qVNY+7X
 66OYMuHQbktoywSK82n+1qdnh0/yfpwt216GC69g95VDOXn77+aXHLsQylWm7Cqci39v5lnJo
 CJJxTwAq/B8Bk2BSwSM/jemS9eDi+5UW3ok/5FZ28kfL41wV6hcod2PTHm3wCJfnXUJ6Rx/6f
 KZg7Q6R6pHL9mq4J4lQE55IbN8tZ/sGtawmDlblC5DczKx+qCaFTEIjqgpFrTk6u4jQg9DrfD
 EmoKIyCBoPVcHXnhC2TGOnGC/BzKYHUbMVu54WW5MvgGjCz0V8bvRlM76+57rkBNxku4fDfpf
 hsgkuzHAb9r2+pXLvjhm27xX4FNOmOFbV6c+pihyoQ+HPQyu1cSia0irYoGJBeKng1DZM0f4s
 e8y76WUqNakmEaEaXWGSAMEdHe03JpP3+6gyc7APVSSN6aetgBHHWDgw+zVOFH2v1UYSVM91J
 +KXzLlAldC1QQwLgr7RccyEewjtRV9u9wqTtU8gWtk4ntBRakcQVm2Mu3O7BfL+fO5wAKu0Mv
 XmV5fl0qpDnN/3+Pvlju1GkkeF1QEJYK3SbOPMpMtjmvJxuI7UpRQamS7U
Precedence: list
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>

Ship Mints <shipmints@HIDDEN> writes:

Hi,

> 'tramp-cleanup-bufferless-connections' cleans up connection-related
> objects for which no associated buffers exist.  This is helpful to
> prune connections after you close remote-file buffers without having
> to either cherry pick via 'tramp-cleanup-connection' or clear them all
> via 'tramp-cleanup-all-connections'.

Thanks for this. However, I feel we have already too many
Tramp-cleanup-* commands; I fear another one would bring more
confusion. Nobody but the author would remember :-)

> I looked for an equivalent but didn't see one so I've been using this
> command, and I think it's ready for contribution.

For this, we have tramp-cleanup-some-buffers. Would it be possible to
implement your feature by adding a proper hook function in
tramp-cleanup-some-buffers-hook?

As a first change, I propose to extend tramp-list-remote-buffers like

--8<---------------cut here---------------start------------->8---
(defsubst tramp-get-buffer-file-name (buffer)
  "Return `buffer-file-name' of BUFFER."
  (buffer-local-value 'buffer-file-name buffer))

(defun tramp-list-remote-buffers ()
  "Return a list of all buffers with remote `default-directory'."
  (tramp-compat-seq-keep
   (lambda (x)
     (when (or (tramp-tramp-file-p (tramp-get-default-directory x))
	       (tramp-tramp-file-p (tramp-get-buffer-file-name x)))
       x))
   (buffer-list)))
--8<---------------cut here---------------end--------------->8---

Could you work on such a patch?

> -Stephane

Best regards, Michael.

Message sent to bug-gnu-emacs@HIDDEN:

Subject: bug#76417: [PATCH] Add tramp-cleanup-bufferless-connections
Resent-From: Ship Mints <shipmints@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Thu, 20 Feb 2025 12:51:02 +0000
Resent-Message-ID: <handler.76417.B76417.17400558462028 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: Michael Albinus <michael.albinus@HIDDEN>
Cc: 76417 <at> debbugs.gnu.org
MIME-Version: 1.0
References: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
 <871pvsn7jr.fsf@HIDDEN>
In-Reply-To: <871pvsn7jr.fsf@HIDDEN>
From: Ship Mints <shipmints@HIDDEN>
Date: Thu, 20 Feb 2025 07:50:25 -0500
Message-ID: <CAN+1HbrCL83C7MS8X=7MMDkw16q7toF_C0RFOG3NvRvOVw+2sg@HIDDEN>
Content-Type: multipart/alternative; boundary="000000000000089576062e925271"
Precedence: list
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>

--000000000000089576062e925271
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Feb 20, 2025 at 7:07=E2=80=AFAM Michael Albinus <michael.albinus@gm=
x.de>
wrote:

> Ship Mints <shipmints@HIDDEN> writes:
>
> > 'tramp-cleanup-bufferless-connections' cleans up connection-related
> > objects for which no associated buffers exist.  This is helpful to
> > prune connections after you close remote-file buffers without having
> > to either cherry pick via 'tramp-cleanup-connection' or clear them all
> > via 'tramp-cleanup-all-connections'.
>
> Thanks for this. However, I feel we have already too many
> Tramp-cleanup-* commands; I fear another one would bring more
> confusion. Nobody but the author would remember :-)
>

No worries.  I'll advertise this on support forums when it comes up, no
problem.  Someone had a problem just yesterday for which this could help.
I added this function to my buffer-list-update-hook to keep clean
automatically.

> I looked for an equivalent but didn't see one so I've been using this
> > command, and I think it's ready for contribution.
>
> For this, we have tramp-cleanup-some-buffers. Would it be possible to
> implement your feature by adding a proper hook function in
> tramp-cleanup-some-buffers-hook?
>

I did look. What concerned me was that tramp-cleanup-some-buffers-hook
could be altered by users, where tramp-cleanup-bufferless-connections does
one and only one thing without interference from potentially modified hooks=
.

As a first change, I propose to extend tramp-list-remote-buffers like
>
> --8<---------------cut here---------------start------------->8---
> (defsubst tramp-get-buffer-file-name (buffer)
>   "Return `buffer-file-name' of BUFFER."
>   (buffer-local-value 'buffer-file-name buffer))
>
> (defun tramp-list-remote-buffers ()
>   "Return a list of all buffers with remote `default-directory'."
>   (tramp-compat-seq-keep
>    (lambda (x)
>      (when (or (tramp-tramp-file-p (tramp-get-default-directory x))
>                (tramp-tramp-file-p (tramp-get-buffer-file-name x)))
>        x))
>    (buffer-list)))
> --8<---------------cut here---------------end--------------->8---
>
> Could you work on such a patch?
>

It looks to me, though, that tramp-cleanup-some-buffers does things
differently.  It does (tramp-cleanup-all-connections) which is not what I
want.  I want only to kill unused tramp connections, not all tramp
connections.  It also kills user buffers which is not what my function
does--it kills only tramp-related objects not user buffers.

I could take a second look today, but it doesn't seem like they solve the
same problem?

--000000000000089576062e925271
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr"><div class=3D"gmail_default" style=3D"fon=
t-family:monospace"><span style=3D"font-family:Arial,Helvetica,sans-serif">=
On Thu, Feb 20, 2025 at 7:07=E2=80=AFAM Michael Albinus &lt;<a href=3D"mail=
to:michael.albinus@HIDDEN">michael.albinus@HIDDEN</a>&gt; wrote:</span></di=
v></div><div class=3D"gmail_quote gmail_quote_container"><blockquote class=
=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rg=
b(204,204,204);padding-left:1ex">Ship Mints &lt;<a href=3D"mailto:shipmints=
@gmail.com" target=3D"_blank">shipmints@HIDDEN</a>&gt; writes:<br>
<br>
&gt; &#39;tramp-cleanup-bufferless-connections&#39; cleans up connection-re=
lated<br>
&gt; objects for which no associated buffers exist.=C2=A0 This is helpful t=
o<br>
&gt; prune connections after you close remote-file buffers without having<b=
r>
&gt; to either cherry pick via &#39;tramp-cleanup-connection&#39; or clear =
them all<br>
&gt; via &#39;tramp-cleanup-all-connections&#39;.<br>
<br>
Thanks for this. However, I feel we have already too many<br>
Tramp-cleanup-* commands; I fear another one would bring more<br>
confusion. Nobody but the author would remember :-)<br></blockquote><div><b=
r></div><div><div class=3D"gmail_default" style=3D"font-family:monospace">N=
o worries.=C2=A0 I&#39;ll advertise this on support forums when it comes up=
, no problem.=C2=A0 Someone had a problem just yesterday for which this cou=
ld help.=C2=A0 I added this function to my=C2=A0buffer-list-update-hook to =
keep clean automatically.</div></div><div><br></div><blockquote class=3D"gm=
ail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,=
204,204);padding-left:1ex">
&gt; I looked for an equivalent but didn&#39;t see one so I&#39;ve been usi=
ng this<br>
&gt; command, and I think it&#39;s ready for contribution.<br>
<br>
For this, we have tramp-cleanup-some-buffers. Would it be possible to<br>
implement your feature by adding a proper hook function in<br>
tramp-cleanup-some-buffers-hook?<br></blockquote><div><br></div><div><div c=
lass=3D"gmail_default" style=3D"font-family:monospace">I did look. What con=
cerned me was that tramp-cleanup-some-buffers-hook could be altered by user=
s, where tramp-cleanup-bufferless-connections does one and only one thing w=
ithout interference from potentially modified hooks.</div></div><div class=
=3D"gmail_default" style=3D"font-family:monospace"><br></div><blockquote cl=
ass=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid=
 rgb(204,204,204);padding-left:1ex">
As a first change, I propose to extend tramp-list-remote-buffers like<br>
<br>
--8&lt;---------------cut here---------------start-------------&gt;8---<br>
(defsubst tramp-get-buffer-file-name (buffer)<br>
=C2=A0 &quot;Return `buffer-file-name&#39; of BUFFER.&quot;<br>
=C2=A0 (buffer-local-value &#39;buffer-file-name buffer))<br>
<br>
(defun tramp-list-remote-buffers ()<br>
=C2=A0 &quot;Return a list of all buffers with remote `default-directory&#3=
9;.&quot;<br>
=C2=A0 (tramp-compat-seq-keep<br>
=C2=A0 =C2=A0(lambda (x)<br>
=C2=A0 =C2=A0 =C2=A0(when (or (tramp-tramp-file-p (tramp-get-default-direct=
ory x))<br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0(tramp-tramp-file-p =
(tramp-get-buffer-file-name x)))<br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0x))<br>
=C2=A0 =C2=A0(buffer-list)))<br>
--8&lt;---------------cut here---------------end---------------&gt;8---<br>
<br>
Could you work on such a patch?<br></blockquote><div><br></div><div><div cl=
ass=3D"gmail_default" style=3D"font-family:monospace">It looks to me, thoug=
h, that tramp-cleanup-some-buffers does things differently.=C2=A0 It does (=
tramp-cleanup-all-connections) which is not what I want.=C2=A0 I want only =
to kill unused tramp connections, not all tramp connections.=C2=A0 It also =
kills user buffers which is not what my function does--it kills only tramp-=
related objects not user buffers.</div></div><div class=3D"gmail_default" s=
tyle=3D"font-family:monospace"><br></div><div class=3D"gmail_default" style=
=3D"font-family:monospace">I could take a second look today, but it doesn&#=
39;t seem like they solve the same problem?</div></div></div>

--000000000000089576062e925271--

Message sent to bug-gnu-emacs@HIDDEN:

X-Loop: help-debbugs@HIDDEN
Subject: bug#76417: [PATCH] Add tramp-cleanup-bufferless-connections
Resent-From: Michael Albinus <michael.albinus@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Thu, 20 Feb 2025 14:04:02 +0000
Resent-Message-ID: <handler.76417.B76417.174006022928613 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
X-GNU-PR-Message: followup 76417
X-GNU-PR-Package: emacs
X-GNU-PR-Keywords: patch
To: Ship Mints <shipmints@HIDDEN>
Cc: 76417 <at> debbugs.gnu.org
Received: via spool by 76417-submit <at> debbugs.gnu.org id=B76417.174006022928613
          (code B ref 76417); Thu, 20 Feb 2025 14:04:02 +0000
Received: (at 76417) by debbugs.gnu.org; 20 Feb 2025 14:03:49 +0000
Received: from localhost ([127.0.0.1]:34946 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1tl79Y-0007RO-7I
	for submit <at> debbugs.gnu.org; Thu, 20 Feb 2025 09:03:48 -0500
Received: from mout.gmx.net ([212.227.15.19]:53143)
 by debbugs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.84_2) (envelope-from <michael.albinus@HIDDEN>)
 id 1tl79V-0007QM-L3
 for 76417 <at> debbugs.gnu.org; Thu, 20 Feb 2025 09:03:46 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmx.de;
 s=s31663417; t=1740060217; x=1740665017; i=michael.albinus@HIDDEN;
 bh=SufG7eEFdhpNLYA3R4CQWUcG2bctyE4zrplGBPHPWG4=;
 h=X-UI-Sender-Class:From:To:Cc:Subject:In-Reply-To:References:Date:
 Message-ID:MIME-Version:Content-Type:cc:content-transfer-encoding:
 content-type:date:from:message-id:mime-version:reply-to:subject:
 to;
 b=LP8mNWB+9k9tb54uUAo8puPX3oDJf0pS9OjjFJp7b326Dr/ueaqOWEX0Z/9TUCBE
 DGw9vaA3hpzLTgznytzpdsLMEdysdNyhQ+Q6NL0mskD8r47V8pOC19nw+tcQVxFSx
 NjoIwokOGVipR9WnrnRGx0iaY0Z07+E4y+laXtkVh8vAJWxXK4f8UmsuRWAUU8EcZ
 9tRlOWXRsCBCNtku6yQlFmBfig93/V7Suj8epadSeovRBAXBz3iByt/IgAAZwhfBA
 R77xa+W167tECEYiUCWEspRWRHLT2Z86O2tx3GKNSMsxTMYBNAOWGqXcuRirUN0VE
 p55mILJdigDsr0p61w==
X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a
Received: from gandalf.gmx.de ([185.89.38.155]) by mail.gmx.net (mrgmx005
 [212.227.17.190]) with ESMTPSA (Nemesis) id 1Mof5H-1t0jpY392n-00j1BI; Thu, 20
 Feb 2025 15:03:37 +0100
From: Michael Albinus <michael.albinus@HIDDEN>
In-Reply-To: <CAN+1HbrCL83C7MS8X=7MMDkw16q7toF_C0RFOG3NvRvOVw+2sg@HIDDEN>
References: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
 <871pvsn7jr.fsf@HIDDEN>
 <CAN+1HbrCL83C7MS8X=7MMDkw16q7toF_C0RFOG3NvRvOVw+2sg@HIDDEN>
Date: Thu, 20 Feb 2025 15:03:37 +0100
Message-ID: <87v7t4lnly.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: text/plain
X-Provags-ID: V03:K1:oNtqmcQD96FkkdElesID4P6XhDBjDuFRYzsN4RZ8Hx29vrIYdjD
 sSoZA2vi+8ekTNdkmJPMYXobAfNEk92s+pUNG+fTNAm46iVjxkzLam3dj2OSIg4LeUsJgU7
 HpgaIeyhKfJbiEQNTvpczxrl24ZE+inkIO3kIWOThf+cDriFWRq2mc6ZiTGpkn2pv9fnL6h
 sxP3kFRY/+ilTDz2gwB7Q==
X-Spam-Flag: NO
UI-OutboundReport: notjunk:1;M01:P0:S2gsiMKmUyY=;hadyDmXjLyG8Xf1UTCn6FtfLRcr
 6+2rPOPPWMe5tglP5xvnk4czdbaRi3Xfk/zSR8w4iNGyKGjKk7PAe0nbCjTXubV/KjPT77IBv
 mBgAQ+0tlugr25PvRce7rrxWS4x1kJio+X55J2VyiEv3l1dlnelXesBE6bpwQOLQiDU8p8hQh
 PdNOlT/459b409WzWdX4haEgD82oUKNvwEFwIixKVG4T9r5+HYkH80IdKyy/Rf7lXTynU9ZAH
 /W7qWNiQIdDrN70evMo4uI1YhivUKS9haRvzURAIWRPKji5PvaqAHnTTmVgYhKfrOd8NlZiVl
 etpI5k43zVrbNOjfB4kHED2tfQtQMYZx1qDdh1HBHfBl0yBL/dUVrdBguzAD0O+G8SuY+MpBA
 EqKmJAJlRt5UNWifuoAsgyMN1wS9TEXYZ2BsFpCjbx90bJesYfEE3NvmIov0kJA7gXllN+e+7
 L0l3Gz+tvxIyg2i68Mr0mSEu18ic555Txq/6FUKBZQnScz/drYFJD5up3vhdy1SWov+kwE2Vy
 +dvFnhJpPD1c7hMEd0lnlc3zdYOjG56I6sRusnh8Mr++dTCqoA/DDJy0E54ryP+bmrcSO8/+z
 GtIGVMHB6QpqOBuSAFHnyKUTdAGWR7PNCcrXSDGrrA0n0iax7SjWsPLJyng65CtSzHqoi5v9e
 8QshJGuTLAfYSacgUbQWe6waKD/TEFy1ucEDxVEl0hpaUVEl0t/1Xa0pL1G5r6rCw5W0xxS+V
 a/y/APRggLsUcPliVZMxv40daJ4cN1NoEAMaE5+jihtT6wecmPk+KdzcCHIync1zvo5NopGwq
 pIn1J7akTrCNu45E+000rhxT9td1McmpalQV9gC8b002JyOrHfJvn3yADtmDexCBWir48Iyzm
 hqtc3LXAhv2PjEZBzwHuzP2d+3to6SPJhH4ARNkw4tPkiwqg/Obp8gvc46QuFMwv0T10ibP/X
 wV74YWSZzlULGhfLE4urejju5Ped5DfTQyoI3qUCSzozWsR1A5Y9zb7Yg407BV4d1LGWc5cZj
 9T1gmH7zbSZ97viK/d8VPyV7hZMKnuhZaof98Ohi3IHJLC4KJTw8KwDuvwn6BkpXjgekK5r2a
 Z/GdB7gcMUmGsfGPzpcZtN/g+vcaloi6s0xc6UDl24Ld1N6+ywWT+GwJoV9Z67lQH8Cy0dK6j
 sWfQGoQNA2+3uvVhKn7UyuMD43rFVnq81Le9AAD3w39WGlb++YeDfY/3ufNpicQqWOr8YpPXm
 AYwvOqk03jdz3z1rAuVdzsch8pSbb/oLaYGCidCDsrIPyAaF0PMjw+yVXJLRAeKOvJo2PXah6
 WH0T/wn91D8CBU969cofDx1QbBAogZJrKaR3AZQGW00qA3Z+RL0EAA3dD2apGkERM2/Ty0JRz
 YQSAAl7w029qSqKJvzUMDFPZt8Jeo3dFGz/hQ1F42BJlMoH2lbcKTUjRsa
X-Spam-Score: -0.7 (/)
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.7 (-)

Ship Mints <shipmints@HIDDEN> writes:

Hi,

tramp-cleanup-some-buffers is intended as frame to be customized for
different use cases.

> I did look. What concerned me was that tramp-cleanup-some-buffers-hook
> could be altered by users, where tramp-cleanup-bufferless-connections
> does one and only one thing without interference from potentially
> modified hooks.

You can write a wrapper function for your package, which binds
tramp-cleanup-some-buffers-hook with proper functions, and which calls
tramp-cleanup-some-buffers then.

> It looks to me, though, that tramp-cleanup-some-buffers does things
> differently.

It is a new command of Emacs 30, so it is not very known to
users. There's much room to customize it for different use cases.

> It does (tramp-cleanup-all-connections) which is not
> what I want.  I want only to kill unused tramp connections, not all
> tramp connections.

I've seen this also today. Hmm, perhaps we can move this call to a hook
function, added by default, which you can suppress in your own binding
of tramp-cleanup-some-buffers-hook.

> It also kills user buffers which is not what my
> function does--it kills only tramp-related objects not user buffers.

This should also be configurable.

> I could take a second look today, but it doesn't seem like they solve
> the same problem?

Yes, please do. I'm interested in pimp up this command with alternative
scenarios.

Best regards, Michael.

Message sent to bug-gnu-emacs@HIDDEN:

X-Loop: help-debbugs@HIDDEN
Subject: bug#76417: [PATCH] Add tramp-cleanup-bufferless-connections
Resent-From: Ship Mints <shipmints@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Fri, 21 Feb 2025 12:35:02 +0000
Resent-Message-ID: <handler.76417.B76417.17401412691640 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
X-GNU-PR-Message: followup 76417
X-GNU-PR-Package: emacs
X-GNU-PR-Keywords: patch
To: Michael Albinus <michael.albinus@HIDDEN>
Cc: 76417 <at> debbugs.gnu.org
Received: via spool by 76417-submit <at> debbugs.gnu.org id=B76417.17401412691640
          (code B ref 76417); Fri, 21 Feb 2025 12:35:02 +0000
Received: (at 76417) by debbugs.gnu.org; 21 Feb 2025 12:34:29 +0000
Received: from localhost ([127.0.0.1]:56210 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1tlSEe-0000QJ-8q
	for submit <at> debbugs.gnu.org; Fri, 21 Feb 2025 07:34:29 -0500
Received: from mail-ua1-x92f.google.com ([2607:f8b0:4864:20::92f]:61650)
 by debbugs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128)
 (Exim 4.84_2) (envelope-from <shipmints@HIDDEN>)
 id 1tlSEb-0000Or-1t
 for 76417 <at> debbugs.gnu.org; Fri, 21 Feb 2025 07:34:25 -0500
Received: by mail-ua1-x92f.google.com with SMTP id
 a1e0cc1a2514c-868da0af0fcso612292241.2
 for <76417 <at> debbugs.gnu.org>; Fri, 21 Feb 2025 04:34:25 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=gmail.com; s=20230601; t=1740141259; x=1740746059; darn=debbugs.gnu.org;
 h=cc:to:subject:message-id:date:from:in-reply-to:references
 :mime-version:from:to:cc:subject:date:message-id:reply-to;
 bh=zPS/+Iwd1UHv/pUgqq7YzlE5Js2WyDj8/N/XO7oriUw=;
 b=A0unbty3xWUvgFxOMKn+bEt6W9g/OQPdfQrqk6r0/uQXLe159gysfQi07IiIYGjG3s
 1GC0XJ7o3gFF3qjvXhQreMH9A0z5jJkpGGeMOWIdWA1Pd/079HcdJxHmWJUuYWBTo7yX
 /Ohv34lgIxO+HVPkuLTpHe144+Mn73HI0Jr4H9+Vonc1F0K+6L7Yr1IKnMqVPwxQ0GgA
 vUWsVPN9hDNxTSrjvBOg7vyKpCgm22+0GbKKR0eLzChNrvy0v39DfT5PT6r/spNizQYV
 lE0yJICQwB96itXXrra9rGsYLzizSgdCi5+8dyykswKj5g/Up2VPn9yqz/U8bpL8nCBt
 8Npw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20230601; t=1740141259; x=1740746059;
 h=cc:to:subject:message-id:date:from:in-reply-to:references
 :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id
 :reply-to;
 bh=zPS/+Iwd1UHv/pUgqq7YzlE5Js2WyDj8/N/XO7oriUw=;
 b=Xip0nj4iRepsQP4M7w+dp7mCzNvmw21GJB6axDu/iE4lHC3ojCZRNyfJECGoMZQBwz
 1NgJ/nOb8AymbdfKz7JbTu4o/B1JLwXJgioZZtGX72BidaUf2V3BL0ZxeZ5Thq17zK7X
 WjlShF7qvCpgmpcGcko9DdeWf++p0pLaWCM5Y3twe55FLTyBYiV4BB/dx6bqb6Z5JmZh
 mDIPwDEbPtv9f04v5q80Oalpf3V+4ZA+a+o5QZMNvqVj0cK1MGuBG/tv18M+elslUi43
 PB0933aXmM93PhMA4u6Q0GXXTXNsXelq9eIjYuTomVCOWKqpxeOVe+QxvnxXWl3lpNqP
 cAXg==
X-Gm-Message-State: AOJu0YzxUe1vz0IMQ9kIzzBQphzCZCCjex9J3Skf4qnQMnS5mxOyEyP+
 QPM68ozstEnSEwF4XBczVT7kHn5nfgDc65iQ5KGduAosW2EqTE1h4ru66d81LypTzR5vDWmedXs
 lS7l4pMR1OyAQtaiiLlMd1TnjaKk=
X-Gm-Gg: ASbGncv7lT0Eq0suq+p2SeCRf87zQ0R4QndQWzgE6SIgRWIeCr+QkIKm74yyDZoUzMz
 TxeXbZ7NGtTe/97MzoMGjfsMvtEaIv4IzlLVJ445JS3orjnyc6WOiJ4qfWuEi7PmFinoYjZ+t+Z
 SFTZOP6OA=
X-Google-Smtp-Source: AGHT+IGwUZj9pOj92wSD3/uOoYsUgnmoDk6Oe2AXL8HbJDOozw0q+WRf6o5i8hw9Xff2YGCHKOZpu1c+8IJc5E8JiTU=
X-Received: by 2002:a05:6102:2912:b0:4af:c519:4e7f with SMTP id
 ada2fe7eead31-4bfc01d337amr1501291137.18.1740141259157; Fri, 21 Feb 2025
 04:34:19 -0800 (PST)
MIME-Version: 1.0
References: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
 <871pvsn7jr.fsf@HIDDEN>
 <CAN+1HbrCL83C7MS8X=7MMDkw16q7toF_C0RFOG3NvRvOVw+2sg@HIDDEN>
 <87v7t4lnly.fsf@HIDDEN>
In-Reply-To: <87v7t4lnly.fsf@HIDDEN>
From: Ship Mints <shipmints@HIDDEN>
Date: Fri, 21 Feb 2025 07:34:08 -0500
X-Gm-Features: AWEUYZkDD2s15mLxEOXXj8NdxOmsxmEIJtqk1oaD8z7dwawniGayiGPh1__GvrI
Message-ID: <CAN+1Hbr8oAxg2tcrsmEDMEhnMvgESxk70=JGMRKyh+=HJOL_Dg@HIDDEN>
Content-Type: multipart/alternative; boundary="0000000000009c77f2062ea63576"
X-Spam-Score: 0.0 (/)
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

--0000000000009c77f2062ea63576
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Feb 20, 2025 at 9:03=E2=80=AFAM Michael Albinus <michael.albinus@gm=
x.de>
wrote:

> Ship Mints <shipmints@HIDDEN> writes:
>
> Hi,
>
> tramp-cleanup-some-buffers is intended as frame to be customized for
> different use cases.
>
> > I did look. What concerned me was that tramp-cleanup-some-buffers-hook
> > could be altered by users, where tramp-cleanup-bufferless-connections
> > does one and only one thing without interference from potentially
> > modified hooks.
>
> You can write a wrapper function for your package, which binds
> tramp-cleanup-some-buffers-hook with proper functions, and which calls
> tramp-cleanup-some-buffers then.
>
> > It looks to me, though, that tramp-cleanup-some-buffers does things
> > differently.
>
> It is a new command of Emacs 30, so it is not very known to
> users. There's much room to customize it for different use cases.
>
> > It does (tramp-cleanup-all-connections) which is not
> > what I want.  I want only to kill unused tramp connections, not all
> > tramp connections.
>
> I've seen this also today. Hmm, perhaps we can move this call to a hook
> function, added by default, which you can suppress in your own binding
> of tramp-cleanup-some-buffers-hook.
>
> > It also kills user buffers which is not what my
> > function does--it kills only tramp-related objects not user buffers.
>
> This should also be configurable.
>
> > I could take a second look today, but it doesn't seem like they solve
> > the same problem?
>
> Yes, please do. I'm interested in pimp up this command with alternative
> scenarios.
>

I spent some more time on this and I think there's an "impedance mismatch"
between cleanup-some-buffers and cleaning up unused connections.  I'm
finding it cumbersome to mix the two as I have to move back and forth
between "buffer space" and "connection space" to achieve connection
cleanup.  My original patch seems like the cleanest approach, and can be
made shorter with a few tramp utility function changes.  Would you like to
see a patch based on those?

--0000000000009c77f2062ea63576
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr"><div class=3D"gmail_default" style=3D"fon=
t-family:monospace"><span style=3D"font-family:Arial,Helvetica,sans-serif">=
On Thu, Feb 20, 2025 at 9:03=E2=80=AFAM Michael Albinus &lt;<a href=3D"mail=
to:michael.albinus@HIDDEN">michael.albinus@HIDDEN</a>&gt; wrote:</span></di=
v></div><div class=3D"gmail_quote gmail_quote_container"><blockquote class=
=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rg=
b(204,204,204);padding-left:1ex">Ship Mints &lt;<a href=3D"mailto:shipmints=
@gmail.com" target=3D"_blank">shipmints@HIDDEN</a>&gt; writes:<br>
<br>
Hi,<br>
<br>
tramp-cleanup-some-buffers is intended as frame to be customized for<br>
different use cases.<br>
<br>
&gt; I did look. What concerned me was that tramp-cleanup-some-buffers-hook=
<br>
&gt; could be altered by users, where tramp-cleanup-bufferless-connections<=
br>
&gt; does one and only one thing without interference from potentially<br>
&gt; modified hooks.<br>
<br>
You can write a wrapper function for your package, which binds<br>
tramp-cleanup-some-buffers-hook with proper functions, and which calls<br>
tramp-cleanup-some-buffers then.<br>
<br>
&gt; It looks to me, though, that tramp-cleanup-some-buffers does things<br=
>
&gt; differently.<br>
<br>
It is a new command of Emacs 30, so it is not very known to<br>
users. There&#39;s much room to customize it for different use cases.<br>
<br>
&gt; It does (tramp-cleanup-all-connections) which is not<br>
&gt; what I want.=C2=A0 I want only to kill unused tramp connections, not a=
ll<br>
&gt; tramp connections.<br>
<br>
I&#39;ve seen this also today. Hmm, perhaps we can move this call to a hook=
<br>
function, added by default, which you can suppress in your own binding<br>
of tramp-cleanup-some-buffers-hook.<br>
<br>
&gt; It also kills user buffers which is not what my<br>
&gt; function does--it kills only tramp-related objects not user buffers.<b=
r>
<br>
This should also be configurable.<br>
<br>
&gt; I could take a second look today, but it doesn&#39;t seem like they so=
lve<br>
&gt; the same problem?<br>
<br>
Yes, please do. I&#39;m interested in pimp up this command with alternative=
<br>
scenarios.<br></blockquote><div><br></div><div><span style=3D"font-family:m=
onospace">I spent some more time on this and=C2=A0<span class=3D"gmail_defa=
ult" style=3D"font-family:monospace">I think there&#39;s an &quot;impedance=
 mismatch&quot; between cleanup-some-buffers and cleaning up unused connect=
ions.=C2=A0 I&#39;m finding it cumbersome to mix the two as I have to move =
back and forth between &quot;buffer space&quot; and &quot;connection space&=
quot; to achieve connection cleanup.=C2=A0 My original patch seems like the=
 cleanest approach, and can be made shorter with a few tramp utility functi=
on changes.=C2=A0 Would you like to see a patch based on those?</span></spa=
n></div></div></div>

--0000000000009c77f2062ea63576--

Message sent to bug-gnu-emacs@HIDDEN:

X-Loop: help-debbugs@HIDDEN
Subject: bug#76417: [PATCH] Add tramp-cleanup-bufferless-connections
Resent-From: Michael Albinus <michael.albinus@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: bug-gnu-emacs@HIDDEN
Resent-Date: Fri, 21 Feb 2025 13:25:02 +0000
Resent-Message-ID: <handler.76417.B76417.174014428031718 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
X-GNU-PR-Message: followup 76417
X-GNU-PR-Package: emacs
X-GNU-PR-Keywords: patch
To: Ship Mints <shipmints@HIDDEN>
Cc: 76417 <at> debbugs.gnu.org
Received: via spool by 76417-submit <at> debbugs.gnu.org id=B76417.174014428031718
          (code B ref 76417); Fri, 21 Feb 2025 13:25:02 +0000
Received: (at 76417) by debbugs.gnu.org; 21 Feb 2025 13:24:40 +0000
Received: from localhost ([127.0.0.1]:56919 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1tlT1D-0008FP-41
	for submit <at> debbugs.gnu.org; Fri, 21 Feb 2025 08:24:39 -0500
Received: from mout.gmx.net ([212.227.15.15]:42521)
 by debbugs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.84_2) (envelope-from <michael.albinus@HIDDEN>)
 id 1tlT1A-0008E7-4h
 for 76417 <at> debbugs.gnu.org; Fri, 21 Feb 2025 08:24:37 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmx.de;
 s=s31663417; t=1740144269; x=1740749069; i=michael.albinus@HIDDEN;
 bh=EBvfphfFCJ0IJoidTXmN/EDr9/ZvSL2M8yzTSC3c4a0=;
 h=X-UI-Sender-Class:From:To:Cc:Subject:In-Reply-To:References:Date:
 Message-ID:MIME-Version:Content-Type:cc:content-transfer-encoding:
 content-type:date:from:message-id:mime-version:reply-to:subject:
 to;
 b=iKnfuiYqOJBR276959ElEl3mpd7aftaxVIpOSt5qmx2gZtteVxjqTMtApOjrrqi7
 G3/0bsbPLX0t7b+JBVglNXjgJSDHzV6eLeuaiFZYD69R/J26EJqjdCbVacQGqI2Y2
 5Nv+5V2pyykJTbv71I7/7PvroDoG0+FAtn6xXp5RJbyjxLEA8cn+CUtF3JQdg4hsx
 0TPDtOrvdLWz1kXLqB8NVBDvrzItJM4VwalhkLFLdr05AaxKC7gqpWNBP1M0JUeSl
 ZLVoHx89OAy/6XwgV82nSV6xcW/Jhm1nL5avLDZFNKcbATAeoCLYIBq7D9w5w8TTE
 gRjBaSCoNhrrTz459w==
X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a
Received: from gandalf.gmx.de ([185.89.38.155]) by mail.gmx.net (mrgmx004
 [212.227.17.190]) with ESMTPSA (Nemesis) id 1MBUqL-1tcLPW0Czv-00G1xC; Fri, 21
 Feb 2025 14:24:29 +0100
From: Michael Albinus <michael.albinus@HIDDEN>
In-Reply-To: <CAN+1Hbr8oAxg2tcrsmEDMEhnMvgESxk70=JGMRKyh+=HJOL_Dg@HIDDEN>
References: <CAN+1Hbp9kCDJ-=XGc6dRE6oCGrvbiD1A=methnGhBa-F3+bMrA@HIDDEN>
 <871pvsn7jr.fsf@HIDDEN>
 <CAN+1HbrCL83C7MS8X=7MMDkw16q7toF_C0RFOG3NvRvOVw+2sg@HIDDEN>
 <87v7t4lnly.fsf@HIDDEN>
 <CAN+1Hbr8oAxg2tcrsmEDMEhnMvgESxk70=JGMRKyh+=HJOL_Dg@HIDDEN>
Date: Fri, 21 Feb 2025 14:24:27 +0100
Message-ID: <87jz9jwhv8.fsf@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: text/plain
X-Provags-ID: V03:K1:dzn7mwP/TAWr7oP9Y+M7t/JMxl7JVEDh4HCOatScX6GmkQ8r74f
 TBdeSynjvrPOgkvCHf8YUQsGRzr5VeR8XDbg+vOFuk8gWm+6Jvs0SZwwnCMKFoC+JiZstuF
 8AFE1mz4cbg35P3qGTUKR1iQA5R4Y8+nOSHAxrZ9jhtKHtfgOzHkU6g6rBK1QcFTy3NA6Hj
 VavpYjk0wSiTbhZeQDkaQ==
X-Spam-Flag: NO
UI-OutboundReport: notjunk:1;M01:P0:djiXQorsHVE=;6qIevlDJsd0aSbucJu3DpUIWYFI
 fGvxrjko9Wq84BDPN7U2bhRfnO7RiI2Ht5Tx/HY1NQGC0X6kyIW8XGwN1/EZziEg3xdOjyLqt
 bX8RvMQ4/eazmxO4kfjv82CyBthwZhpKyPmYiFAoL590Je9tjRO/qp89KLug6MfKONztawepE
 aU9jBQm25rrY29JF6eVxanDpxG4tefi0xs3wvgA3TbujTM+dXG0FdF6ZpItcrlRysl0wctpmR
 GgjZ7SSqacWI6oYjO4fQHnkc+CVDbICOGCl7KbdhxJfQN18znNWJhYPL/b2fxBPt0ZtTXIZWL
 jrSaY3mT9auWI0dME/bHu3Kna3cGIzdEE3/6jKBXVt81umvgmZ4vHNC95JwVGylt/5S6KOdW3
 OOT8oQzM5ZUoJwATEalQSzu6W6qFXzG3wS3/f+E7+DDNEVN+puEIfT8fcPxmiq+u8X3cRF2vq
 RB4p59ZVb9CM5hR0JkKLuo8LaOYXR3QDhImNlwfqFiBqjvqoFtbygZHJmbp8eqmBTdZjoK4ss
 UmRGCeJtjNfkJD/uhUpNI1nTAsw0uU0loqtAoIydIOUMpjb6B4VA+eMaZRuKROaErSX9/XicK
 fdTYGN0fwJMDsavn0nLVSJ7l1mNMx+YA3BllyVf98T3c+lTqC9Y14f0fxEpOjre6oyQfZQLeh
 xQBsJMlzvzQb3tOMyYI8fZIBhnFSicXchmMuG7aMgqI+qraZhb9wAocSE4hPGGaplWY8cgvAh
 ShlF9poJC/QEARHtE6vmTsPXGY2sOBD6gVtLiHtDngDq/6n9XIJi5KjIL6JMX3iiW4QYcIRt9
 nATeaP3ZhlXTnmWX2VuQAsjmWBTnuH9EpU27kRgCFZjmaGhmFWligQ65RP+b1ykJFoD+oP86C
 lvOPrYIG/u+8r6IO64rLvHJbGb/Ytj5aWRWG+qdbZByE7p0Ogb/N0ykhKv5OHFI54Chysz4Rp
 AHPAwnW72xvnIrUQNb/khNcBo4l2fcelYJkj02RKryxXRaCFXJ+J3X9fD/tK0/odqnkKlUMCE
 zg2vquyq05a52cKkQlKrSUJLUt8KdkvHY1LVGTtBEnKY6by2tiGkr52kBzgMVjLf5lORf47Wj
 nzGqrCbCVu3+bFV24/pTPEac2aXG1pInwBYJ01UaEG5tTRZTxS/aE2SGk/GTIjJmb0HhUTamm
 cp2jygl3xQT0lBOBkZGLOPAN+9egEmd4MOdKPcXPUXOnLqLFVH4V8ZNBgNnA7N5lQx2Gu1Uto
 lkanYaRQF3IfGTRX4WLvfCuPIB6LnT+LrE0aqS4OcFrxCOYVRkcJRpGQLGdu6YfgTxYpEQlPz
 or2ynky60V4P/E0iwLajNTuRENaWlVmsQ0k2Y6W20UnE1aZmZxlk/x3B7e3f8EZsMSjhhQSP9
 Nk71/K4XnNF3BwEQ9J/7lWwCEX6kk/DARZN3R0P5kAwtxJ/2IYdKXT86w3
X-Spam-Score: -0.7 (/)
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.7 (-)

Ship Mints <shipmints@HIDDEN> writes:

Hi,

> I spent some more time on this and I think there's an "impedance
> mismatch" between cleanup-some-buffers and cleaning up unused
> connections.  I'm finding it cumbersome to mix the two as I have to
> move back and forth between "buffer space" and "connection space" to
> achieve connection cleanup.  My original patch seems like the cleanest
> approach, and can be made shorter with a few tramp utility function
> changes.  Would you like to see a patch based on those?

Well, likely it will be a ship-only command in Tramp. But so what, show
it. If time permits (I hope so!), I'll play with your patch and see,
whether I could improve it in order to approach my target, use of
tramp-cleanup-some-buffers.

If I fail, I will install your patch.

Best regards, Michael.

Last modified: Fri, 21 Feb 2025 13:30:05 UTC

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