Skip to content

VS2019 msbuild#6500

Merged
RussKie merged 1 commit intogitextensions:masterfrom
gerhardol:bugfix/vs2019-msbuild
May 28, 2019
Merged

VS2019 msbuild#6500
RussKie merged 1 commit intogitextensions:masterfrom
gerhardol:bugfix/vs2019-msbuild

Conversation

@gerhardol
Copy link
Copy Markdown
Member

@gerhardol gerhardol commented Apr 22, 2019

Proposed changes

Prepare to build with VS2019

Test methodology

Build from commandline

Test environment(s)

VS2019 installed in addition to VS2017


✒️ I contribute this code under The Developer Certificate of Origin.

@ghost ghost assigned gerhardol Apr 22, 2019
@ghost ghost added the status: ready label Apr 22, 2019
@RussKie
Copy link
Copy Markdown
Member

RussKie commented Apr 22, 2019

Do you know if it still builds with VS2017?

@gerhardol gerhardol force-pushed the bugfix/vs2019-msbuild branch from 6719a4d to fbe32c4 Compare April 23, 2019 00:23
@gerhardol gerhardol changed the title Bugfix/vs2019 msbuild WIP vs2019 msbuild Apr 23, 2019
@gerhardol gerhardol force-pushed the bugfix/vs2019-msbuild branch from fbe32c4 to 82d021e Compare April 23, 2019 00:42
@gerhardol
Copy link
Copy Markdown
Member Author

Awaiting discussions in upstream before removing WIP

@gerhardol gerhardol force-pushed the bugfix/vs2019-msbuild branch from 82d021e to 430e6a4 Compare April 23, 2019 22:10
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 26, 2019

Codecov Report

Merging #6500 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6500      +/-   ##
==========================================
+ Coverage   47.35%   47.35%   +<.01%     
==========================================
  Files         705      705              
  Lines       52846    52847       +1     
  Branches     6955     6955              
==========================================
+ Hits        25024    25025       +1     
  Misses      26497    26497              
  Partials     1325     1325
Flag Coverage Δ
#production 36.56% <ø> (ø) ⬆️
#tests 97.63% <ø> (ø) ⬆️

@gerhardol gerhardol force-pushed the bugfix/vs2019-msbuild branch 2 times, most recently from e444ec0 to d389fe5 Compare April 26, 2019 22:49
@gerhardol gerhardol mentioned this pull request May 23, 2019
@gerhardol gerhardol force-pushed the bugfix/vs2019-msbuild branch 2 times, most recently from 278095b to 6921f49 Compare May 26, 2019 13:53
Updated hMSBuild to 2.1.0 from https://github.com/3F/hMSBuild/
Use 64bit build where possible
Note: hMSBuild.bat is compressed/compiled compared to the "source"
A later release may include hMSBuild.full.bat that is easier to debug
@gerhardol gerhardol force-pushed the bugfix/vs2019-msbuild branch from 6921f49 to 2827bf4 Compare May 27, 2019 22:14
@gerhardol gerhardol changed the title WIP vs2019 msbuild VS2019 msbuild May 27, 2019
@gerhardol
Copy link
Copy Markdown
Member Author

hMSbuild 2.1.0 released
Tested with VS2019 on AppVeyor too

@RussKie
Copy link
Copy Markdown
Member

RussKie commented May 27, 2019

do we need to update the appveyor script to target vs2019?

@RussKie RussKie closed this May 27, 2019
@RussKie RussKie reopened this May 27, 2019
@RussKie
Copy link
Copy Markdown
Member

RussKie commented May 27, 2019

sorry, wrong button 😊

@gerhardol
Copy link
Copy Markdown
Member Author

AppVeyor explicitly targets vs2017 now. When we raise the requirement to use vs2019 appveyor.yml must be updated too.

@RussKie RussKie merged commit 4777d81 into gitextensions:master May 28, 2019
@gerhardol gerhardol deleted the bugfix/vs2019-msbuild branch May 28, 2019 12:24
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

Successfully merging this pull request may close these issues.

2 participants