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

Move build settings out of the project file. #599

Merged
merged 2 commits into from Jan 24, 2012

Conversation

tiennou
Copy link
Member

@tiennou tiennou commented Nov 30, 2011

As it says on the tin ;-).

@HenningJ
Copy link
Contributor

@HenningJ HenningJ commented Nov 30, 2011

:-)

While you're at it, you could also remove ppc from valid archs, and remove ~/Library/Frameworks from FRAMEWORK_SEARCH_PATHS

@pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Jan 20, 2012

I can go ahead and remove ~/Library/Frameworks when I merge this if everybody's happy.

@henning?

@HenningJ
Copy link
Contributor

@HenningJ HenningJ commented Jan 24, 2012

yes, happy :-)

@pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Jan 24, 2012

Just a bit too late for ß64, but since it doesn't change the app itself, it doesn't really matter :)

I'll wait until Rob's finished fiddling with commits...

@henning
Copy link

@henning henning commented Jan 24, 2012

Please take me out of this thread. Thanks.

You accidently forgot the j when mentioninh henningJ....

Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.

Patrick Robertson reply@reply.github.com schrieb:

Just a bit too late for ß64, but since it doesn't change the app itself, it doesn't really matter :)

I'll wait until Rob's finished fiddling with commits...


Reply to this email directly or view it on GitHub:
#599 (comment)

@burningTyger
Copy link

@burningTyger burningTyger commented Jan 24, 2012

@henning you can disable notifications for this thread a t the bottom of the issue page.

pjrobertson added a commit that referenced this issue Jan 24, 2012
Move build settings out of the project file.
@pjrobertson pjrobertson merged commit 880f97a into quicksilver:master Jan 24, 2012
pjrobertson added a commit that referenced this issue Jan 24, 2012
Performed as per discussion at #599 (with Henning's blessings)
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.

None yet

5 participants