-
Notifications
You must be signed in to change notification settings - Fork 651
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
GitVersion 4.0.0 fails on VSTS #1496
Comments
Hmm, looking at the task.json it's looking for the GitVersion.js in the root, but looking at the screenshot it is inside of a scripts folder. So I guess the task.json should be modified to match the actual path of the GitVersion.js. I can submit a PR to fix this, but I don't think I'll be able to release it, right? |
@arturcic I just tried it, but I'm still getting the same error. Since the error also contains the version number, which is still 4.0.0, I'm wondering if the version number on the task has been updated. If not I believe that VSTS will still use the old version (due to caching). |
Ok then, I'll update that as well |
@jmezach Updated, moved the files to the same location as task.json, can you try the new version? |
Okay, it picked up the new version of the task now, but now I'm getting a different error:
Are the node_modules part of the VSIX? |
looks like we need to include the node_modules as well |
@jmezach can you try the new version? this one included node_modules |
Okay, so it looks like it's working now, at least on Windows. It still fails on macOS and Linux though. Not entirely sure if that is supported at all. I thought it was, but I guess it's not, but I should probably open a separate issue for that. |
For unix you need mono installed |
I'll create a PR for this issue with the changes I've done so far, and I'll close the issue. |
Sounds good. I'll file another issue for the macOS and Linux failures. |
@yohanb , can you check the new version on your agent, I guess you have mono installed there. |
@arturcic why is mono required if it's compiled as .NET Core? |
The version that is used in the VSTS package is a .NET Framework package - il-merged. That was the way it was packaged before v.4.0.0, and I did not changed that, so it's still running with mono. Probably we might consider using the .NET core version, but that will add a dependency on .net core to be installed, now it's on mono |
When will this be released? |
a PR for this change is welcomed. |
Perhaps this is also the reason for #1497. I guess taking a dependency on .NET Core isn't that weird, since the hosted agents on VSTS (including Windows, macOS and Linux) all have .NET Core SDK's installed. |
Now that the 4.0.0 version of GitVersion has been published to the Marketplace I've been trying to use it in my projects, but unfortunately I'm getting an error when using the 4.0 version of the build task:
This looks like a packaging issue with the VSTS extension. Looks like the build task is written in TypeScript, so I'm guessing that the TypeScript wasn't transpiled to JavaScript at the time of the release. I believe @arturcic published the 4.0.0 version of the VSTS task as a part of #1445.
The text was updated successfully, but these errors were encountered: