* [BUG] gitk assumes availability of `osascript` on macos
@ 2025-09-20 11:59 Robert Scott
2025-09-20 19:52 ` Eric Sunshine
0 siblings, 1 reply; 4+ messages in thread
From: Robert Scott @ 2025-09-20 11:59 UTC (permalink / raw)
To: git
Hi,
Tiny one here. On macos, gitk (tested 2.50.1) attempts to call `osascript` so
it can focus/raise the UI. But it fails hard if `osascript` isn't present in
the $PATH.
There are a number of reasons `osascript` might not be available in a
particular environment, e.g. if launched from inside a "pure" nix shell, but
seeing as it's just a quality-of-life feature it would be nice if failures to
call `osascript` could just be ignored.
Thanks,
robert.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [BUG] gitk assumes availability of `osascript` on macos
2025-09-20 11:59 [BUG] gitk assumes availability of `osascript` on macos Robert Scott
@ 2025-09-20 19:52 ` Eric Sunshine
2025-09-22 6:37 ` Johannes Sixt
0 siblings, 1 reply; 4+ messages in thread
From: Eric Sunshine @ 2025-09-20 19:52 UTC (permalink / raw)
To: lists; +Cc: git, Johannes Sixt
On Sat, Sep 20, 2025 at 8:07 AM Robert Scott <lists@humanleg.org.uk> wrote:
> Tiny one here. On macos, gitk (tested 2.50.1) attempts to call `osascript` so
> it can focus/raise the UI. But it fails hard if `osascript` isn't present in
> the $PATH.
>
> There are a number of reasons `osascript` might not be available in a
> particular environment, e.g. if launched from inside a "pure" nix shell, but
> seeing as it's just a quality-of-life feature it would be nice if failures to
> call `osascript` could just be ignored.
A patch[*] was posted to the mailing list years ago which, I believe,
should fix this problem for you. Unfortunately, the patch was never
picked up due to the then-maintainer of Gitk being missing-in-action.
Perhaps the current Gitk maintainer (Cc:'d) will pick up the patch.
[*]: https://lore.kernel.org/git/20180724065120.7664-1-sunshine@sunshineco.com/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [BUG] gitk assumes availability of `osascript` on macos
2025-09-20 19:52 ` Eric Sunshine
@ 2025-09-22 6:37 ` Johannes Sixt
2025-09-22 9:30 ` Eric Sunshine
0 siblings, 1 reply; 4+ messages in thread
From: Johannes Sixt @ 2025-09-22 6:37 UTC (permalink / raw)
To: Eric Sunshine, lists; +Cc: git
Am 20.09.25 um 21:52 schrieb Eric Sunshine:
> On Sat, Sep 20, 2025 at 8:07 AM Robert Scott <lists@humanleg.org.uk> wrote:
>> Tiny one here. On macos, gitk (tested 2.50.1) attempts to call `osascript` so
>> it can focus/raise the UI. But it fails hard if `osascript` isn't present in
>> the $PATH.
>>
>> There are a number of reasons `osascript` might not be available in a
>> particular environment, e.g. if launched from inside a "pure" nix shell, but
>> seeing as it's just a quality-of-life feature it would be nice if failures to
>> call `osascript` could just be ignored.
>
> A patch[*] was posted to the mailing list years ago which, I believe,
> should fix this problem for you. Unfortunately, the patch was never
> picked up due to the then-maintainer of Gitk being missing-in-action.
> Perhaps the current Gitk maintainer (Cc:'d) will pick up the patch.
>
> [*]: https://lore.kernel.org/git/20180724065120.7664-1-sunshine@sunshineco.com/
I've picked up the patch:
https://github.com/j6t/gitk/compare/master...es/ignore-osascript-failure
Please test and suggest improvements. Should the links in the commit
message be redirected to lore.kernel.org?
-- Hannes
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [BUG] gitk assumes availability of `osascript` on macos
2025-09-22 6:37 ` Johannes Sixt
@ 2025-09-22 9:30 ` Eric Sunshine
0 siblings, 0 replies; 4+ messages in thread
From: Eric Sunshine @ 2025-09-22 9:30 UTC (permalink / raw)
To: Johannes Sixt; +Cc: lists, git
On Mon, Sep 22, 2025 at 2:37 AM Johannes Sixt <j6t@kdbg.org> wrote:
> Am 20.09.25 um 21:52 schrieb Eric Sunshine:
> > On Sat, Sep 20, 2025 at 8:07 AM Robert Scott <lists@humanleg.org.uk> wrote:
> >> Tiny one here. On macos, gitk (tested 2.50.1) attempts to call `osascript` so
> >> it can focus/raise the UI. But it fails hard if `osascript` isn't present in
> >> the $PATH.
> >>
> >> There are a number of reasons `osascript` might not be available in a
> >> particular environment, e.g. if launched from inside a "pure" nix shell, but
> >> seeing as it's just a quality-of-life feature it would be nice if failures to
> >> call `osascript` could just be ignored.
> >
> > A patch[*] was posted to the mailing list years ago which, I believe,
> > should fix this problem for you. Unfortunately, the patch was never
> > picked up due to the then-maintainer of Gitk being missing-in-action.
> > Perhaps the current Gitk maintainer (Cc:'d) will pick up the patch.
> >
> > [*]: https://lore.kernel.org/git/20180724065120.7664-1-sunshine@sunshineco.com/
>
> I've picked up the patch:
> https://github.com/j6t/gitk/compare/master...es/ignore-osascript-failure
Thank you. Looks good to me.
> Please test and suggest improvements. Should the links in the commit
> message be redirected to lore.kernel.org?
Updating the links would not be a bad idea. Here are the updated links
if you want to amend locally:
[1]: https://lore.kernel.org/git/D295145E-7596-4409-9681-D8ADBB9EBB0C@me.com/
[2]: https://lore.kernel.org/git/CABNJ2G+h3zh+=wLA0KHjUn8TsfhqUK1Kn-1_=6hnXVRJUPhuuA@mail.gmail.com/
Otherwise, I can resend.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-09-22 9:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-20 11:59 [BUG] gitk assumes availability of `osascript` on macos Robert Scott
2025-09-20 19:52 ` Eric Sunshine
2025-09-22 6:37 ` Johannes Sixt
2025-09-22 9:30 ` Eric Sunshine
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).