Don't send certain setting overrides to the daemon
These are already handled separately. This fixes warnings like warning: ignoring the user-specified setting 'max-jobs', because it is a restricted setting and you are not a trusted user when using the -j flag.
This commit is contained in:
		
							parent
							
								
									ad03159e25
								
							
						
					
					
						commit
						7298a38a07
					
				
					 1 changed files with 7 additions and 0 deletions
				
			
		|  | @ -191,6 +191,13 @@ void RemoteStore::setOptions(Connection & conn) | ||||||
|     if (GET_PROTOCOL_MINOR(conn.daemonVersion) >= 12) { |     if (GET_PROTOCOL_MINOR(conn.daemonVersion) >= 12) { | ||||||
|         std::map<std::string, Config::SettingInfo> overrides; |         std::map<std::string, Config::SettingInfo> overrides; | ||||||
|         globalConfig.getSettings(overrides, true); |         globalConfig.getSettings(overrides, true); | ||||||
|  |         overrides.erase(settings.keepFailed.name); | ||||||
|  |         overrides.erase(settings.keepGoing.name); | ||||||
|  |         overrides.erase(settings.tryFallback.name); | ||||||
|  |         overrides.erase(settings.maxBuildJobs.name); | ||||||
|  |         overrides.erase(settings.maxSilentTime.name); | ||||||
|  |         overrides.erase(settings.buildCores.name); | ||||||
|  |         overrides.erase(settings.useSubstitutes.name); | ||||||
|         conn.to << overrides.size(); |         conn.to << overrides.size(); | ||||||
|         for (auto & i : overrides) |         for (auto & i : overrides) | ||||||
|             conn.to << i.first << i.second.value; |             conn.to << i.first << i.second.value; | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue