Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add SteamCMD #20087

Closed
wants to merge 1 commit into from
Closed

Add SteamCMD #20087

wants to merge 1 commit into from

Conversation

cprecioso
Copy link
Contributor

The wrapper is because of #19999


auto_updates true

binary '__cask_wrapper.sh', target: 'steamcmd'
Copy link
Contributor

Choose a reason for hiding this comment

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

Why the __?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Given that the downloaded tarball may be changed in the server with new files, I wanted to namespace the file just in case.

Copy link
Contributor

Choose a reason for hiding this comment

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

Fair enough.

@vitorgalvao vitorgalvao added the awaiting user reply Issue needs response from a user. label Mar 25, 2016
@cprecioso
Copy link
Contributor Author

Wait, don't merge yet

@cprecioso
Copy link
Contributor Author

There was an error in the wrapper script, arguments weren't being passed to the actual script. I fixed it now, it can be merged.

@vitorgalvao
Copy link
Contributor

Thank you for the contribution. It was merged directly as ce6375c to keep commit history cleaner. Your contribution is still credited to you.

@adidalal adidalal removed the awaiting user reply Issue needs response from a user. label Apr 12, 2016
@cprecioso cprecioso deleted the steamcmd branch June 30, 2016 12:12
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants