BrickPi icon indicating copy to clipboard operation
BrickPi copied to clipboard

Compile BrickPiFW_Compressed_Communication.ino get error.

Open timesong opened this issue 10 years ago • 8 comments

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 avatar Sep 05 '14 04:09 timesong

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...

chahld avatar Sep 07 '14 20:09 chahld

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

timesong avatar Sep 11 '14 02:09 timesong

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...

chahld avatar Sep 11 '14 04:09 chahld

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 avatar Sep 15 '14 13:09 DexterInd

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.

DexterInd avatar Sep 15 '14 13:09 DexterInd

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 avatar Sep 16 '14 00:09 timesong

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

timesong avatar Sep 16 '14 00:09 timesong

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

dmitrij9992905 avatar Mar 17 '19 19:03 dmitrij9992905