Re: [PATCH v2 resend] gitk: Use an external icon file on Windows

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: [PATCH v2 resend] gitk: Use an external icon file on Windows

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:54:27

Sebastian Schuberth [off-list ref] writes:
Git for Windows now ships with the new Git icon from git-scm.com. Use that
icon file instead of the old procedurally drawn one if it exists.

Signed-off-by: Sebastian Schuberth <redacted>
---
Forwarding a misdirected patch to the maintainer who is free to pick
or ignore.

Personally I am negative on it (nobody on the list asked for the
"new" Git icon as far as I recall), but my voice on this counts just
as little as others.

Thanks.
quoted hunk
 gitk-git/gitk | 49 ++++++++++++++++++++++++++-----------------------
 1 file changed, 26 insertions(+), 23 deletions(-)
diff --git a/gitk-git/gitk b/gitk-git/gitk
index 59693c0..5127e55 100755
--- a/gitk-git/gitk
+++ b/gitk-git/gitk
@@ -11664,7 +11664,6 @@ if { [info exists ::env(GITK_MSGSDIR)] } {
     set gitk_prefix [file dirname [file dirname [file normalize $argv0]]]
     set gitk_libdir [file join $gitk_prefix share gitk lib]
     set gitk_msgsdir [file join $gitk_libdir msgs]
-    unset gitk_prefix
 }
 
 ## Internationalization (i18n) through msgcat and gettext. See
@@ -11821,28 +11820,32 @@ if {[expr {[exec git rev-parse --is-inside-work-tree] == "true"}]} {
 set worktree [exec git rev-parse --show-toplevel]
 setcoords
 makewindow
-catch {
-    image create photo gitlogo      -width 16 -height 16
-
-    image create photo gitlogominus -width  4 -height  2
-    gitlogominus put #C00000 -to 0 0 4 2
-    gitlogo copy gitlogominus -to  1 5
-    gitlogo copy gitlogominus -to  6 5
-    gitlogo copy gitlogominus -to 11 5
-    image delete gitlogominus
-
-    image create photo gitlogoplus  -width  4 -height  4
-    gitlogoplus  put #008000 -to 1 0 3 4
-    gitlogoplus  put #008000 -to 0 1 4 3
-    gitlogo copy gitlogoplus  -to  1 9
-    gitlogo copy gitlogoplus  -to  6 9
-    gitlogo copy gitlogoplus  -to 11 9
-    image delete gitlogoplus
-
-    image create photo gitlogo32    -width 32 -height 32
-    gitlogo32 copy gitlogo -zoom 2 2
-
-    wm iconphoto . -default gitlogo gitlogo32
+if {$::tcl_platform(platform) eq {windows} && [file exists $gitk_prefix/etc/git.ico]} {
+    wm iconbitmap . -default $gitk_prefix/etc/git.ico
+} else {
+    catch {
+        image create photo gitlogo      -width 16 -height 16
+
+        image create photo gitlogominus -width  4 -height  2
+        gitlogominus put #C00000 -to 0 0 4 2
+        gitlogo copy gitlogominus -to  1 5
+        gitlogo copy gitlogominus -to  6 5
+        gitlogo copy gitlogominus -to 11 5
+        image delete gitlogominus
+
+        image create photo gitlogoplus  -width  4 -height  4
+        gitlogoplus  put #008000 -to 1 0 3 4
+        gitlogoplus  put #008000 -to 0 1 4 3
+        gitlogo copy gitlogoplus  -to  1 9
+        gitlogo copy gitlogoplus  -to  6 9
+        gitlogo copy gitlogoplus  -to 11 9
+        image delete gitlogoplus
+
+        image create photo gitlogo32    -width 32 -height 32
+        gitlogo32 copy gitlogo -zoom 2 2
+
+        wm iconphoto . -default gitlogo gitlogo32
+    }
 }
 # wait for the window to become visible
 tkwait visibility .

Re: [PATCH v2 resend] gitk: Use an external icon file on Windows

From: Sebastian Schuberth <hidden>
Date: 2016-06-15 22:54:27

On Wed, Aug 8, 2012 at 11:13 PM, Junio C Hamano [off-list ref] wrote:
Forwarding a misdirected patch to the maintainer who is free to pick
or ignore.
How am I supposed to know if a patch has been ignored as an oversight
(in which case I would resend), or because the maintainer decided not
to include it (in which case I would not resend in order to not annoy
the maintainer)?
Personally I am negative on it (nobody on the list asked for the
"new" Git icon as far as I recall), but my voice on this counts just
as little as others.
I guess most patches on the list come in unasked, so I cannot follow
your negative attitude because of this. Also, the patch does not
enforce the new icon on any platform. From a user's perspective, [1]
as mentioned on [2] is the official Git homepage, thus the icon used
on the homepage can be regarded as the official Git icon. In Git for
Windows, we try to have a consistent user experience, and we had a
report about gitk still using the "old" icon [3]. This patch fixes
that, and I'm sending it upstream in an effort to not let msysgit's
fork of git diverge too much from upstream.

[1] http://git-scm.com/
[2] http://en.wikipedia.org/wiki/Git_%28software%29#External_links
[3] https://github.com/msysgit/msysgit/issues/44

-- 
Sebastian Schuberth
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help