Ensure commands run are not logged twice in Python output channel (microsoft#17697)

Kartik Raj · karthiknadig · commit e3aac924280e · 2021-10-11T14:40:06.000-07:00
* Ensure commands run are not logged twice in Python output channel

* News entry
diff --git a/news/2 Fixes/7160.md b/news/2 Fixes/7160.md
@@ -0,0 +1 @@
+Ensure commands run are not logged twice in Python output channel.
diff --git a/src/client/common/process/pythonExecutionFactory.ts b/src/client/common/process/pythonExecutionFactory.ts
@@ -85,7 +85,6 @@ export class PythonExecutionFactory implements IPythonExecutionFactory {
             pythonPath = this.configService.getSettings(options.resource).pythonPath;
         }
         const processService: IProcessService = await this.processServiceFactory.create(options.resource);
-        processService.on('exec', this.logger.logProcess.bind(this.logger));
 
         const windowsStoreInterpreterCheck = (await inDiscoveryExperiment(this.experimentService))
             ? // Class methods may depend on other properties which belong to the class, so bind the correct context.

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Ensure commands run are not logged twice in Python output channel.`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,6 @@ export class PythonExecutionFactory implements IPythonExecutionFactory {`
`85`	`85`	`pythonPath = this.configService.getSettings(options.resource).pythonPath;`
`86`	`86`	`}`
`87`	`87`	`const processService: IProcessService = await this.processServiceFactory.create(options.resource);`
`88`		`- processService.on('exec', this.logger.logProcess.bind(this.logger));`
`89`	`88`
`90`	`89`	`const windowsStoreInterpreterCheck = (await inDiscoveryExperiment(this.experimentService))`
`91`	`90`	`? // Class methods may depend on other properties which belong to the class, so bind the correct context.`