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 compile error on 64 bit Linux. Fix some warnings. #556

Merged
merged 7 commits into from Nov 1, 2011

Conversation

leon-li
Copy link

@leon-li leon-li commented Oct 19, 2011

No description provided.

@minggo
Copy link
Contributor

minggo commented Oct 19, 2011

#STATICLIBS = $(STATICLIBS_DIR)/libfreetype.a \

$(STATICLIBS_DIR)/libcurl.a \

$(STATICLIBS_DIR)/libxml2.a \

$(STATICLIBS_DIR)/libpng.a \

$(STATICLIBS_DIR)/libjpeg.a

Why comment these lines? Doesn't it cause error?

@leon-li
Copy link
Author

leon-li commented Oct 19, 2011

Those static libs were compiled on 32 bit system. Cause linking errors on 64 bit system.

@minggo
Copy link
Contributor

minggo commented Oct 19, 2011

Oh, it is the problem. Could you offer 64 bit version?
And should modify the makefile to link correct libs.

@leon-li
Copy link
Author

leon-li commented Oct 19, 2011

Do we really need to bundle those libraries for Linux? For any Linux distribution, it is very trivial to install those libs.

@minggo
Copy link
Contributor

minggo commented Oct 20, 2011

You comment the link, then it can not work on 32 bit and 64 bit.

@leon-li
Copy link
Author

leon-li commented Oct 24, 2011

I will address this a bit later.

@tangzero
Copy link

I have the static libs for 64 bits systems: http://db.tt/0TFvzjLb

@minggo
Copy link
Contributor

minggo commented Oct 28, 2011

tangzero, have you tested on 64-bits linux?

@leon-li
Copy link
Author

leon-li commented Oct 28, 2011

Hi tangzero,
Do you have 64 bit libmodex for CocosDenshion?

@leon-li
Copy link
Author

leon-li commented Oct 28, 2011

I have bundled the 64 bit libs provided by tangzero. Please help me do a test? CocosDenshion does not work at the moment because of the lack of 64 bit libfmodex.

@tangzero
Copy link

I tested on Ubuntu 11.10 64bit.

@minggo
Copy link
Contributor

minggo commented Nov 1, 2011

Thank you.

minggo added a commit that referenced this pull request Nov 1, 2011
Fix compile error on 64 bit Linux.  Fix some warnings.
@minggo minggo merged commit 78facce into cocos2d:master Nov 1, 2011
angeltown pushed a commit to angeltown/cocos2d-x that referenced this pull request Apr 28, 2014
Fix compile error on 64 bit Linux.  Fix some warnings.
angeltown pushed a commit to angeltown/cocos2d-x that referenced this pull request Apr 29, 2014
Fix compile error on 64 bit Linux.  Fix some warnings.
minggo added a commit that referenced this pull request May 4, 2014
Fix compile error on 64 bit Linux.  Fix some warnings.
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

3 participants