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

With Visual Studio 2017 doesnt work #13

Open
helirrodriguez opened this issue Apr 3, 2019 · 6 comments
Open

With Visual Studio 2017 doesnt work #13

helirrodriguez opened this issue Apr 3, 2019 · 6 comments

Comments

@helirrodriguez
Copy link

with Visual Studio 2017 doesn't work, a lot of problems and identifiers with no definitions, etc

@jupe
Copy link
Owner

jupe commented Apr 3, 2019

Could you try this Pr: #12 .
I haven’t had time to keep this project up-to-date so most probably it doesn’t work out of the box anymore with recent visual studio..Originally I did use code:blocks..

@helirrodriguez
Copy link
Author

helirrodriguez commented Apr 3, 2019 via email

@jupe
Copy link
Owner

jupe commented Apr 3, 2019

there aren't any software packet that plotting easily in c++

really? even still no good options ?😯 All help here would be more than welcome ! I also would like to set up CI for this project but unfortunately haven’t free time much...

@helirrodriguez
Copy link
Author

helirrodriguez commented Apr 4, 2019 via email

@Anton-V-K
Copy link

Anton-V-K commented May 27, 2019

I've successfully built the library and the sample with VS2017 (using v140 toolset and wxWidgets-3.1.2) with minimal changes to projects:

  • Set character set to Unicode
  • C++ preprocessor defines are needed: __WXMSW__;wxMSVC_VERSION_AUTO;wxUSE_UNICODE;WXUSINGDLL;wxMSVC_VERSION=140
  • C++ include directories should be specified: $(WX)\include\msvc;$(WX)\include

Specific changes for the sample project:

  • Reset libraries list for the linker to project's defaults - they are choosen automagically when you specify wxMSVC_VERSION_AUTO and wxMSVC_VERSION=140
  • Specify additional libraries directories for the linker: $(WX)\lib\vc$(PlatformToolsetVersion)_$(PlatformTarget)_dll (the path may be different for your setup)

When running the sample there are a couple of asserts complaining about the sizer setup - they may be fixed with changes in gpSizer::Realize() (wxALIGN_XXX should be commented out, since they are ignored):

wxBoxSizer *newsizer = new wxBoxSizer(wxHORIZONTAL);
it->Rewind();
while( it->NextWindow(window) )
{
    newsizer->Add( window.GetWindow(),
                   window.GetProportion(),
                   wxALL|wxEXPAND/*|wxALIGN_CENTER_HORIZONTAL|wxALIGN_CENTER_VERTICAL*/, m_border);

}
m_topSizer->Add(newsizer, 1, wxALL|wxEXPAND/*|wxALIGN_TOP|wxALIGN_BOTTOM*/, 0);

@helirrodriguez
Copy link
Author

helirrodriguez commented May 30, 2019 via email

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

No branches or pull requests

3 participants