Skip to content
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

Up to date checking in build #797

Closed
pavel-khritonenko opened this issue Dec 16, 2015 · 7 comments
Closed

Up to date checking in build #797

pavel-khritonenko opened this issue Dec 16, 2015 · 7 comments
Labels
Bug good first issue Impact-Low (Internal MS Team use only) Describes an issue with limited impact on existing code.

Comments

@pavel-khritonenko
Copy link

When last changed item in project is app.config Visual Studio rebuilds that project on every build.

Steps to reproduce (Visual Studio 2015 Update 1):

  1. Create new console F# application
  2. Build it
  3. Build it again, ensure application is up to date
  4. Change app.config (add empty line for example)
  5. Build solution
  6. Build it again, note that application rebuilds every time.
@KevinRansom
Copy link
Member

Yes, I can see that ... thanks for the bug report.

@KevinRansom KevinRansom changed the title Up to date checking Up to date checking in build Dec 16, 2015
@forki
Copy link
Contributor

forki commented Dec 16, 2015

@pavelhritonenko does the same thing happen in C#?

@KevinRansom
Copy link
Member

@forki yes it looks to me like it does.

@vasily-kirichenko
Copy link
Contributor

Another (related?) strange behavior:

-    <None Include="app.config" />
+    <None Include="app.config">
+      <SubType>Designer</SubType>
+    </None>

@forki
Copy link
Contributor

forki commented Dec 17, 2015

Ok so the bug is more general. I assumed this since Vasily's PR didn't
contain any IFs at all.
On Dec 17, 2015 08:07, "Vasily Kirichenko" [email protected] wrote:

Another (related?) strange behavior:

  • + + Designer+


Reply to this email directly or view it on GitHub
#797 (comment)
.

@enricosada
Copy link
Contributor

i think this was already fixed ( #11 ), see my comment in #374 (comment)

let me check for regression, but i think is by design the behaviour of c#

@enricosada
Copy link
Contributor

also #388 ( content in up to date cause rebuild )

@dsyme dsyme added Area-IDE Project System Impact-Low (Internal MS Team use only) Describes an issue with limited impact on existing code. labels Jan 8, 2016
@dsyme dsyme closed this as completed Jul 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug good first issue Impact-Low (Internal MS Team use only) Describes an issue with limited impact on existing code.
Projects
None yet
Development

No branches or pull requests

6 participants