Skip to content

Commit 1670c9b

Browse files
add logs
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
1 parent 78eaa47 commit 1670c9b

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

code/src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -281,9 +281,12 @@ export abstract class AbstractExtensionManagementService extends CommontExtensio
281281

282282
const getInstallExtensionTaskKey = (extension: IGalleryExtension, profileLocation: URI) => `${ExtensionKey.create(extension).toString()}-${profileLocation.toString()}`;
283283
const createInstallExtensionTask = (manifest: IExtensionManifest, extension: IGalleryExtension | URI, options: InstallExtensionTaskOptions, root: IInstallExtensionTask | undefined): void => {
284+
this.logService.info('======= createInstallExtensionTask');
284285
let uninstallTaskToWaitFor;
285286
if (!URI.isUri(extension)) {
287+
this.logService.info('=== createInstallExtensionTask === NOT URI ', extension.identifier.id);
286288
if (installingExtensionsMap.has(`${extension.identifier.id.toLowerCase()}-${options.profileLocation.toString()}`)) {
289+
this.logService.info('=== createInstallExtensionTask === NOT URI === RETURN ', extension.identifier.id);
287290
return;
288291
}
289292
const existingInstallingExtension = this.installingExtensions.get(getInstallExtensionTaskKey(extension, options.profileLocation));
@@ -345,6 +348,7 @@ export abstract class AbstractExtensionManagementService extends CommontExtensio
345348
if (task.options.donotIncludePackAndDependencies) {
346349
this.logService.info('Installing the extension without checking dependencies and pack', task.identifier.id);
347350
} else {
351+
this.logService.info('Installing the extension WITH dependencies and pack', task.identifier.id);
348352
try {
349353
let preferPreRelease = this.preferPreReleases;
350354
if (task.options.installPreReleaseVersion) {
@@ -355,13 +359,17 @@ export abstract class AbstractExtensionManagementService extends CommontExtensio
355359
}
356360
const installed = await this.getInstalled(undefined, task.options.profileLocation, task.options.productVersion);
357361
const allDepsAndPackExtensionsToInstall = await this.getAllDepsAndPackExtensions(task.identifier, task.manifest, preferPreRelease, task.options.productVersion, installed);
362+
this.logService.info('DEPS ', allDepsAndPackExtensionsToInstall.length);
358363
const options: InstallExtensionTaskOptions = { ...task.options, pinned: false, installGivenVersion: false, context: { ...task.options.context, [EXTENSION_INSTALL_DEP_PACK_CONTEXT]: true } };
359364
for (const { gallery, manifest } of distinct(allDepsAndPackExtensionsToInstall, ({ gallery }) => gallery.identifier.id)) {
365+
this.logService.info('DEPS === gallery.identifier.id ', gallery.identifier.id);
360366
const existing = installed.find(e => areSameExtensions(e.identifier, gallery.identifier));
361367
// Skip if the extension is already installed and has the same application scope
362368
if (existing && existing.isApplicationScoped === !!options.isApplicationScoped) {
369+
this.logService.info('DEPS === gallery.identifier.id === SKIP ', gallery.identifier.id);
363370
continue;
364371
}
372+
this.logService.info('DEPS === gallery.identifier.id === NOT SKIP ', gallery.identifier.id);
365373
createInstallExtensionTask(manifest, gallery, options, task);
366374
}
367375
} catch (error) {

0 commit comments

Comments
 (0)