Bug #2720
closed
arm build comple crash :arm/convert code
Added by o8ko8k over 6 years ago.
Updated over 6 years ago.
Description
when running ./configure --with-neon-vfpv4
There appears to be a leftover endif statement from the move of common convert code.
make3: Entering directory '/home/username/source_code/osmo-trx/Transceiver52M/arm'
CC convert.lo
CPPAS convert_neon.lo
CC convolve.lo
convert.c:57:2: error: #endif without #if
#endif
Also, possibly a left over reference causing compile crash for arm builds
/home/username/source_code/osmo-trx/Transceiver52M/osmo-trx.cpp:453: undefined reference to `convert_init'
Files
General Note: We should add a compile testing job for osmo-trx on arm in our jenkins.
I've added #2721 for that purpose.
On Thu, Dec 07, 2017 at 04:43:14AM +0000, o8ko8k [REDMINE] wrote:
There appears to be a leftover endif statement from the move of common convert code.
As you appear to have a build (and test?) environment for ARM and have tracked it down
already to that point: Would you mind sending us a patch? Thanks.
- Related to Bug #2721: OsmoTRX build verification job for arm added
I'm sorry i don't know how to make a patch. I really appreciate all the work you guys put in. I will learn about patches and help where i can.. in the mean time:
./Transceiver52M/arm/convert.c
line 57
/* #endif I commented this line out */
./Transeiver52M/osmo-trx.cpp
line 453
/* convert_init() i commented this line out too */
- Status changed from New to Feedback
- % Done changed from 0 to 100
Fixed once these are merged:
remote: https://gerrit.osmocom.org/5760 arm/convert.c: Fix compilation error
remote: https://gerrit.osmocom.org/5761 arm/convert.c: Add missing convert_init implementation
On Thu, Jan 11, 2018 at 06:19:51PM +0000, pespin [REDMINE] wrote:
Fixed once these are merged:
> remote: https://gerrit.osmocom.org/5760 arm/convert.c: Fix compilation error
> remote: https://gerrit.osmocom.org/5761 arm/convert.c: Add missing convert_init implementation
>
both merged now.
- Status changed from Feedback to Resolved
Also available in: Atom
PDF
