-
Notifications
You must be signed in to change notification settings - Fork 113
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
class BWAPI::Race has no member "Protoss" #81
Comments
I have the same problem. I run on ubuntu18, bwapi install from github git clone. |
Maybe you should change "CCRace::Protoss" to "BWAPI::Races::Protoss" in BuildManager.cpp |
I also have this problem, and I foud the code of I clone the version of 2017-12-04, it's build success and work fine. |
The line of code you're referencing is not the same as what's in the repo. When did you clone it? I cannot reproduce any of these errors |
@davechurchill oh ,sorry, I make a mistake, the original code is |
I will close this for now as it appears to be fixed? |
@davechurchill Sorry, maybe my English is not good, causing misunderstanding. The original code from github: The code I compile the CommandCenter project with the Win32 Platform. |
What is the build error message? |
The error message is: |
That is impossible though. That line is a single bracket https://github.com/davechurchill/commandcenter/blob/master/src/BuildingManager.cpp#L422 |
I just copy the Migdalin's message. |
I confirm that the error is still present. After changing that line of code to: |
Can you submit a pull request with that fix?
…On Sat., Dec. 28, 2019, 7:22 p.m. vlad-gregor, ***@***.***> wrote:
I confirm that the error is still present. After changing that line of
code to:
if (m_bot.GetPlayerRace(Players::Self) == BWAPI::Races::Protoss)
Then the project builds fine.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#81?email_source=notifications&email_token=AAJCJKA3CWGXPNXIY3FRW33Q27KBJA5CNFSM4GIUYAB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHYTUQI#issuecomment-569457217>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJCJKAVYURFBPKTRXLE7X3Q27KBJANCNFSM4GIUYABQ>
.
|
I think I just did that. However, I have to say, that I'm not able to check, if the code compiles with "x64" build configuration, as I do not have SC2API installed. |
Compile time error. Only a problem with the Win32 build, but maybe that's too obvious.
The text was updated successfully, but these errors were encountered: