Skip to content

Remove mention of example #550

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mjsabby
Copy link

@mjsabby mjsabby commented Mar 20, 2025

This poorly placed comment is causing people to download Windows binaries on macOS.

This poorly placed comment is causing people to download Windows binaries on macOS.
@prezaei
Copy link

prezaei commented Mar 20, 2025

Looks to me what is missing is something like this:

export USE_NET6_ARTIFACTS_CREDENTIAL_PROVIDER=false
export USE_NET8_ARTIFACTS_CREDENTIAL_PROVIDER=true
export ARTIFACTS_CREDENTIAL_PROVIDER_RID=osx-arm64

@embetten
Copy link
Contributor

embetten commented Mar 20, 2025

@mjsabby the examples download the current default platform agnostic, non-self-contained version of the credprovider that requires the dotnet runtime. The self-contained platform specific binaries were just released in the latest version this week. We are still in an opt-in phase while we confirm the experience. While we work to make the self-contained and the .NET 8 experience the default, please use the variables mentioned by @prezaei.

Looks to me what is missing is something like this:

export USE_NET6_ARTIFACTS_CREDENTIAL_PROVIDER=false
export USE_NET8_ARTIFACTS_CREDENTIAL_PROVIDER=true
export ARTIFACTS_CREDENTIAL_PROVIDER_RID=osx-arm64

As I am planning to update the README with more detailed self-contained examples for both windows or non windows, shall we close ?

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