Skip to content

Installation into projects using nCrunch #11

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

Closed
JakeGinnivan opened this issue Oct 1, 2013 · 8 comments
Closed

Installation into projects using nCrunch #11

JakeGinnivan opened this issue Oct 1, 2013 · 8 comments

Comments

@JakeGinnivan
Copy link
Contributor

Add AND $(nCrunch) =='' to the condition of importing the targets file

@andreasohlund
Copy link
Contributor

Pull request?;)

Den tisdagen den 1:e oktober 2013 skrev Jake Ginnivan:

Add AND $(nCrunch) =='' to the condition of importing the targets file


Reply to this email directly or view it on GitHubhttps://github.com//issues/11
.

http://andreasohlund.net
http://twitter.com/andreasohlund

@JakeGinnivan
Copy link
Contributor Author

Will do, just recording it for now :)

Sent from my Windows Phone


From: Andreas Öhlundmailto:[email protected]
Sent: ‎01/‎10/‎2013 18:10
To: Particular/GitFlowVersionmailto:[email protected]
Cc: Jake Ginnivanmailto:[email protected]
Subject: Re: [GitFlowVersion] Installation into projects using nCrunch (#11)

Pull request?;)

Den tisdagen den 1:e oktober 2013 skrev Jake Ginnivan:

Add AND $(nCrunch) =='' to the condition of importing the targets file


Reply to this email directly or view it on GitHubhttps://github.com//issues/11
.

http://andreasohlund.net
http://twitter.com/andreasohlund


Reply to this email directly or view it on GitHubhttps://github.com//issues/11#issuecomment-25469464.

@JakeGinnivan
Copy link
Contributor Author

Where is the install.ps1? I cannot find it

@SimonCropp
Copy link
Contributor

@JakeGinnivan there is no install.ps1. the nuget build directory convention is used to import the targets https://github.com/Particular/GitFlowVersion/blob/master/GitFlowVersionTask/GitFlowVersionTask.csproj#L109

But isnt the problem with NCrunch two-fold

@SimonCropp
Copy link
Contributor

@JakeGinnivan any update on this?

@JakeGinnivan
Copy link
Contributor Author

Not yet, not a high priority right now. More focused on trying to make core changes to allow support for GitHubFlow soon :)

@JakeGinnivan
Copy link
Contributor Author

Going to close this, I don't need this fixed as Im using the exe version. If someone else wants it we can reopen

@mkoertgen
Copy link
Contributor

Currently, GitVersion fails to find the .git directory during standard NCrunch builds:

WARNING - WARN [03/19/16 9:36:36:33] Could not determine assembly version:
System.NullReferenceException: Object reference not set to an instance of an object.
   at GitVersion.GitPreparer.GetDotGitDirectory()
   at GitVersion.ExecuteCore.ExecuteGitVersion(String targetUrl, String dynamicRepositoryLocation, Authentication authentication, String targetBranch, Boolean noFetch, String workingDirectory, String commitId)
   at GitVersion.ExecuteCore.TryGetVersion(String directory, VersionVariables& versionVariables, Boolean noFetch, Authentication authentication)

This is plausible because by default, NCrunch shadow copies the projects (without .git of course).

Certainly not a show stopper as you can just disable GitVersion UpdateAssemblyInfo and GetVersion when $(NCrunch) == 1

mkoertgen added a commit to awesome-inc/OneClickBuild that referenced this issue Mar 19, 2016
cf.:
- GitVersion issues with NCrunch: GitTools/GitVersion#11 (comment)
- Skip including SolutionInfo.cs during NCrunch builds (fixes #20)
- Disable ClickOnce during NCrunch (fixes #18)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants