dotnet/msbuild

Use current runtime + architecture for GenerateResource on .NET Core

Open

#1 293 ouverte le 1 nov. 2016

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)C# (1 364 forks)batch import
.NET Corebughelp wantedtriaged

Métriques du dépôt

Stars
 (5 062 stars)
Métriques de merge PR
 (Merge moyen 11j 7h) (125 PRs mergées en 30 j)

Description

See https://github.com/dotnet/sdk/issues/346#issuecomment-257654120.

On .NET Core, MSBuild doesn't support task hosts of different architectures/runtime versions. But the CoreResGen target does some gymnastics to figure out the right architecture/runtime version to use for the GenerateResource task.

On .NET Core we should just use CurrentArchitecture/CurrentRuntime.

Or as @nguerrera suggested, maybe just add the DoesTaskHostExist check for Architecture as well as Runtime.

Guide contributeur

Direction de recherche
Examinez le fichier src/XMakeTasks/Microsoft.Common.CurrentVersion.targets autour des lignes 2785-2809, où la cible CoreResGen effectue des gymnastiques pour déterminer l'architecture/le runtime. Le problème suggère de simplifier en utilisant CurrentArchitecture/CurrentRuntime pour .NET Core. De plus, consultez le commentaire lié dans dotnet/sdk#346 pour un contexte supplémentaire. Aucune PR liée n'existe, donc la correction impliquerait de modifier la cible pour utiliser conditionnellement le runtime actuel sur .NET Core.
Stack technique
csharp
Domaine
build system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#MSBuild.NET Core
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30