-
Notifications
You must be signed in to change notification settings - Fork 2
PR for Issues #2, #3, and #4 #5
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
base: master
Are you sure you want to change the base?
Conversation
Mike-E-angelo
commented
Dec 27, 2014
- Added .targets file to source. (Issue AssemblyReferencesTT.targets is Missing from Source #4)
- Updated targets to emit References.tt (non-hintpath references). (Issue Include References Too. :) #3)
- Added .ps1 files to tools path in .nuspec
- Removed Custom Tool from .tt files (Issue Ensure that CustomTool is not set on project item #2)
|
Thanks for your pull request. I like your changes. I'm not sure I understand what you are doing with the References.tt? I thought I had that covered in the line 28 of the (missing lol) targets file? The difference - I output the path of the referenced dll file (fragile guess at it's location) - you just output the name of the assembly? what is the use case for your approach? |
|
The difference is how the I also realized that I forgot to do this on line 21 as well. |
|
Ah! I purposely avoided using macros as they do not work outside in the command line. In stead of using macros i would have it generate the file on pre-build. See how this project came about: http://stackoverflow.com/questions/25990163/using-project-references-as-assembly-paths-in-t4 |
|
AH ok. Well then I guess I haven't thought this through enough, then. If you check this into source control, doesn't the full path go along with the file? And also, have you tried this in a automated build/continuous integration scenario? I believe AssemblyReferences.tt will be read-only and will generate an exception on build, as this task writes to a read-only file... |
|
I have tried it in a minimal build setup in conjunction with https://github.com/clariuslabs/TransformOnBuild I am aware that the file will be ever changing and that you would probably not want the AssemblyReferences in your VCS. See issue #7 Re: read-only - this would probably only be an issue if you are using TFS as your VCS. |
|
Alright, cool. Yeah, T4 is rather a pain when it comes to this stuff. So incredibly frustrating as most of this has already been handled and solved by .*proj files. As for read-only + TFS... you might just be right on that. I am also thinking about Azure continous deployment but I have no experience with that just yet. Once I do I will let you know how I fare. Thanks again! |