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

The default seems to be BuildTools_ instead of Secret_ #300

Merged
merged 1 commit into from
Mar 19, 2022

Conversation

jtorvald
Copy link
Contributor

As mentioned in this issue #272 the default in the code is BuildTools_ and not Secret_ as in Mobile Builds tools version 1.x

As mentioned in this issue dansiegel#272 the default in the code is BuildTools_ and not Secret_ as in Mobile Builds tools version 1.x
Copy link
Owner

@dansiegel dansiegel left a comment

Choose a reason for hiding this comment

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

Good catch. Thanks for the update

@dansiegel dansiegel merged commit 2819874 into dansiegel:master Mar 19, 2022
@jtorvald
Copy link
Contributor Author

@dansiegel maybe it's good to also add Secret_ as an alias to BuildTools_ because everybody that upgrades from v1 to v2 will run into this problem I think.

@dansiegel
Copy link
Owner

It was changed because some agents like Azure DevOps / AppCenter Windows build agents actually prevent variables with the prefix Secret_ from being read. They also do a ToUpper() which was a major reason why the definitions were added to the config.

@jtorvald
Copy link
Contributor Author

I understand. Good to know the reason behind the chance. Thanks for the explanation.

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