Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.2 does not work with gitk and dot #6

Open
trajano opened this issue Sep 10, 2019 · 16 comments
Open

1.2 does not work with gitk and dot #6

trajano opened this issue Sep 10, 2019 · 16 comments
Assignees
Labels
bug Something isn't working

Comments

@trajano
Copy link

trajano commented Sep 10, 2019

I reshimmed and found out that gitk no longer functions with the new shim. Reverting back to the original shims brought the functionality back.

@71
Copy link
Owner

71 commented Sep 11, 2019

What do you mean by "no longer functions"? I can launch it through the shim without a problem here.

@trajano
Copy link
Author

trajano commented Sep 11, 2019

When I type gitk from the command line it does not show the gitk window

@trajano
Copy link
Author

trajano commented Sep 11, 2019

This is with 1.2 release

@71 71 added the bug Something isn't working label Sep 11, 2019
@trajano
Copy link
Author

trajano commented Sep 11, 2019

@71 were you able to replicate the bug?

@rasa
Copy link
Contributor

rasa commented Sep 13, 2019

I am using https://github.com/71/scoop-better-shimexe/releases/download/1.2/shim.exe and gitk works just fine from the command line on:

OS Name:                   Microsoft Windows 10 Pro for Workstations
OS Version:                10.0.18362 N/A Build 18362
System Type:               x64-based PC
Hotfix(s):                 4 Hotfix(s) Installed.
                           [01]: KB4506991
                           [02]: KB4503308
                           [03]: KB4509096
                           [04]: KB4507453

@trajano
Copy link
Author

trajano commented Sep 13, 2019

I'm on

C:\Users\trajano>ver

Microsoft Windows [Version 10.0.18947.1000]

I am running from "Windows Terminal"

@rasa
Copy link
Contributor

rasa commented Sep 13, 2019

Works for me with cmd.exe and under Windows Terminal (Preview) Version: 0.4.2382.0

@71
Copy link
Owner

71 commented Sep 13, 2019

No one is able to repro for now so I doubt I can help. I'll take a look at it eventually, but I am extremely busy these days, so please don't expect a fix soon.

In any case, are you sure you used the right shim? Maybe you're still using the old one (you can compare their size to make sure you're using the latest version). Are you sure nothing else may cause it to crash? Does it work fine when invoked directly without the shim?

PS: Thanks for your help, @rasa. And thanks for finding all the bugs, @trajano! :p

@trajano
Copy link
Author

trajano commented Sep 14, 2019

Maybe it's just something on my setup. Hopefully no one else hits it.

@rasa
Copy link
Contributor

rasa commented Sep 15, 2019

@trajano Your build, 10.0.18947.1000, is a 20_H1 prerelease. See https://msbuilds.rg-adguard.net/.

Can you try a released version?

@trajano
Copy link
Author

trajano commented Sep 15, 2019

Just closing this for now it could be because I am on insider builds

@trajano trajano closed this as completed Sep 15, 2019
@trajano
Copy link
Author

trajano commented Sep 23, 2019

Hmm found a laptop running the current release version of Windows 10 and I am getting the behaviour where gitk does not open up. I ran from Cmder and CMD using the 1.2 shim

Microsoft Windows [Version 10.0.18362.356]

$ which gitk
/c/Users/trajano/scoop/shims/gitk

@trajano trajano reopened this Sep 23, 2019
@trajano
Copy link
Author

trajano commented Oct 1, 2019

Another app that does not work for me with the new shims is dot from graphviz. It is supposed to be a console mode application, but with the new shim it just goes back to the command prompt after starting.

@trajano trajano changed the title 1.2 does not work with gitk 1.2 does not work with gitk and dot Oct 2, 2019
@71 71 self-assigned this Oct 2, 2019
@71
Copy link
Owner

71 commented Oct 2, 2019

Thanks for the other program, I'll check it out. I'm still busy though, so I doubt I'll come up with a fix soon.

Edit: I'm still unable to repro with dot. It works fine as a command line utility on my end.

@trajano
Copy link
Author

trajano commented Oct 3, 2019

@71 thanks not really sure why afaik I am using your 1.2 shim. However, no one else has been reporting any issues so maybe it will get merged soon. I am sort of thinking it's a PATH issue.

@mnghsn
Copy link

mnghsn commented Apr 3, 2022

I also notice this issue but somehow solved by just adding a character to the path.

path = C:\path_to_git\git.exe        # => doesn't work :(

path = C:\path_to_git\\git.exe       # => works by adding a slash.
path = C:\\path_to_git\git.exe       # => works by adding a slash.
path = C:\path_to_git\git.exe<space> # => works by adding a trailing space.
path = "C:\path_to_git\git.exe"      # => works by quoting the path.
path = "C:\path_to_git\git.exe       # => works even no ending quote.

rivy pushed a commit to rivy-t/scoop-better-shimexe that referenced this issue Aug 24, 2022
* Fix the detection of targets that use the GUI subsystem

* Allocate path on the stack
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants