dotnet/msbuild

Use current runtime + architecture for GenerateResource on .NET Core

Open

#1293 aperta il 1 nov 2016

Vedi su GitHub
 (2 commenti) (1 reazione) (0 assegnatari)C# (1364 fork)batch import
.NET Corebughelp wantedtriaged

Metriche repository

Star
 (5062 star)
Metriche merge PR
 (Merge medio 11g 7h) (125 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina il file src/XMakeTasks/Microsoft.Common.CurrentVersion.targets intorno alle righe 2785-2809, dove il target CoreResGen esegue ginnastiche per determinare l'architettura/runtime. Il problema suggerisce di semplificare utilizzando CurrentArchitecture/CurrentRuntime per .NET Core. Inoltre, rivedi il commento collegato in dotnet/sdk#346 per ulteriore contesto. Non esistono PR collegati, quindi la correzione comporterebbe la modifica del target per utilizzare condizionalmente il runtime corrente su .NET Core.
Tech stack
csharp
Dominio
build system
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#MSBuild.NET Core
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30