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.

Sign up for GitHub

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

Jump to bottom

(Windows) GPU detection fails when started from command line (take 2) #2657

Closed
RichardHaselgrove opened this issue Aug 24, 2018 · 2 comments · Fixed by #2713
Closed

(Windows) GPU detection fails when started from command line (take 2) #2657

RichardHaselgrove opened this issue Aug 24, 2018 · 2 comments · Fixed by #2713
Assignees
@JuhaSointusalo
Labels
C: Client E: 1 day OS: Windows P: Major

Comments

@RichardHaselgrove
Copy link
Contributor

Previous references: issue #2029 , PR #2036.

A year ago we tackled an issue where GPU detection failed if the client was started from the command line with "boinc", rather than "boinc.exe"

The problem has been re-reported for BOINC v7.10.2 and v7.12.1, but this time when the invocation is from a batch file or from BoincTasks, if the correct working directory is not used. Compare these screenshots (from Process Explorer)

boinc client 2 - v7 8 3
boinc client 2 - v7 12 1

After all the multiple restarts to detach from the command window and do the GPU detection, the final running client instance was still started with the full path on the command line (D:\BOINC... on my machine). By contrast, v7.12.1 starts with the now hard-wired "boinc.exe", but no path.

I think commit 7930686 just moved the problem around a bit, without fully fixing it. We need both the path and the .exe
@JuhaSointusalo JuhaSointusalo self-assigned this Aug 24, 2018
@JuhaSointusalo JuhaSointusalo mentioned this issue Sep 22, 2018
Merged
@RichardHaselgrove RichardHaselgrove mentioned this issue Oct 9, 2018
Closed
@AenBleidd
Copy link
Member

I'm reopening this because of new issue that was found and changes are reverted in #2747

@AenBleidd AenBleidd reopened this Oct 9, 2018
@AenBleidd AenBleidd mentioned this issue Oct 9, 2018
Merged
@JuhaSointusalo
Copy link
Contributor

Closing. The issue was fixed in #2713 and the problems in that PR were fixed in #2748.

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

@JuhaSointusalo JuhaSointusalo

Labels
C: Client E: 1 day OS: Windows P: Major
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

client: fixes for launching GPU detection
3 participants
@AenBleidd @RichardHaselgrove @JuhaSointusalo