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
Remove CustomTasks #453
Remove CustomTasks #453
Conversation
This removes CustomTasks, instead relying on the tasks from MSBuildCommunityTasks that we are already using.
What about generating a single CurrentVersion.Generated.cs instead of one per project? Maybe something to think about for the future. |
That's something I was thinking as well. I'll try and work that out. |
…thon2 into slide-remove_custom_tasks
The main issue I see with creating only one file is building in Visual Studio. I could easily generate the files in Build.proj as a pre-Target to other targets, but in Visual Studio, I don't know of a way to do that because it may be that only one project is built if only one project has changes. So, in that case, do I generate it every time a project is being built, which would then cause all projects needing to be built? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't the template generation target be skipped because the the inputs haven't changed? Anyway, I'm fine with the PR as is.
* Small updates * [WIP] Remove CustomTasks This removes CustomTasks, instead relying on the tasks from MSBuildCommunityTasks that we are already using. * Pre-create the Properties directory * Make directories if they don't exist
* IronLanguages/ironpython2#367 * IronLanguages/ironpython2#368 * IronLanguages/ironpython2#452 and IronLanguages/ironpython2#406 * IronLanguages/ironpython2#413 * IronLanguages/ironpython2#372 * IronLanguages/ironpython2#453 * IronLanguages/ironpython2#435 * IronLanguages/ironpython2#458 * IronLanguages/ironpython2#383 * Update DLR * Fix copy/paste issues * IronLanguages/ironpython2#423
This removes CustomTasks, instead relying on the tasks from MSBuildCommunityTasks that we are already using.