Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MSB4044: The "RemoveTypeScriptStep" task was not given a value for the required parameter "Original". #50

Closed
PandaWood opened this issue Oct 23, 2017 · 3 comments
Labels

Comments

@PandaWood
Copy link
Contributor

PandaWood commented Oct 23, 2017

The only thing I've changed in my codebase, to get this error, is to move my tsconfig.json
Which technically, has nothing to do with Reinforced.Typings, since it doesn't build my typescript or need to know anything about it.

So maybe this is an issue to request that if RtBypassTypeScriptCompilation=true, then cater for not finding or having anything to do with my tsconfig.json file?

Reinforced.Typings.settings.xml : warning : TypeScript sources will not be built before project compile because it is disabled by Reinforced.Typings configuration [C: \code\src\Web.Site\Web.Site.csproj]
RT is fixing build tasks
RT has fixed build tasks:ResolveReferences;ResolveKeySource;SetWin32ManifestProperties;_GenerateCompileInputs;BeforeCompile;_TimeStampBeforeCompile;CoreCompile;_TimeStampAfterCompile;AfterCompile;;_AfterCompileWinFXInternal
C:\code\packages\Reinforced.Typings.1.3.0\build\Reinforced.Typings.targets(58,3): error MSB4044: The "RemoveTypeScriptStep" task was not given a value for the
required parameter "Original". [C:\code\src\Web.Site\Web.Site.csproj]
@pavel-b-novikov
Copy link
Member

I have dirty hack to suggest :D

Add to your Reinforced.Typings.settings.xml, at early beginning, right after opening <PropertyGroup> tag, the following lines:

<PublishPipelineCollectFilesCore>Stub</PublishPipelineCollectFilesCore>
<BuiltProjectOutputGroupDependsOn>Stub</BuiltProjectOutputGroupDependsOn>

Check if it is working correctly. If no - remove 2nd line

@PandaWood
Copy link
Contributor Author

Thanks, yes this works for now. I left just the first line in

@masteroleary
Copy link

masteroleary commented Apr 24, 2018

@pavel-b-novikov
While your solution temporarily fixed the immediate issue and allowed dotnet watch to compile, it also made visual studio 2017 unable to find any nuget references (as in ALL packages including system and aspnet.mvc)

I wasn't aware that this was causing it and spent two days dealing with nuget restore, changing the location of the user nuget packages, forcing it to not use global nuget packages, etc. Finally I recreate my branch one change at a time until i found that applying your change and then closing and opening VS caused the issue. Just wanted to put this here and hopefully safe another developer some hair.

Because I believed it was "working" and did remove the MSB4044 error I did not follow your next step, which was to remove the 2nd line. After doing so, all the nuget references and intellisense was restored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants