Skip to content

Commit b842813

Browse files
Swich away from VSIXExpInstaller
Just use the real installer rather than our own.
1 parent 25a24cb commit b842813

File tree

6 files changed

+4
-27
lines changed

6 files changed

+4
-27
lines changed

eng/Directory.Packages.props

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,6 @@
311311
<PackageVersion Include="Mono.Options" Version="6.6.0.161" />
312312
<PackageVersion Include="Roslyn.Diagnostics.Analyzers" Version="$(RoslynDiagnosticsNugetPackageVersion)" />
313313
<PackageVersion Include="SourceBrowser" Version="1.0.21" />
314-
<PackageVersion Include="RoslynTools.VSIXExpInstaller" Version="1.1.0-beta3.22474.1" />
315314
<PackageVersion Include="MicroBuildPluginsSwixBuild" Version="1.1.87" />
316315
<PackageVersion Include="vswhere" Version="$(vswhereVersion)" />
317316
<!--

eng/build.ps1

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -556,13 +556,6 @@ function EnablePreviewSdks() {
556556
# deploying at build time.
557557
function Deploy-VsixViaTool() {
558558

559-
$vsixExe = Join-Path $ArtifactsDir "bin\RunTests\$configuration\net9.0\VSIXExpInstaller\VSIXExpInstaller.exe"
560-
Write-Host "VSIX EXE path: " $vsixExe
561-
if (-not (Test-Path $vsixExe)) {
562-
Write-Host "VSIX EXE not found: '$vsixExe'." -ForegroundColor Red
563-
ExitWithExitCode 1
564-
}
565-
566559
$vsInfo = LocateVisualStudio
567560
if ($vsInfo -eq $null) {
568561
throw "Unable to locate required Visual Studio installation"
@@ -575,7 +568,7 @@ function Deploy-VsixViaTool() {
575568

576569
$hive = "RoslynDev"
577570
Write-Host "Using VS Instance $vsId ($displayVersion) at `"$vsDir`""
578-
$baseArgs = "/rootSuffix:$hive /vsInstallDir:`"$vsDir`""
571+
$baseArgs = "/rootSuffix:$hive /quiet /shutdownprocesses"
579572

580573
Write-Host "Uninstalling old Roslyn VSIX"
581574

@@ -604,9 +597,10 @@ function Deploy-VsixViaTool() {
604597

605598
foreach ($vsixFileName in $orderedVsixFileNames) {
606599
$vsixFile = Join-Path $VSSetupDir $vsixFileName
600+
$vsixInstallerExe = Join-Path $vsDir "Common7\IDE\VSIXInstaller.exe"
607601
$fullArg = "$baseArgs $vsixFile"
608602
Write-Host "`tInstalling $vsixFileName"
609-
Exec-Command $vsixExe $fullArg
603+
Exec-Command $vsixInstallerExe $fullArg
610604
}
611605

612606
# Set up registry

eng/test-rebuild.ps1

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,6 @@ try {
6969
" --exclude net9.0\Microsoft.CodeAnalysis.Threading.Package.dll" +
7070
" --exclude netstandard2.0\Microsoft.CodeAnalysis.Extensions.Package.dll" +
7171
" --exclude netcoreapp3.1\Microsoft.CodeAnalysis.Workspaces.UnitTests.dll" +
72-
" --exclude net472\Zip\tools\vsixexpinstaller\System.ValueTuple.dll" +
73-
" --exclude net472\Zip\tools\vsixexpinstaller\VSIXExpInstaller.exe" +
7472

7573
# Semantic Search reference assemblies can't be reconstructed from source.
7674
# The assemblies are not marked with ReferenceAssemblyAttribute attribute.

src/Setup/Installer/Installer.Package.csproj

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
<ItemGroup>
88
<PackageReference Include="vswhere" />
9-
<PackageReference Include="RoslynTools.VSIXExpInstaller" />
109
<ProjectReference Include="..\..\Deployment\RoslynDeployment.csproj"/>
1110
</ItemGroup>
1211

@@ -29,7 +28,6 @@
2928
<_Files Include="$(MSBuildProjectDirectory)\tools\*.*" TargetDir="tools"/>
3029
<_Files Include="$(MSBuildProjectDirectory)\scripts\*.*" TargetDir=""/>
3130
<_Files Include="$(NuGetPackageRoot)vswhere\$(vswhereVersion)\tools\*.*" TargetDir="tools\vswhere"/>
32-
<_Files Include="$(NuGetPackageRoot)roslyntools.vsixexpinstaller\$(RoslynToolsVSIXExpInstallerVersion)\tools\*.*" TargetDir="tools\vsixexpinstaller"/>
3331
<_Files Include="$(_DeploymentVsixPath)" TargetDir="vsix"/>
3432
</ItemGroup>
3533

src/Setup/Installer/tools/utils.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ function Uninstall-VsixViaTool([string]$vsDir, [string]$vsId, [string]$rootSuffi
128128
}
129129

130130
function Use-VsixTool([string]$vsDir, [string]$vsId, [string]$rootSuffix, [string]$additionalArgs = "") {
131-
$installerExe = Join-Path $PSScriptRoot "vsixexpinstaller\VsixExpInstaller.exe"
131+
$installerExe = Join-Path $vsDir "Common7\IDE\VSIXInstaller.exe"
132132
Write-Host "Using VS Instance $vsId at `"$vsDir`"" -ForegroundColor Gray
133133

134134
$vsixFileNames = @("RoslynDeployment.vsix")

src/Tools/Source/RunTests/RunTests.csproj

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -17,16 +17,4 @@
1717
<PackageReference Include="Microsoft.TeamFoundationServer.Client" />
1818
<ProjectReference Include="..\..\..\Compilers\Test\Core\Microsoft.CodeAnalysis.Test.Utilities.csproj" />
1919
</ItemGroup>
20-
21-
22-
<!-- Include the VSIX installer so we have it in the build payload for integration tests -->
23-
<ItemGroup>
24-
<PackageReference Include="RoslynTools.VSIXExpInstaller" GeneratePathProperty="true" />
25-
</ItemGroup>
26-
<ItemGroup>
27-
<_CopyItems Include="$(PkgRoslynTools_VSIXExpInstaller)\tools\*.*" />
28-
</ItemGroup>
29-
<Target Name="CopyCustomContent" AfterTargets="AfterBuild" Condition="$([MSBuild]::IsOSPlatform('Windows'))">
30-
<Copy SourceFiles="@(_CopyItems)" DestinationFolder="$(OutDir)/VSIXExpInstaller" />
31-
</Target>
3220
</Project>

0 commit comments

Comments
 (0)