Don't try to deploy vsix during OSS build. Specify prereq in vsixman… #1613
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
What is the build error? The changes look fine to me, though it would be nice to have build just deploy a vsix that overrides the installed one in a specific hive for testing/debugging. |
Contributor
Author
|
The deploy error is that the version being built is already installed, and then fails. The missing prereq's error looks like this: |
Member
|
In Roslyn we solve that problem by having the normal build with no parameters generate an abnormally high version number (42.42.42.42), and our official build process that produces builds that will be installed generate a real version. That way the tools don't complain because you're installing a higher version when you build, but it allows you to just build and F5 the vsix project to test out changes. |
dsyme
pushed a commit
to dsyme/fsharp
that referenced
this pull request
Oct 17, 2016
dsyme
added a commit
to dsyme/fsharp
that referenced
this pull request
Oct 17, 2016
commit 2a991b6 Author: Don Syme <[email protected]> Date: Mon Oct 17 20:40:15 2016 +0100 Fix microbuild (dotnet#1629) commit 169d366 Author: Kevin Ransom (msft) <[email protected]> Date: Sun Oct 16 15:04:50 2016 -0700 Add missed file. (dotnet#1625) * Ensure that portable libraries have correct hintpath for preview 5 * Replaced missed file commit d64ff35 Author: Don Syme <[email protected]> Date: Sun Oct 16 22:37:08 2016 +0100 cleanup build and test scripts (dotnet#1615) * cleanup build and test scripts commit 39480c4 Author: Kevin Ransom (msft) <[email protected]> Date: Sun Oct 16 10:19:03 2016 -0700 Ensure that portable libraries have correct hintpath for preview 5 (dotnet#1624) commit 2fe5a3b Author: Kevin Ransom (msft) <[email protected]> Date: Sat Oct 15 14:03:48 2016 -0700 Add support for generating sourcelink record in portable pdbs. Fixes dotnet#1585 commit e0318a8 Author: Kevin Ransom (msft) <[email protected]> Date: Sat Oct 15 13:32:44 2016 -0700 FSharp ms build task to support new xommand line options (dotnet#1616) commit 749ba2d Author: Omar Tawfik <[email protected]> Date: Sat Oct 15 13:27:14 2016 -0700 Don't match braces at the end of the caret (dotnet#1619) commit 00bab99 Author: Kevin Ransom (msft) <[email protected]> Date: Sat Oct 15 13:25:15 2016 -0700 (Nugetize F# templates. Ensure value tuple installed on create template. (dotnet#1620) commit 2a9b5ef Author: Omar Tawfik <[email protected]> Date: Fri Oct 14 22:45:35 2016 -0700 Add fsx files to workspaces (dotnet#1612) * Add fsx files to workspaces * Stop passing project output paths to CPS * Read script file contents from memory (on workspace initialization) commit 8b434e0 Author: Kevin Ransom (msft) <[email protected]> Date: Fri Oct 14 10:32:59 2016 -0700 Don't try to deploy vsix during OSS build. Specify prereq in vsixmanifest (dotnet#1613) commit 79430d2 Author: Don Syme <[email protected]> Date: Fri Oct 14 15:58:08 2016 +0100 simplify use of if-defs (dotnet#1614) commit b708921 Author: Don Syme <[email protected]> Date: Fri Oct 14 10:25:28 2016 +0100 Always use FSharp.Core not fsharp.core (dotnet#1610)
KevinRansom
added a commit
that referenced
this pull request
Oct 22, 2016
* Always use FSharp.Core not fsharp.core (#1610) * simplify use of if-defs (#1614) * Don't try to deploy vsix during OSS build. Specify prereq in vsixmanifest (#1613) * Add fsx files to workspaces (#1612) * Add fsx files to workspaces * Stop passing project output paths to CPS * Read script file contents from memory (on workspace initialization) * (Nugetize F# templates. Ensure value tuple installed on create template. (#1620) * Don't match braces at the end of the caret (#1619) * FSharp ms build task to support new xommand line options (#1616) * Add support for generating sourcelink record in portable pdbs. Fixes #1585 * Ensure that portable libraries have correct hintpath for preview 5 * Ensure that portable libraries have correct hintpath for preview 5 (#1624) * cleanup build and test scripts (#1615) * cleanup build and test scripts * Replaced missed file * Fix merge issue * fix merge error * Fix ngen build error
KevinRansom
added a commit
that referenced
this pull request
Oct 25, 2016
* Always use FSharp.Core not fsharp.core (#1610) * simplify use of if-defs (#1614) * Don't try to deploy vsix during OSS build. Specify prereq in vsixmanifest (#1613) * Add fsx files to workspaces (#1612) * Add fsx files to workspaces * Stop passing project output paths to CPS * Read script file contents from memory (on workspace initialization) * (Nugetize F# templates. Ensure value tuple installed on create template. (#1620) * Don't match braces at the end of the caret (#1619) * FSharp ms build task to support new xommand line options (#1616) * Add support for generating sourcelink record in portable pdbs. Fixes #1585 * Ensure that portable libraries have correct hintpath for preview 5 * Ensure that portable libraries have correct hintpath for preview 5 (#1624) * cleanup build and test scripts (#1615) * cleanup build and test scripts * Replaced missed file * Fix merge issue * fix merge error * update build tools * reset build_coreclr * Fix coreclr proto build * Typo
KevinRansom
added a commit
that referenced
this pull request
Oct 26, 2016
* Always use FSharp.Core not fsharp.core (#1610) * simplify use of if-defs (#1614) * Don't try to deploy vsix during OSS build. Specify prereq in vsixmanifest (#1613) * Add fsx files to workspaces (#1612) * Add fsx files to workspaces * Stop passing project output paths to CPS * Read script file contents from memory (on workspace initialization) * (Nugetize F# templates. Ensure value tuple installed on create template. (#1620) * Don't match braces at the end of the caret (#1619) * FSharp ms build task to support new xommand line options (#1616) * Add support for generating sourcelink record in portable pdbs. Fixes #1585 * Ensure that portable libraries have correct hintpath for preview 5 * Ensure that portable libraries have correct hintpath for preview 5 (#1624) * cleanup build and test scripts (#1615) * cleanup build and test scripts * Replaced missed file * Fix merge issue * fix merge error * Update to nunit 3.5
KevinRansom
added a commit
that referenced
this pull request
Oct 27, 2016
* Always use FSharp.Core not fsharp.core (#1610) * simplify use of if-defs (#1614) * Don't try to deploy vsix during OSS build. Specify prereq in vsixmanifest (#1613) * Add fsx files to workspaces (#1612) * Add fsx files to workspaces * Stop passing project output paths to CPS * Read script file contents from memory (on workspace initialization) * (Nugetize F# templates. Ensure value tuple installed on create template. (#1620) * Don't match braces at the end of the caret (#1619) * FSharp ms build task to support new xommand line options (#1616) * Add support for generating sourcelink record in portable pdbs. Fixes #1585 * Ensure that portable libraries have correct hintpath for preview 5 * Ensure that portable libraries have correct hintpath for preview 5 (#1624) * cleanup build and test scripts (#1615) * cleanup build and test scripts * Replaced missed file * Fix merge issue * fix merge error * Update to nunit 3.5 * revert unit runner to 3.0.0 * move back to local packages. * revert to nuget
KevinRansom
added a commit
that referenced
this pull request
Oct 27, 2016
* Always use FSharp.Core not fsharp.core (#1610) * simplify use of if-defs (#1614) * Don't try to deploy vsix during OSS build. Specify prereq in vsixmanifest (#1613) * Add fsx files to workspaces (#1612) * Add fsx files to workspaces * Stop passing project output paths to CPS * Read script file contents from memory (on workspace initialization) * (Nugetize F# templates. Ensure value tuple installed on create template. (#1620) * Don't match braces at the end of the caret (#1619) * FSharp ms build task to support new xommand line options (#1616) * Add support for generating sourcelink record in portable pdbs. Fixes #1585 * Ensure that portable libraries have correct hintpath for preview 5 * Ensure that portable libraries have correct hintpath for preview 5 (#1624) * cleanup build and test scripts (#1615) * cleanup build and test scripts * Replaced missed file * Fix merge issue * fix merge error * Update to nunit 3.5 * revert unit runner to 3.0.0 * Reduce number of times check path for illegal chars occurs in cropextension
nosami
pushed a commit
to xamarin/visualfsharp
that referenced
this pull request
Jan 26, 2022
commit 2a991b6 Author: Don Syme <[email protected]> Date: Mon Oct 17 20:40:15 2016 +0100 Fix microbuild (dotnet#1629) commit 169d366 Author: Kevin Ransom (msft) <[email protected]> Date: Sun Oct 16 15:04:50 2016 -0700 Add missed file. (dotnet#1625) * Ensure that portable libraries have correct hintpath for preview 5 * Replaced missed file commit d64ff35 Author: Don Syme <[email protected]> Date: Sun Oct 16 22:37:08 2016 +0100 cleanup build and test scripts (dotnet#1615) * cleanup build and test scripts commit 39480c4 Author: Kevin Ransom (msft) <[email protected]> Date: Sun Oct 16 10:19:03 2016 -0700 Ensure that portable libraries have correct hintpath for preview 5 (dotnet#1624) commit 2fe5a3b Author: Kevin Ransom (msft) <[email protected]> Date: Sat Oct 15 14:03:48 2016 -0700 Add support for generating sourcelink record in portable pdbs. Fixes dotnet#1585 commit e0318a8 Author: Kevin Ransom (msft) <[email protected]> Date: Sat Oct 15 13:32:44 2016 -0700 FSharp ms build task to support new xommand line options (dotnet#1616) commit 749ba2d Author: Omar Tawfik <[email protected]> Date: Sat Oct 15 13:27:14 2016 -0700 Don't match braces at the end of the caret (dotnet#1619) commit 00bab99 Author: Kevin Ransom (msft) <[email protected]> Date: Sat Oct 15 13:25:15 2016 -0700 (Nugetize F# templates. Ensure value tuple installed on create template. (dotnet#1620) commit 2a9b5ef Author: Omar Tawfik <[email protected]> Date: Fri Oct 14 22:45:35 2016 -0700 Add fsx files to workspaces (dotnet#1612) * Add fsx files to workspaces * Stop passing project output paths to CPS * Read script file contents from memory (on workspace initialization) commit 8b434e0 Author: Kevin Ransom (msft) <[email protected]> Date: Fri Oct 14 10:32:59 2016 -0700 Don't try to deploy vsix during OSS build. Specify prereq in vsixmanifest (dotnet#1613) commit 79430d2 Author: Don Syme <[email protected]> Date: Fri Oct 14 15:58:08 2016 +0100 simplify use of if-defs (dotnet#1614) commit b708921 Author: Don Syme <[email protected]> Date: Fri Oct 14 10:25:28 2016 +0100 Always use FSharp.Core not fsharp.core (dotnet#1610)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When building setups with latest vsroslyn vsix installed a build error occurs.vssdk builds require this.
This change disables deployment of oss vsix during build.
Additionally it adds a prerequesites section to the vsixmanifest. The latest vssdk requires this.