Skip to content

Add WaitForEXE2=git-bash.exe#19

Merged
sheabunge merged 1 commit into
sheabunge:masterfrom
3D1T0R:patch-1
Apr 18, 2016
Merged

Add WaitForEXE2=git-bash.exe#19
sheabunge merged 1 commit into
sheabunge:masterfrom
3D1T0R:patch-1

Conversation

@3D1T0R

@3D1T0R 3D1T0R commented Apr 3, 2016

Copy link
Copy Markdown
Contributor

Sorry this didn't get into #18, but I just noticed this today.

This issue basically mirrors #6, #14, #17 & #18, but is caused by a user opening Git Bash from within Git GUI, then closing Git GUI while Git Bash remains open. In this case Git Bash (and Git GUI, if launched from within Git Bash with the git gui command) will occasionally print "could not find /tmp" errors, and random .tmp files will appear in git's scans of the local repository.

This fixes the following issue: if a user opens GitGUIPortable.exe, chooses a repository, launches Git Bash from the "Repository" menu, and closes Git GUI, GitGUIPortable.exe will delete git's %TEMP% directory and Git Bash will start having the "could not find /tmp" error.
@AetherCollective

Copy link
Copy Markdown

I second this.

@sheabunge

Copy link
Copy Markdown
Owner

Sorry, I thought I merged this. I'll update and rebuild the package when I'm next on a Windows machine.

@sheabunge sheabunge merged commit bc00803 into sheabunge:master Apr 18, 2016
@3D1T0R

3D1T0R commented Apr 19, 2016

Copy link
Copy Markdown
Contributor Author

No problem. I'm just glad to see it'll be in the next Dev Test.
Any chance you could update the PA.c forum topic when you do?

Also, FYI: it's recommended for Development Test releases to use a PackageVersion that's lower than the base app's current version, this way if/when it becomes an official PortableApps.com app, the Platform's Updater will be able to tell users about the update without the official release's PackageVersion being higher than the base app's version.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants