Skip to content

Commit

Permalink
client: don't show notices about not being able to do certain types o…
Browse files Browse the repository at this point in the history
…f work

after a scheduler RPC.
These are shown at startup.
The only thing that may have changed is project prefs (e.g. no GPU)
and the user did that - we don't need to remind them that they did it.
  • Loading branch information
davidpanderson committed Dec 31, 2018
1 parent aed33a4 commit 9f8f52b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 2 additions & 0 deletions client/cs_account.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ static void handle_no_rsc_pref(PROJECT* p, const char* name) {

// parse an account_*.xml file, ignoring <venue> elements
// (since we don't know the host venue yet)
// Called at startup and after scheduler RPC
//
int PROJECT::parse_account(FILE* in) {
char buf2[256];
Expand Down Expand Up @@ -195,6 +196,7 @@ int PROJECT::parse_account(FILE* in) {
// and parsing that for resource share and prefs.
// Call this only after client_state.xml has been read
// (so that we know the host venue)
// Called at startup and after scheduler RPC
//
int PROJECT::parse_account_file_venue() {
char attr_buf[256], venue[256], path[MAXPATHLEN], buf2[256];
Expand Down
7 changes: 4 additions & 3 deletions client/cs_scheduler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -763,10 +763,11 @@ int CLIENT_STATE::handle_scheduler_reply(
active_tasks.request_reread_prefs(project);
}

// show notice if we can't possibly get work from this project.
// This must come after parsing project prefs
// notices here serve no purpose.
// The only thing that may have changed is project prefs,
// and there's no reason to tell the user what they just did.
//
project->show_no_work_notice();
//project->show_no_work_notice();

// if the scheduler reply includes a code-signing key,
// accept it if we don't already have one from the project.
Expand Down

0 comments on commit 9f8f52b

Please sign in to comment.