Skip to content

appveyor: upgrade VS2017 to VS2019 - #4383

Closed
bagder wants to merge 3 commits into
masterfrom
bagder/appveyor-vs2019
Closed

appveyor: upgrade VS2017 to VS2019#4383
bagder wants to merge 3 commits into
masterfrom
bagder/appveyor-vs2019

Conversation

@bagder

@bagder bagder commented Sep 19, 2019

Copy link
Copy Markdown
Member

No description provided.

Comment thread appveyor.yml Outdated
@bagder

bagder commented Sep 19, 2019

Copy link
Copy Markdown
Member Author

Ugha, that failed so spectacularly I'm afraid I don't know where to go next...

@jay

jay commented Sep 19, 2019

Copy link
Copy Markdown
Member

It looks like there's an extra space
Configuration='Debug' Platform=' x64'.

my guess "-A x64" is quoted and it should be -A x64 but you could probably do -Ax64 if you need to quote TARGET

@bagder

bagder commented Sep 19, 2019

Copy link
Copy Markdown
Member Author

hm, yes. So where does that originate from? The documentation says -A x64 so surely that's not the space?

@bagder

bagder commented Sep 19, 2019

Copy link
Copy Markdown
Member Author

Hm, I bet the quotes make it so...

@jay

jay commented Sep 19, 2019

Copy link
Copy Markdown
Member

I updated my comment after I made it, my guess you are quoting so it sees it as one argument
"-A x64"
then I guess cmake parses it as -A " x64"

bagder added a commit that referenced this pull request Sep 20, 2019
@bagder bagder mentioned this pull request Sep 20, 2019
@bagder

bagder commented Sep 21, 2019

Copy link
Copy Markdown
Member Author

I'm testing this out in the #4387 PR

@bagder bagder closed this in 0b7d7ab Sep 21, 2019
@bagder
bagder deleted the bagder/appveyor-vs2019 branch September 26, 2019 11:53
@lock lock Bot locked as resolved and limited conversation to collaborators Dec 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants