Skip to content

Commit b925279

Browse files
author
Ravi Chande
authored
Merge branch 'master' into omnisharp1.29
2 parents 2af617b + 6122e27 commit b925279

23 files changed

+7531
-16
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
* Fixes symbol loading while debugging apps running under .NET Core 2.1 (ex: netcoreapp 2.1) on Linux and macOS
1111
* Fixes debug console message encoding issue on Windows ([#1775](https://github.com/OmniSharp/omnisharp-vscode/issues/1775)).
12+
* Adds support for extracting source files embedded in PDBs. See the C# [EmbeddedFiles](https://github.com/dotnet/roslyn/issues/19127) feature for more information.
1213
* Adds preliminary support for Linux ARM debugging
1314

1415
#### Editor

package.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "csharp",
33
"publisher": "ms-vscode",
4-
"version": "1.14.0-beta2",
4+
"version": "1.14.0-beta3",
55
"description": "C# for Visual Studio Code (powered by OmniSharp).",
66
"displayName": "C#",
77
"author": "Microsoft Corporation",
@@ -159,8 +159,8 @@
159159
},
160160
{
161161
"description": ".NET Core Debugger (Windows / x64)",
162-
"url": "https://download.visualstudio.microsoft.com/download/pr/100317420/a30d7e11bc435433d297adc824ee837f/coreclr-debug-win7-x64.zip",
163-
"fallbackUrl": "https://vsdebugger.blob.core.windows.net/coreclr-debug-1-14-4/coreclr-debug-win7-x64.zip",
162+
"url": "https://download.visualstudio.microsoft.com/download/pr/11522728/3d79abf5bc635c3b55ad2ee590c1eb0e/coreclr-debug-win7-x64.zip",
163+
"fallbackUrl": "https://vsdebugger.blob.core.windows.net/coreclr-debug-1-14-7/coreclr-debug-win7-x64.zip",
164164
"installPath": ".debugger",
165165
"platforms": [
166166
"win32"
@@ -172,8 +172,8 @@
172172
},
173173
{
174174
"description": ".NET Core Debugger (macOS / x64)",
175-
"url": "https://download.visualstudio.microsoft.com/download/pr/100317420/a30d7e11bc435433d297adc824ee837f/coreclr-debug-osx-x64.zip",
176-
"fallbackUrl": "https://vsdebugger.blob.core.windows.net/coreclr-debug-1-14-4/coreclr-debug-osx.10.11-x64.zip",
175+
"url": "https://download.visualstudio.microsoft.com/download/pr/11522728/3d79abf5bc635c3b55ad2ee590c1eb0e/coreclr-debug-osx-x64.zip",
176+
"fallbackUrl": "https://vsdebugger.blob.core.windows.net/coreclr-debug-1-14-7/coreclr-debug-osx-x64.zip",
177177
"installPath": ".debugger",
178178
"platforms": [
179179
"darwin"
@@ -189,8 +189,8 @@
189189
},
190190
{
191191
"description": ".NET Core Debugger (linux / x64)",
192-
"url": "https://download.visualstudio.microsoft.com/download/pr/100317420/a30d7e11bc435433d297adc824ee837f/coreclr-debug-linux-x64.zip",
193-
"fallbackUrl": "https://vsdebugger.blob.core.windows.net/coreclr-debug-1-14-4/coreclr-debug-linux-x64.zip",
192+
"url": "https://download.visualstudio.microsoft.com/download/pr/11522728/3d79abf5bc635c3b55ad2ee590c1eb0e/coreclr-debug-linux-x64.zip",
193+
"fallbackUrl": "https://vsdebugger.blob.core.windows.net/coreclr-debug-1-14-7/coreclr-debug-linux-x64.zip",
194194
"installPath": ".debugger",
195195
"platforms": [
196196
"linux"
@@ -2125,4 +2125,4 @@
21252125
}
21262126
]
21272127
}
2128-
}
2128+
}

src/omnisharp/server.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -351,11 +351,10 @@ export class OmniSharpServer {
351351
});
352352
}
353353

354-
public restart(launchTarget: LaunchTarget = this._launchTarget): Promise<void> {
354+
public async restart(launchTarget: LaunchTarget = this._launchTarget): Promise<void> {
355355
if (launchTarget) {
356-
return this.stop().then(() => {
357-
this._start(launchTarget);
358-
});
356+
await this.stop();
357+
await this._start(launchTarget);
359358
}
360359
}
361360

test/integrationTests/hoverProvider.integration.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,22 +21,22 @@ suite(`Tasks generation: ${testAssetWorkspace.description}`, function() {
2121
suiteSetup(async function() {
2222
should();
2323

24+
await testAssetWorkspace.cleanupWorkspace();
25+
2426
let csharpExtension = vscode.extensions.getExtension("ms-vscode.csharp");
2527
if (!csharpExtension.isActive) {
2628
await csharpExtension.activate();
2729
}
2830

29-
await testAssetWorkspace.cleanupWorkspace();
30-
3131
await csharpExtension.exports.initializationFinished;
32-
32+
await omnisharp.restart();
33+
3334
await vscode.commands.executeCommand("dotnet.generateAssets");
3435

3536
await poll(async () => await fs.exists(testAssetWorkspace.launchJsonPath), 10000, 100);
3637

3738
});
3839

39-
4040
test("Hover returns structured documentation with proper newlines", async function () {
4141

4242
let program =

test/integrationTests/testAssets/singleCsproj/obj/Debug/netcoreapp2.0/singleCsproj.AssemblyInfo.cs

Whitespace-only changes.
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
430bdcfd8bad4c9c4b8cb192efe5e61394e77ebc
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
fe2f654e9c48e6a308b6a264aa38a2b3defe5730

test/integrationTests/testAssets/singleCsproj/obj/project.assets.json

Lines changed: 721 additions & 0 deletions
Large diffs are not rendered by default.
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"version": 1,
3+
"dgSpecHash": "F/2npLNJmV62pdE6xnL6onDE/ZUOXXwAbeOh12Em+cH/3uZOrmJZgsI0iJtb+26OOF3cRtdrW33x5W5+YX4DVw==",
4+
"success": true
5+
}
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
<?xml version="1.0" encoding="utf-8" standalone="no"?>
2+
<Project ToolsVersion="14.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
3+
<PropertyGroup Condition=" '$(ExcludeRestorePackageImports)' != 'true' ">
4+
<RestoreSuccess Condition=" '$(RestoreSuccess)' == '' ">True</RestoreSuccess>
5+
<RestoreTool Condition=" '$(RestoreTool)' == '' ">NuGet</RestoreTool>
6+
<ProjectAssetsFile Condition=" '$(ProjectAssetsFile)' == '' ">C:\omnisharp-vscode\test\integrationTests\testAssets\singleCsproj\obj\project.assets.json</ProjectAssetsFile>
7+
<NuGetPackageRoot Condition=" '$(NuGetPackageRoot)' == '' ">$(UserProfile)\.nuget\packages\</NuGetPackageRoot>
8+
<NuGetPackageFolders Condition=" '$(NuGetPackageFolders)' == '' ">C:\Users\rchande\.nuget\packages\;C:\Program Files\dotnet\sdk\NuGetFallbackFolder</NuGetPackageFolders>
9+
<NuGetProjectStyle Condition=" '$(NuGetProjectStyle)' == '' ">PackageReference</NuGetProjectStyle>
10+
<NuGetToolVersion Condition=" '$(NuGetToolVersion)' == '' ">4.5.0</NuGetToolVersion>
11+
</PropertyGroup>
12+
<PropertyGroup>
13+
<MSBuildAllProjects>$(MSBuildAllProjects);$(MSBuildThisFileFullPath)</MSBuildAllProjects>
14+
</PropertyGroup>
15+
<ImportGroup Condition=" '$(ExcludeRestorePackageImports)' != 'true' ">
16+
<Import Project="$(NuGetPackageRoot)microsoft.netcore.app\2.0.0\build\netcoreapp2.0\Microsoft.NETCore.App.props" Condition="Exists('$(NuGetPackageRoot)microsoft.netcore.app\2.0.0\build\netcoreapp2.0\Microsoft.NETCore.App.props')" />
17+
</ImportGroup>
18+
</Project>

0 commit comments

Comments
 (0)