* [PATCH] git-gui: bring Wish process to front on Mac
@ 2013-06-06 8:17 Stefan Haller
2013-06-06 14:09 ` Pat Thoyts
0 siblings, 1 reply; 8+ messages in thread
From: Stefan Haller @ 2013-06-06 8:17 UTC (permalink / raw)
To: Pat Thoyts; +Cc: git
On Mac OS X, any application that is started from the Terminal will open
behind all running applications; as a work-around, manually bring ourselves
to the front. (Stolen from gitk, commit 76bf6ff93e.)
We do this as the very first thing, so that any message boxes that might pop
up during the rest of the startup sequence are actually seen by the user.
Signed-off-by: Stefan Haller <stefan@haller-berlin.de>
---
git-gui.sh | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/git-gui.sh b/git-gui.sh
index e133331..c464928 100755
--- a/git-gui.sh
+++ b/git-gui.sh
@@ -29,6 +29,19 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA}]
######################################################################
##
+## On Mac, bring the current Wish process window to front
+
+if {[tk windowingsystem] eq "aqua"} {
+ exec osascript -e [format {
+ tell application "System Events"
+ set frontmost of processes whose unix id is %d to true
+ end tell
+ } [pid] ]
+}
+
+
+######################################################################
+##
## Tcl/Tk sanity check
if {[catch {package require Tcl 8.4} err]
--
1.8.3.14.g33f718c
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] git-gui: bring Wish process to front on Mac
2013-06-06 8:17 [PATCH] git-gui: bring Wish process to front on Mac Stefan Haller
@ 2013-06-06 14:09 ` Pat Thoyts
2013-06-06 17:12 ` Stefan Haller
2013-06-06 18:15 ` [PATCH] " Junio C Hamano
0 siblings, 2 replies; 8+ messages in thread
From: Pat Thoyts @ 2013-06-06 14:09 UTC (permalink / raw)
To: Stefan Haller; +Cc: Git Mailing List
On 6 June 2013 09:17, Stefan Haller <stefan@haller-berlin.de> wrote:
> On Mac OS X, any application that is started from the Terminal will open
> behind all running applications; as a work-around, manually bring ourselves
> to the front. (Stolen from gitk, commit 76bf6ff93e.)
>
> We do this as the very first thing, so that any message boxes that might pop
> up during the rest of the startup sequence are actually seen by the user.
>
> Signed-off-by: Stefan Haller <stefan@haller-berlin.de>
> ---
> git-gui.sh | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/git-gui.sh b/git-gui.sh
> index e133331..c464928 100755
> --- a/git-gui.sh
> +++ b/git-gui.sh
> @@ -29,6 +29,19 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA}]
>
> ######################################################################
> ##
> +## On Mac, bring the current Wish process window to front
> +
> +if {[tk windowingsystem] eq "aqua"} {
> + exec osascript -e [format {
> + tell application "System Events"
> + set frontmost of processes whose unix id is %d to true
> + end tell
> + } [pid] ]
> +}
> +
> +
> +######################################################################
> +##
> ## Tcl/Tk sanity check
>
> if {[catch {package require Tcl 8.4} err]
> --
> 1.8.3.14.g33f718c
>
Seems fine to me. I can't test this as I have no access to this
platform. Possibly you should run this in a catch statement so it can
ignore any errors and I would tend to use the 'auto_execok' command to
ensure that osascript actually exists. Something like
set arg [format {tell application......}]
catch {exec {*}[auto_execok osascript] -e $arg [pid]}
but possibly this is guaranteed to exist on all macs which would make
the above redundant. What I'm thinking is you dont want the app to
exit just because something goes wrong in this call.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] git-gui: bring Wish process to front on Mac
2013-06-06 14:09 ` Pat Thoyts
@ 2013-06-06 17:12 ` Stefan Haller
2013-06-07 21:56 ` [PATCH v2] " Stefan Haller
2013-06-06 18:15 ` [PATCH] " Junio C Hamano
1 sibling, 1 reply; 8+ messages in thread
From: Stefan Haller @ 2013-06-06 17:12 UTC (permalink / raw)
To: Pat Thoyts; +Cc: Paul Mackerras, Git Mailing List
Pat Thoyts <patthoyts@gmail.com> wrote:
> On 6 June 2013 09:17, Stefan Haller <stefan@haller-berlin.de> wrote:
> > +## On Mac, bring the current Wish process window to front
> > +
> > +if {[tk windowingsystem] eq "aqua"} {
> > + exec osascript -e [format {
> > + tell application "System Events"
> > + set frontmost of processes whose unix id is %d to true
> > + end tell
> > + } [pid] ]
> > +}
>
> Seems fine to me. I can't test this as I have no access to this
> platform. Possibly you should run this in a catch statement so it can
> ignore any errors and I would tend to use the 'auto_execok' command to
> ensure that osascript actually exists. Something like
>
> set arg [format {tell application......}]
> catch {exec {*}[auto_execok osascript] -e $arg [pid]}
>
> but possibly this is guaranteed to exist on all macs which would make
> the above redundant. What I'm thinking is you dont want the app to
> exit just because something goes wrong in this call.
I don't think we need auto_execok here, as osascript is available on
every Mac system. We might even specify the exact path, it's always
/usr/bin/osascript. Is that preferable?
I agree that "catch" might be a good idea though. It raises two
questions though:
1) Should we make the same change in gitk then? It already has the same
code without the catch (released in 1.8.3 already, btw).
2) Should we think about some way of sharing code between gitk and
git gui, so that these kinds of changes don't have to be made twice?
--
Stefan Haller
Berlin, Germany
http://www.haller-berlin.de/
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2] git-gui: bring Wish process to front on Mac
2013-06-06 17:12 ` Stefan Haller
@ 2013-06-07 21:56 ` Stefan Haller
2013-06-14 17:54 ` Junio C Hamano
0 siblings, 1 reply; 8+ messages in thread
From: Stefan Haller @ 2013-06-07 21:56 UTC (permalink / raw)
To: Pat Thoyts; +Cc: git
On Mac OS X, any application that is started from the Terminal will open
behind all running applications; as a work-around, manually bring ourselves
to the front. (Stolen from gitk, commit 76bf6ff93e.)
We do this as the very first thing, so that any message boxes that might pop
up during the rest of the startup sequence are actually seen by the user.
Signed-off-by: Stefan Haller <stefan@haller-berlin.de>
---
Changes since the first patch:
- add catch
- specify full path to /usr/bin/osascript
git-gui.sh | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/git-gui.sh b/git-gui.sh
index e133331..a792924 100755
--- a/git-gui.sh
+++ b/git-gui.sh
@@ -29,6 +29,21 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA}]
######################################################################
##
+## On Mac, bring the current Wish process window to front
+
+if {[tk windowingsystem] eq "aqua"} {
+ catch {
+ exec /usr/bin/osascript -e [format {
+ tell application "System Events"
+ set frontmost of processes whose unix id is %d to true
+ end tell
+ } [pid] ]
+ }
+}
+
+
+######################################################################
+##
## Tcl/Tk sanity check
if {[catch {package require Tcl 8.4} err]
--
1.8.3.14.g33f718c
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2] git-gui: bring Wish process to front on Mac
2013-06-07 21:56 ` [PATCH v2] " Stefan Haller
@ 2013-06-14 17:54 ` Junio C Hamano
2013-06-15 23:07 ` Pat Thoyts
0 siblings, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2013-06-14 17:54 UTC (permalink / raw)
To: Pat Thoyts; +Cc: Stefan Haller, git
Stefan Haller <stefan@haller-berlin.de> writes:
> On Mac OS X, any application that is started from the Terminal will open
> behind all running applications; as a work-around, manually bring ourselves
> to the front. (Stolen from gitk, commit 76bf6ff93e.)
>
> We do this as the very first thing, so that any message boxes that might pop
> up during the rest of the startup sequence are actually seen by the user.
>
> Signed-off-by: Stefan Haller <stefan@haller-berlin.de>
> ---
Pat, is there any progress on this? I do not mind, and I actually
would prefer, a pull request early in the development cycle.
"I've been busy and haven't got around to it", is a perfectly
acceptable response. Just pinging to check.
> Changes since the first patch:
> - add catch
> - specify full path to /usr/bin/osascript
>
> git-gui.sh | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/git-gui.sh b/git-gui.sh
> index e133331..a792924 100755
> --- a/git-gui.sh
> +++ b/git-gui.sh
> @@ -29,6 +29,21 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA}]
>
> ######################################################################
> ##
> +## On Mac, bring the current Wish process window to front
> +
> +if {[tk windowingsystem] eq "aqua"} {
> + catch {
> + exec /usr/bin/osascript -e [format {
> + tell application "System Events"
> + set frontmost of processes whose unix id is %d to true
> + end tell
> + } [pid] ]
> + }
> +}
> +
> +
> +######################################################################
> +##
> ## Tcl/Tk sanity check
>
> if {[catch {package require Tcl 8.4} err]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] git-gui: bring Wish process to front on Mac
2013-06-14 17:54 ` Junio C Hamano
@ 2013-06-15 23:07 ` Pat Thoyts
0 siblings, 0 replies; 8+ messages in thread
From: Pat Thoyts @ 2013-06-15 23:07 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Stefan Haller, Git Mailing List
On 14 June 2013 18:54, Junio C Hamano <gitster@pobox.com> wrote:
> Stefan Haller <stefan@haller-berlin.de> writes:
>
>> On Mac OS X, any application that is started from the Terminal will open
>> behind all running applications; as a work-around, manually bring ourselves
>> to the front. (Stolen from gitk, commit 76bf6ff93e.)
>>
>> We do this as the very first thing, so that any message boxes that might pop
>> up during the rest of the startup sequence are actually seen by the user.
>>
>> Signed-off-by: Stefan Haller <stefan@haller-berlin.de>
>> ---
>
> Pat, is there any progress on this? I do not mind, and I actually
> would prefer, a pull request early in the development cycle.
>
yep - I applied this and a couple of others and sent up a pull request
now. I see there are some commits in git's tree that I don't have on
this side so I'll merge those in here shortly.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] git-gui: bring Wish process to front on Mac
2013-06-06 14:09 ` Pat Thoyts
2013-06-06 17:12 ` Stefan Haller
@ 2013-06-06 18:15 ` Junio C Hamano
2013-06-07 21:58 ` Stefan Haller
1 sibling, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2013-06-06 18:15 UTC (permalink / raw)
To: Pat Thoyts; +Cc: Stefan Haller, Git Mailing List
Pat Thoyts <patthoyts@gmail.com> writes:
> Seems fine to me. I can't test this as I have no access to this
> platform. Possibly you should run this in a catch statement so it can
> ignore any errors and I would tend to use the 'auto_execok' command to
> ensure that osascript actually exists. Something like
>
> set arg [format {tell application......}]
> catch {exec {*}[auto_execok osascript] -e $arg [pid]}
>
> but possibly this is guaranteed to exist on all macs which would make
> the above redundant. What I'm thinking is you dont want the app to
> exit just because something goes wrong in this call.
Sounds like a sensible concern; gitk in v1.8.3 has it without catch
so we may see failure reports from OSX users soonish, in which case
both this patch and gitk need to be updated for it.
Stefan (as your name appears in 76bf6ff93e, I am assuming that you
were the OSX-osascript guru in that commit) could you keep an eye on
the list traffic to see if users of latest gitk have issues with
that change, please?
Thanks.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-06-15 23:07 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-06 8:17 [PATCH] git-gui: bring Wish process to front on Mac Stefan Haller
2013-06-06 14:09 ` Pat Thoyts
2013-06-06 17:12 ` Stefan Haller
2013-06-07 21:56 ` [PATCH v2] " Stefan Haller
2013-06-14 17:54 ` Junio C Hamano
2013-06-15 23:07 ` Pat Thoyts
2013-06-06 18:15 ` [PATCH] " Junio C Hamano
2013-06-07 21:58 ` Stefan Haller
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.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).