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

ARM builds #3

Merged
merged 2 commits into from
Dec 1, 2015
Merged

ARM builds #3

merged 2 commits into from
Dec 1, 2015

Conversation

CRImier
Copy link
Contributor

@CRImier CRImier commented Jul 19, 2015

Referencing arduino/Arduino#3549 in Arduino IDE repo - this is the script that I used to compile libastyle.

cmaglie added a commit that referenced this pull request Dec 1, 2015
@cmaglie cmaglie merged commit 794929c into arduino:master Dec 1, 2015
@cmaglie
Copy link
Member

cmaglie commented Dec 1, 2015

LGTM, I didn't actually tried it but it should not harm anyone.

@NicoHood
Copy link

NicoHood commented Dec 2, 2015

Do we have a chance that someone updates the ARM PR for arduino now?

@NicoHood
Copy link

NicoHood commented Dec 5, 2015

It turns out that this "someone" is me now.

I had to compile astyle for arm now. I noticed that it gave me an error if I crosscompile the file from a linux x64 distribution. I am currently searching for a solution. We somehow need to pass the arch to the compiler. I tried -march=armv7 without success.

We should fix this, once a solution is there.

@NicoHood
Copy link

NicoHood commented Dec 5, 2015

This is a solution, I just dont know how to integrate it (so linux builds dont break instead). Maybe they should add an arm folder upstream?

# The arm compiler is only required for cross compiling
sudo apt-get install gcc-arm-linux-gnueabi g++-arm-linux-gnueabi gcc-arm-linux-gnueabihf g++-arm-linux-gnueabihf

# For cross compiling change the CXX compiler flag to:
nano ./astyle-code/AStyle/build/gcc/Makefile
CXX = arm-linux-gnueabihf-g++

@cmaglie
Copy link
Member

cmaglie commented Dec 7, 2015

@NicoHood
Copy link

NicoHood commented Dec 7, 2015

I tried this, but it did not seem to overwrite the setting. Maybe I did something wrong.

@ShorTie8
Copy link

I found to be able to compile it I had to set the JAVA_HOME variable some where

JAVA_HOME=readlink -f /usr/bin/javac | sed "s:bin/javac::"

@NicoHood
Copy link

Huh? We are talking about cross compiling. What does this have to do with selecting an arm compiler?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants