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

Add AppVeyor support #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add AppVeyor support #20

wants to merge 2 commits into from

Conversation

JayFoxRox
Copy link
Owner

@JayFoxRox JayFoxRox commented Jun 8, 2018

This adds support for AppVeyor builds and deployment of the 32 bit MinGW builds to this repository.

Problems which need an issue after merge:

  • MSVC build is broken. I could not make this work due to various errors with trying to put it in x86 mode; but not even x64 compilation worked. I have no idea why that is.
  • The binaries are not static yet, so they might not even work without the respective libraries.
  • The MinGW build puts a "a.exe" in it's output folder. I'm not sure where this one comes from, but it's the reason why I only explicitly copy "swe1r-.exe" instead of ".exe".
  • swe1r-patcher.exe might be confusing to people (who are looking for swe1r-loader.exe). Ideally these tools should be merged into a single tool, or a warning should be displayed if swe1r-patcher.exe is used incorrectly.

Add string display when a trigger is executed
@JayFoxRox JayFoxRox force-pushed the appveyor branch 7 times, most recently from 4a2b03a to 9ee821d Compare June 8, 2018 17:32
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.

1 participant