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

Signal fails to update when git linux commands are in path. #5856

Open
2 tasks done
theAddict opened this issue Apr 6, 2022 · 9 comments
Open
2 tasks done

Signal fails to update when git linux commands are in path. #5856

theAddict opened this issue Apr 6, 2022 · 9 comments
Labels

Comments

@theAddict
Copy link

theAddict commented Apr 6, 2022

Official solution

  • I have searched open and closed issues for duplicates
  • I am using Signal-Desktop as provided by the Signal team, not a 3rd-party package.

Bug Description

When find.exe from C:\Program Files\Git\usr\bin is in the path, the installer will fail, indicating that Signal is running and could not be closed.

Work-around

  1. Open command prompt
  2. execute set PATH=c:\windows\system32;%PATH%
  3. run signal update from command prompt

Potential solution

Call %SYSTEMROOT%\System32\find.exe instead of find when running the tasklist command.

Steps to Reproduce

  1. Have linux version of find in path. For example, from C:\Program Files\Git\usr\bin
  2. Attempt to update Signal
  3. Update will fail

Actual Result: Signal is successfully closed, if open. The signal installer indicates that Signal is open and could not be closed.

Expected Result: Signal would update

Screenshots

Platform Info

Signal Version: 5.37.0

Operating System: Windows 10

Linked Device Version:

Link to Debug Log

@indutny-signal
Copy link
Contributor

Oh my goodness. I think this could be a source of #5854. Thanks for tracking it down.

@fattybacon
Copy link

Oh my goodness. I think this could be a source of #5854. Thanks for tracking it down.

I've been following the journey, and yep I've got git's linux tools installed and I've got the same signal update issues.. I've always wondered why dir doesn't behave like a Windows tool anymore. This is why!

@fattybacon
Copy link

@theAddict how do you manually update signal from the command line? I've resolved my issue by running latest installer from the command line after switching the PATH as above, but it would be good to know how to run signal update.

@theAddict
Copy link
Author

Oh my goodness. I think this could be a source of #5854. Thanks for tracking it down.

@indutny-signal, glad it was helpful. I saw #5854. The symptoms look pretty similar to what I experienced. I thought I'd make a separate ticket because I wasn't certain. Hopefully this fixes @TMMC's issue too.

@theAddict
Copy link
Author

@theAddict how do you manually update signal from the command line? I've resolved my issue by running latest installer from the command line after switching the PATH as above, but it would be good to know how to run signal update.

@fattybacon , I ran the installer manually as described in the workaround to update my machine.

Here are a couple ideas that would probably work for invoking the installer via the in-app update mechanism:

  1. You could invoke signal.exe using the workaround above, since the environment should be copied to any child processes.
  2. You could also update the shortcut similarly to what is described here, but set set PATH as described above instead if you want the environment to persist for each execution: https://netlicensing.io/blog/2012/06/15/set-environment-variables-in-windows-shortcut/

@gentlemanjohn
Copy link

I had the same issue on Windows 11 with cygwin in my PATH. I couldn't update or uninstall Signal until I removed the offending item from my PATH.

@colajunkie13
Copy link

For anyone finding this: it was WinAVR in my case.

@indutny-signal
Copy link
Contributor

Posted official solution: #5854 (comment)

@pschneider1968
Copy link

I was also affected by this problem. I have MSYS2 and Git for Windows installed, and their bin directories in my %PATH%. Temporarily renaming the find.exe from Git and MSYS2 /usr/bin to find.exe.BAK resolved the problem, and I could then uninstall 5.36 and newly install 5.41.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

6 participants