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

(GH-129) When using .NET Global tool, resolve Cake.dll to netcoreapp3.1 #130

Conversation

augustoproiete
Copy link
Member

Bakery currently needs Cake.Core and Cake.Common targeting netstandard2.0 only as per @bjorkstromm's comment #129 (comment)

gep13 and others added 3 commits March 6, 2021 11:49
* master:
  (build) Add dotnet cli environment variables
* hotfix/0.6.1:
  (cake-buildGH-127) Fixed wrong path in cake-tool detection
…etcoreapp3.1

Bakery currently needs Cake.Core and Cake.Common targeting netstandard2.0 only
cake-build#129 (comment)
Copy link
Member

@bjorkstromm bjorkstromm left a comment

Choose a reason for hiding this comment

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

LGTM!

@augustoproiete, have you tested that this works?

@nils-a
Copy link
Member

nils-a commented Mar 8, 2021

I have tested it and it does work - but only for Cake 1.1.0. (Neiter Cake 1.0.0, nor Cake 0.38.5 will work - since none of them ship netcoreapp3.1)

Would it be possible to change the glob from netcoreapp3.1 to something matching netcoreapp3*? If we manage that I guess the change would be compatible with older versions, too.

@augustoproiete augustoproiete force-pushed the force-netcoreapp3-1-for-dotnet-global-tool branch from b5d09df to 9cf5942 Compare March 8, 2021 14:12
@augustoproiete
Copy link
Member Author

@bjorkstromm Yes, tested with 1.1.0.

@nils-a Good catch. I updated the PR earlier this morning to resolve netcoreapp2.1 as well:

Cake Tool 1.1.0 resolves to:

C:/Users/augustoproiete/.dotnet/tools/.store/cake.tool/1.1.0/cake.tool/1.1.0/tools/netcoreapp3.1/any/Cake.dll

  • Cake Tool 0.38.5 resolves to:
C:/Users/augustoproiete/.dotnet/tools/.store/cake.tool/0.38.5/cake.tool/0.38.5/tools/netcoreapp2.1/any/Cake.dll

Copy link
Member

@bjorkstromm bjorkstromm left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks @augustoproiete

Copy link
Member

@nils-a nils-a left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

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

LGTM!

@gep13
Copy link
Member

gep13 commented Mar 8, 2021

@augustoproiete I have rebased and merged this into a hotfix branch here: c1e22a1

As a result, going to close out this PR.

@gep13 gep13 closed this Mar 8, 2021
@gep13 gep13 deleted the force-netcoreapp3-1-for-dotnet-global-tool branch March 8, 2021 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants