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

Fix the creation of a native environment on MacOS (#79) #118

Merged
merged 1 commit into from
Feb 12, 2015

Conversation

clopez
Copy link
Contributor

@clopez clopez commented Feb 12, 2015

  • Clang is the default compiler on MacOS, so "icecc --build-native"
    should default to build a Clang toolchain on MacOS.
  • Fix on icecc-create-env the detection of the path for gcc.

  * Clang is the default compiler on MacOS, so "icecc --build-native"
    should default to build a Clang toolchain on MacOS.

  * Fix on icecc-create-env the detection of the path for gcc.
@@ -60,6 +60,7 @@
#include <sys/wait.h>

#include "client.h"
#include "platform.h"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have neither platform.h nor determine_platform on linux

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On 12/02/15 14:28, Stephan Kulow wrote:

@@ -60,6 +60,7 @@
#include <sys/wait.h>

#include "client.h"
+#include "platform.h"

I don't have neither platform.h nor determine_platform on linux


Reply to this email directly or view it on GitHub:
https://github.com/icecc/icecream/pull/118/files#r24579917

platform.h is services/platform.h

I tested it on Linux, works fine.

??

@coolo
Copy link
Contributor

coolo commented Feb 12, 2015

that happens if you visit code only once a year :)

coolo added a commit that referenced this pull request Feb 12, 2015
Fix the creation of a native environment on MacOS (#79)
@coolo coolo merged commit 7449982 into icecc:master Feb 12, 2015
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

Successfully merging this pull request may close these issues.

2 participants