Skip to content

Fixed process start for application and webhelper#28

Merged
JohnnyCrazy merged 2 commits intoJohnnyCrazy:masterfrom
wmbch:master
Aug 2, 2015
Merged

Fixed process start for application and webhelper#28
JohnnyCrazy merged 2 commits intoJohnnyCrazy:masterfrom
wmbch:master

Conversation

@wmbch
Copy link
Copy Markdown
Contributor

@wmbch wmbch commented Aug 2, 2015

The method Path.Combine() does not work (it has no string-based return of the combined value with AppData-path and the const string values for spotify.exe and spotifywebhelper.exe).

@JohnnyCrazy
Copy link
Copy Markdown
Owner

I just tested it and found out why it isn't working, the first slash in @"\spotify\spotify.exe" is the problem. Would you mind if we still go for the Path.Combine-Way and just replace it with @"spotify\spotify.exe"? Would love to merge it :)

@wmbch
Copy link
Copy Markdown
Contributor Author

wmbch commented Aug 2, 2015

Allright! Reverted it to Path.Combine and fixed the backslash! :)

@JohnnyCrazy
Copy link
Copy Markdown
Owner

Nice, thanks :)

JohnnyCrazy pushed a commit that referenced this pull request Aug 2, 2015
Fixed process start for application and webhelper
@JohnnyCrazy JohnnyCrazy merged commit 9ac735f into JohnnyCrazy:master Aug 2, 2015
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.

2 participants