zsh-workers
 help / color / mirror / code / Atom feed
* test-repo-git-rebase-apply still in use?
@ 2024-08-23  6:59 Jörg Sommer
  2024-09-11  9:27 ` Daniel Shahaf
  0 siblings, 1 reply; 2+ messages in thread
From: Jörg Sommer @ 2024-08-23  6:59 UTC (permalink / raw)
  To: Zsh Workers

[-- Attachment #1: Type: text/plain, Size: 523 bytes --]

Hi,

is Functions/VCS_Info/test-repo-git-rebase-apply used anywhere? My git-grep
doesn't find any reference.

This file contains a hard-coded path for zsh (in the #!) which doesn't work
if it's installed somewhere else.

And is it useful to have this file on a common installation or should it get
excluded?

Regards Jörg

-- 
Geld allein macht nicht glücklich, aber es ist besser in einem Taxi zu
weinen, als in der Straßenbahn.
                                                (Marcel Reich‐Ranicki)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

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

* Re: test-repo-git-rebase-apply still in use?
  2024-08-23  6:59 test-repo-git-rebase-apply still in use? Jörg Sommer
@ 2024-09-11  9:27 ` Daniel Shahaf
  0 siblings, 0 replies; 2+ messages in thread
From: Daniel Shahaf @ 2024-09-11  9:27 UTC (permalink / raw)
  To: Jörg Sommer, Zsh Workers

Jörg Sommer wrote on Fri, 23 Aug 2024 06:59 +00:00:
> is Functions/VCS_Info/test-repo-git-rebase-apply used anywhere? My git-grep
> doesn't find any reference.

It's essentially a setup function for a manual test suite.  It's not user-facing,
but a developer might invoke it manually to test a patch being worked on.

> This file contains a hard-coded path for zsh (in the #!) which doesn't work
> if it's installed somewhere else.

That path doesn't exist on my system either.  Having it still helps
$EDITOR default to the right syntax highlighting, though.

I suppose it could be changed to use /usr/bin/env, or even /bin/sh but
for the use of ${foo:P} on line 47.

> And is it useful to have this file on a common installation or should it get
> excluded?

The latter.  And is there any way to make the script's purpose clearer?

Cheers,

Daniel


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

end of thread, other threads:[~2024-09-11  9:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-08-23  6:59 test-repo-git-rebase-apply still in use? Jörg Sommer
2024-09-11  9:27 ` Daniel Shahaf

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

	https://git.vuxu.org/mirror/zsh/

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 NNTP newsgroup(s).