Skip to content

Update gradle builds#281

Merged
timotei merged 17 commits intogluck:masterfrom
ngyukman:master
Feb 22, 2021
Merged

Update gradle builds#281
timotei merged 17 commits intogluck:masterfrom
ngyukman:master

Conversation

@ngyukman
Copy link
Copy Markdown
Contributor

To support builds by latest msbuild

Change-Id: I81c2745afc61f7fd29dc34e88ea0b463f33c3238
Change-Id: Ia63fa35d22ca1c97f4eb1277ccb08ada5d157a81
Change-Id: Ife00b5f45d180d92e57297b45a7c5b3e73f62e47
TLS 1.2 is now required to download package from nuget repository

Change-Id: I7809ef17a25261c360e36336ef371221e7dda734
Change-Id: Id273591c91b8a24315663e4f337936e9ac05fe7e
Change-Id: I0313da92372eb623a3dea7b746f927c69cbe7a71
@ngyukman ngyukman mentioned this pull request Feb 19, 2021
Comment thread ILRepack.Tests/ILRepack.Tests.csproj
timotei and others added 3 commits February 19, 2021 12:28
Change-Id: I62c9bf4c9628604aef77f44bdb5d4ca3c4d4e782
Change-Id: I91d2383eaeaaf55cd30645245dc2cf3700ed4036
Copy link
Copy Markdown
Contributor

@florian-gandon florian-gandon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good for me.

@ngyukman
Copy link
Copy Markdown
Contributor Author

not sure why gradle cannot be downloaded yesterday https://ci.appveyor.com/project/gluck/il-repack/builds/37847904
it's all good now

not sure why it doesn't appear in build check now
@ngyukman
Copy link
Copy Markdown
Contributor Author

looks really hard to resolve the msbuild issue on mono with the following error, dotnet/msbuild#2554
probably need to skip finding msbuild in ProjectFileParser, and that should be passed the result from vswhere

@ngyukman
Copy link
Copy Markdown
Contributor Author

ok finally it is building in travis
It's because MSBuildSDKsPath is missing...

https://travis-ci.org/github/gluck/il-repack/builds/759767020

Comment thread .travis.yml
dotnet: 3.1.301

env:
- MSBuildSDKsPath=/usr/share/dotnet/sdk/3.1.301/Sdks
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, this error. it was supposed to be fixed following dotnet/sdk#9632 and the related bugs though 🤔 But oh well

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.

3 participants