-
Notifications
You must be signed in to change notification settings - Fork 123
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
Fixes compilation errors on Mac OS X #273
Conversation
@mpranj you should consider enabling git pull rebase per default: |
@@ -98,7 +98,11 @@ KeySet *elektraDocu = ksNew(20, | |||
#include "readme_elektrify-getenv.c" | |||
KS_END); | |||
|
|||
pthread_mutex_t elektraGetEnvMutex = PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP; | |||
#if defined(__APPLE__) && defined(__MACH__) | |||
pthread_mutex_t elektraGetEnvMutex = PTHREAD_RECURSIVE_MUTEX_INITIALIZER; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ohh, is PTHREAD_RECURSIVE_MUTEX_INITIALIZE really available? I think we can make resolver code much shorter then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it appears it's available on OS X (not tested extensively yet), don't know about other BSDs
Current state: 99% tests passed, 1 tests failed out of 69 can you check this on the build server please, I want to make sure I didn't break anything else |
libgetenv should be deactivated at CMake Level for non-glibc systems. (But I can do it) I will run it on the build server. But something is broken in the pull request, e.g. the changes in doc/Doxyfile, doc/NEWS.md, examples/ksCallback.c do not seem intended. Can you fix this please? I would not recommend rebasing on merge conflicts. You can also run |
add to whitelist |
jenkins start build please |
@@ -142,7 +142,7 @@ check_resolver dir b /a/b /tmp/a/b | |||
check_resolver dir b a /tmp/@KDB_DB_DIR@/a | |||
check_resolver dir b a/b /tmp/@KDB_DB_DIR@/a/b | |||
|
|||
T=`mktemp -d` | |||
T=`mktemp -d 2>/dev/null || mktemp -d -t 'libelektra-test'` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you define that as mktemp-elektra function?
@markus2330 I also had to disable libgetenv for mingw. My local modifications include:
|
just tested 86b3797. works for mingw |
Any progress here? Does Elektra compiles and run on Mac OS X? Please give me a status update, this issue is stalling the release. |
@mpranj please create a new branch that only contains the Mac OS X fixes for this pull request. |
this patch only fixes compilation errors on Mac OS X