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

Compile BrickPiFW_Compressed_Communication.ino get error. #20

Open
timesong opened this issue Sep 5, 2014 · 8 comments
Open

Compile BrickPiFW_Compressed_Communication.ino get error. #20

timesong opened this issue Sep 5, 2014 · 8 comments

Comments

@timesong
Copy link

timesong commented Sep 5, 2014

I tried to compile BrickPi/Firmware_BrickPi/Firmware_2.0/BrickPiFW_Compressed_Communication/BrickPiFW_Compressed_Communication.ino, but always get below error messages:

BrickPiEV3/BrickPiEV3.cpp.o: In function EV3_Setup_Touch(unsigned char)': /Users/timesong/Documents/Arduino/libraries/BrickPiEV3/BrickPiEV3.cpp:246: multiple definition ofnxt_debounce'
BrickPiA/BrickPiA.cpp.o:/Users/timesong/Documents/Arduino/libraries/BrickPiA/BrickPiA.cpp:33: first defined here

Is this a issue?

@timesong timesong changed the title Compiled BrickPiFW_Compressed_Communication.ino get error. Compile BrickPiFW_Compressed_Communication.ino get error. Sep 5, 2014
@chahld
Copy link

chahld commented Sep 7, 2014

I fixed this by deleting the definition of nxt_debounce in the BrickPiA.cpp. It isn't referenced anywhere in that file, so seems like a typo...

@timesong
Copy link
Author

Sure?I fixed following your solution, but the compiled firmware can't work.

@chahld
Copy link

chahld commented Sep 11, 2014

can you include a bit more info? are you saying that it compiled? but that when you try to use it, it doesn't work? (did you burn the new firmware onto your device? etc?)

Honestly I'm not sure what the state of this codebase is. Since it has compile problems, it is possible that the head version that is checked in is bad in more than just this one way...

@DexterInd
Copy link
Collaborator

Hey Zhang, you said the firmware doesn't work? How so?

@chahld , I will open up the code and try to find the problem, recompile to make sure we didn't mess it up with a checkin or something.

@DexterInd DexterInd reopened this Sep 15, 2014
@DexterInd
Copy link
Collaborator

Reopened because I would like to know what happened: Is it still a compile issue, or is it an operational issue?

Also, just pushed another commit with @chahld 's recommendation, stripped out the duplicate.

@timesong
Copy link
Author

It's still a compile issue, As a temporary measure, I remark some code in BrickPiEV3.cpp to fixed this issue. but compiled firmware can't work up to date. and I believe this is independent of the compile issue.

@timesong
Copy link
Author

I just notice the latest fixed has removed duplicated debounce variable, so I think the compile issue is fixed.

@dmitrij9992905
Copy link

dmitrij9992905 commented Mar 17, 2019

Compile issues are still present

libraries\BrickPiEV3\BrickPiEV3.cpp.o (symbol from plugin): In function `sensor2':

(.text+0x0): multiple definition of `nxt_debounce'

libraries\BrickPiA\BrickPiA.cpp.o (symbol from plugin):(.text+0x0): first defined here

collect2.exe: error: ld returned 1 exit status

I tried to change name of nxt_debounce, comment it in BrickPi_A, but issues are still present

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

No branches or pull requests

3 participants