Skip to content

Commit

Permalink
Merge pull request #383 from microsoft/dev/daeunyim/revert-service-do…
Browse files Browse the repository at this point in the history
…wnloader

Revert service downloader to old one
  • Loading branch information
DaeunYim authored May 16, 2023
2 parents 36aa94e + 9e24753 commit e0a19c1
Show file tree
Hide file tree
Showing 6 changed files with 268 additions and 132 deletions.
2 changes: 1 addition & 1 deletion .pipeline/release-pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ steps:
script: |
yarn install
yarn run compile
yarn run package-offline
yarn run package
- task: EsrpCodeSigning@2
displayName: 'Code Signing'
inputs:
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -419,7 +419,7 @@
}
},
"dependencies": {
"@microsoft/ads-service-downloader": "^1.1.0",
"service-downloader": "github:daeunyim/service-downloader#0.1.1",
"@microsoft/ads-extension-telemetry": "^1.3.4",
"crypto": "^1.0.1",
"dataprotocol-client": "github:Microsoft/sqlops-dataprotocolclient#1.3.1",
Expand Down
2 changes: 1 addition & 1 deletion src/config.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"downloadUrl": "https://github.com/Microsoft/pgtoolsservice/releases/download/{#version#}/pgsqltoolsservice-{#fileName#}",
"version": "v1.6.0-alpha",
"version": "v1.7.0",
"downloadFileNames": {
"Windows_64": "win-x64.zip",
"Windows_86": "win-x86.zip",
Expand Down
2 changes: 1 addition & 1 deletion src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
import * as vscode from 'vscode';
import * as path from 'path';
import { SqlOpsDataClient, ClientOptions } from 'dataprotocol-client';
import { IConfig, ServerProvider, Events } from '@microsoft/ads-service-downloader';
import { IConfig, ServerProvider, Events } from 'service-downloader';
import { ServerOptions, TransportKind } from 'vscode-languageclient';

import * as Constants from './constants';
Expand Down
25 changes: 21 additions & 4 deletions tasks/packagetasks.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ var cproc = require('child_process');
var os = require('os');
var del = require('del');
var path = require('path');
var serviceDownloader = require('@microsoft/ads-service-downloader');
var serviceDownloader = require('service-downloader');


function getServiceInstallConfig() {
Expand Down Expand Up @@ -70,10 +70,27 @@ function doOfflinePackage(runtimeId, runtime, packageName) {

//Install vsce to be able to run this task: npm install -g vsce
gulp.task('package:online', () => {
return cleanServiceInstallFolder().then(() => {
doPackageSync();
return getOrDownloadServer();
var json = JSON.parse(fs.readFileSync('package.json'));
var name = json.name;
var version = json.version;
var packageName = name + '-' + version;

var packages = [];
packages.push({rid: 'win-x64', runtime: 'Windows_64'});
packages.push({rid: 'osx', runtime: 'OSX'});
packages.push({rid: 'osx-arm64', runtime: 'OSX_ARM64'});

var promise = cleanServiceInstallFolder();

packages.forEach(data => {
promise = promise.then(() => {
return cleanServiceInstallFolder().then(() => {
return doPackageSync(packageName + '-' + data.rid + '.vsix');
});
});
});

return promise;
});

//Install vsce to be able to run this task: npm install -g vsce
Expand Down
Loading

0 comments on commit e0a19c1

Please sign in to comment.