Fix return value of GetTargetPath target

[ghost]

The implementation of Microsoft.NodejsTools.targets has a bug in it that prevents the Node.js project type from being referenced properly by CPS-based projects (Common Project System in VS). Symptoms include the following:

• An additional project reference shows up programmatically when enumerating the project references of the CPS-based project. This "project reference" doesn't actually have a source project and just references the Microsoft.NodejsTools.WebRole.dll file in the Node.js project's bin folder.
• The project reference shows up as unresolved in Solution Explorer.

The root of the issue is in the implementation of the GetTargetPath target. The items returned by that target end up having metadata for OriginalItemSpec. Having that item metadata set is what causes all the issues. The fix is to return the same set of files but without the OriginalItemSpec metadata.

[msftclas]

Hi @mthalman, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

[paulvanbrenk]

👍