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

Appveyor build script #3

Merged
merged 1 commit into from
Dec 21, 2018

Conversation

Dahlgren
Copy link
Contributor

The MSVC builds can be downloaded directly from Appveyor and be used on Windows.

Sadly Appveyor does not yet offer Linux Subsystem so I couldn't add that as a target.

@jonpas
Copy link
Collaborator

jonpas commented Dec 10, 2018

Do they not provide VS 2017?

@Dahlgren
Copy link
Contributor Author

Dahlgren commented Dec 10, 2018

Sure, should I change to 2017?
Changed to 2017. It could also build with both 2015 and 2017 if desired.

@jonpas
Copy link
Collaborator

jonpas commented Dec 10, 2018

One is enough, and 2017 is nothing new, so all good. :)

@jonpas
Copy link
Collaborator

jonpas commented Dec 11, 2018

Can we name artifacts armake_w32.exe and armake_w64.exe, to mirror what armake1 had.

@Dahlgren
Copy link
Contributor Author

@jonpas sure, I can fix that

@Soldia1138
Copy link
Contributor

Wouldn’t armake2_w32.exe and armake2_w64.exe be even better?

@jonpas
Copy link
Collaborator

jonpas commented Dec 11, 2018

Ooh yes, that is what I meant!

@KoffeinFlummi KoffeinFlummi merged commit 45a235e into KoffeinFlummi:master Dec 21, 2018
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.

4 participants