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

Added support for non-Windows build environments. #394

Closed
wants to merge 1 commit into from

Conversation

wizzwizz4
Copy link

Shebangs are supported by practically everything, and if you've got DOSBox then this will make it run! Problem solved, thank you very much. This fixes everything with no side-effects whatsoever.

Just merge this please. No need to check that it runs; I know it does and you can see that I'm not sneaking anything past this or anything like that. You can thank me later.

Shebangs are supported by practically everything, and if you've got DOSBox then this will make it run! Problem solved, thank you very much. This fixes everything with no side-effects whatsoever.
@KazWolfe
Copy link

-1, we are a serious enterprise using only the most serious tools. Linux and OS X are not serious business operating systems, and therefore will not be supported by the serious business platform.

@wizzwizz4
Copy link
Author

Whilst Windows is the One True Operating System™, Ubuntu is used by Facebook, so it's definitely a business operating system.

Plus, there's nothing wrong with my code so there's no reason not to merge it; it works and it fixes problems nad it makes it so much better.

@emiln emiln closed this Jul 5, 2020
@emiln
Copy link
Member

emiln commented Jul 6, 2020

As a company we are deeply grateful for the many open source contributions we're receiving, but we can no longer in good faith support merge requests into the problematically named master branch, which has been removed.

Instead we hope that you will reopen your merge request and target the new default branch, artisanally crafted by our Swedish Inclusivity Consultant, called uinverse. You can read more about the inclusivity success story in the issue.

@wizzwizz4
Copy link
Author

@emiln I made #484. Can you merge it please/

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.

None yet

3 participants