-
Notifications
You must be signed in to change notification settings - Fork 4
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
extconf.rb #11
Comments
is it my problem when ruby with clang on osx is broken? (i currently havnt a osx machine to test it) also in your issue its difficult to read what changes i need to do, |
I need to figure out how to make a pull request. I'm using Xcode 5 and the apple clang. On OS X wx uses clang directly while ruby uses gcc and "clang"!="gcc" however clang -v 2>&1 and gcc -v 2>&1 have matching lines as they are the same compiler. Sent from my iPhone
|
pull changes from https://github.com/rinkevichjm/rwx.git On Wed, May 27, 2015 at 11:52 PM, James Rinkevich [email protected]
|
The changes now pass those tests some sort of issue with pkg_config of gtk files. Were you having problems with that? this should fix those too. |
the correct test for clang with libc++ is have_macro("_LIBCPP_VERSION","ciso646") # The C++ specification On Thu, May 28, 2015 at 8:56 PM, James Rinkevich [email protected]
|
i added have_macro("_LIBCPP_VERSION","ciso646") for extra checking but i need to make the stdlib option optional because i cant check if wxwidgets is build against it or not, and if it isnt it will cause runtime errors. means i got 2 * 2 different configurations (and even more) because of yes/no libc++ and yes/no c++11 PS: you did compile wxwidgets and maybe ruby on purpose with libc++ and c++11 right? because without everything would be fine. |
I have MAC OS X Yosemite so clang is built with libc++. I wish I knew what the build set up for that is. I look around as it may be a configure or cmake option. Sent from my iPhone
|
Looking ing the mac ports port file for clang 3.6 suggests that setting the option --cxx_stdlib=libc++ will cause clang to be built with Libc++ as the default library, so you could configure build and install that. I'd think that if you could build a binary of that and just install the binary for tests that would be what is needed. Sent from my iPhone
|
you still not know what i mean right? setting --cxx_stdlib=libc++ as default doesnt solve my problem and the problem is if wxwidgets is build with libc++, not clang |
But you need to be able to test with clang as is being built on OS X or FreeBSD. Where if --stdlib is not set libc++ is used. No? Sent from my iPhone
|
No i mean i can build rwx with or without --stdlib, both of them does work BUT if i start the program, AND wxwidgets does not have the SAME stdlib setting TOO then it does crash. means i need to check somehow what setting wxwidgets has |
test have_macro "HAVE_TR1_TYPE_TRAITS","wx/setup.h" # if this exists you On Thu, Jun 4, 2015 at 8:34 PM, Hans Mackowiak [email protected]
|
checking for "HAVE_TR1_TYPE_TRAITS" and "HAVE_TYPE_TRAITS" is not check enough for libc++ but for c++11 (libc++ can be build without c++11 too) but HAVE_ABI_FORCEDUNWIND is something that does not exist on libc++ (i can use that for detecting libc++) i added checks for both settings (and mixed), so it should work for your system now |
checking for HAVE_TR1_TYPE_TRAITS in WX's setup.h is because WX's configure writes wx/setup.h, via auto configure test using the correct compiler setup. probably should flag an issue for wxwidgets team to fix configure to save -stdlib= flag from env CXXFLAGS for wx-config --cxxflags. The makefiles run with it but it but it isn't saved for wx-config Sent from my iPhone
|
no, adding libc++ does not automaticlly enable c++11, did you test it if that latest changes does works for you? |
checking for HAVE_TYPE_TRAITS in wx/setup.h will tell you if c++11 is set
|
that is what i already told you about, can you test it yet that if the newest changes makes it working on your osx? |
prebuilt versions of ruby sometimes contain paths to nonstandard folders. need removing in $LDFLAGS
clang on osx masquerades as gcc need to test output of -v
WX uses 3 C++tr1 header versus C++11 header flags that need testing for to set (twice for 2 of them).
--------diffs ----------
@@ -20,8 +20,8 @@ if(wx_config = find_executable('wx-config'))
abort("wx version outdated, please update to 3.0.0 or newer")
end
@@ -32,19 +32,67 @@ if(wx_config = find_executable('wx-config'))
end
@@ -56,30 +104,52 @@ if(wx_config = find_executable('wx-config'))
extra_libs.each {|l|
pkg = pkg_config(l)
#because pkg forgot to add the include paths to cxx flags
puts L
puts pkg[0]
$CXXFLAGS << " " << pkg[0] if pkg && !$CXXFLAGS[pkg[0]]
}
@@ -104,7 +174,7 @@ if(wx_config = find_executable('wx-config'))
have_const("wxFD_NO_FOLLOW","wx/filedlg.h")
have_const("wxDIRCTRL_DEFAULT_STYLE",["wx/wx.h", "wx/dirctrl.h"])
have_func("wxDirCtrl()",["wx/wx.h", "wx/dirctrl.h"])
have_const("wxSTC_LEX_DMAP",["wx/wx.h", "wx/stc/stc.h"])
have_const("wxSTC_LEX_DMAP",["wx/wx.h", "wx/stc/stc.h"])
@@ -124,7 +194,7 @@ CONFIG["warnflags"].gsub!(
"-Wextra" #wxAUI is a bit buggy
), "")
with_cppflags("-std=c++11") {
with_cppflags("-std=c++11") {
create_header
create_makefile "rwx"
}
}
The text was updated successfully, but these errors were encountered: