ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-16 10:58:30 -05:00
Code Issues Packages Projects Releases Wiki Activity

Use Existing Install option by default when setting up python on Mac and Linux. (#21093)

Browse Source
This commit is contained in:
Cory Rivera
2022-11-03 11:01:18 -07:00
committed by GitHub
parent ec03b3f4aa
commit 90d0310523
2 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
extensions/notebook/src/common/constants.ts
View File

@@ -28,6 +28,7 @@ export const allowRoot = 'allowRoot';
export const winPlatform = 'win32'; export const winPlatform = 'win32';
export const macPlatform = 'darwin'; export const macPlatform = 'darwin';
export const linuxPlatform = 'linux';
export const jupyterNotebookProviderId = 'jupyter'; export const jupyterNotebookProviderId = 'jupyter';
export const jupyterConfigRootFolder = 'jupyter_config'; export const jupyterConfigRootFolder = 'jupyter_config';

6
extensions/notebook/src/dialog/configurePython/configurePythonWizard.ts
View File

@@ -13,6 +13,8 @@ import * as utils from '../../common/utils';
import { promises as fs } from 'fs'; import { promises as fs } from 'fs';
import { Deferred } from '../../common/promise'; import { Deferred } from '../../common/promise';
import { PythonPathLookup } from '../pythonPathLookup'; import { PythonPathLookup } from '../pythonPathLookup';
import { linuxPlatform, macPlatform } from '../../common/constants';
import * as os from 'os';
const localize = nls.loadMessageBundle(); const localize = nls.loadMessageBundle();
@@ -56,6 +58,10 @@ export class ConfigurePythonWizard {
pythonLocation: JupyterServerInstallation.getPythonPathSetting(), pythonLocation: JupyterServerInstallation.getPythonPathSetting(),
useExistingPython: JupyterServerInstallation.getExistingPythonSetting() useExistingPython: JupyterServerInstallation.getExistingPythonSetting()
}; };
// Default to using existing Python on Mac and Linux, since they have python installed by default
if (os.platform() === macPlatform || os.platform() === linuxPlatform) {
this.model.useExistingPython = true;
}
let pages: Map<number, BasePage> = new Map<number, BasePage>(); let pages: Map<number, BasePage> = new Map<number, BasePage>();