C#: Only use nuget.exe on Windows or machines with Mono.#21993
Draft
michaelnebel wants to merge 6 commits into
Draft
C#: Only use nuget.exe on Windows or machines with Mono.#21993michaelnebel wants to merge 6 commits into
nuget.exe on Windows or machines with Mono.#21993michaelnebel wants to merge 6 commits into
Conversation
michaelnebel
force-pushed
the
csharp/dropmono
branch
from
94cac45 to
892517e
Compare
michaelnebel
changed the title
nuget.exe on Windows or machines with Mono.
michaelnebel
changed the title
nuget.exe on Windows or machines with Mono.nuget.exe on Windows or machines with Mono.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
michaelnebel
force-pushed
the
csharp/dropmono
branch
from
892517e to
e23819d
Compare
Comment on lines
+115
to
+118
| catch (Exception e) | ||
| { | ||
| logger.LogError($"Failed to add default package source to {nugetConfigPath}: {e}"); | ||
| } |
Comment on lines
+143
to
+146
| catch (Exception exc) | ||
| { | ||
| logger.LogInfo($"Download of nuget.exe failed: {exc.Message}"); | ||
| } |
Comment on lines
+269
to
+273
| catch (Exception e) | ||
| { | ||
| logger.LogWarning($"Failed to check if default package source is added: {e}"); | ||
| return true; | ||
| } |
Comment on lines
+337
to
+340
| catch (Exception exc) | ||
| { | ||
| logger.LogError($"Failed to restore original nuget.config file: {exc}"); | ||
| } |
michaelnebel
force-pushed
the
csharp/dropmono
branch
from
e23819d to
042ed54
Compare
…ry method for constructing package config restorers.
michaelnebel
force-pushed
the
csharp/dropmono
branch
from
042ed54 to
2675042
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
As mono is being deprecated (phased out) on the MacOS and Linux runners, we need to adjust the dependency manager implementation to only attempt to use
nuget.exeon Windows or machines wheremonois installed asnuget.execan't be run natively on Linux or MacOS (it requiresmono).