Project

General

Profile

Bug #2720

arm build comple crash :arm/convert code

Added by o8ko8k 8 days ago. Updated 3 days ago.

Status:
New
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
12/07/2017
Due date:
% Done:

0%

Spec Reference:

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'

osmo-trx.cpp Magnifier - missing call to x86 function convert_init() (13.5 KB) o8ko8k, 12/11/2017 04:33 PM

convert.c Magnifier - bad #endif (2.23 KB) o8ko8k, 12/11/2017 04:33 PM


Related issues

Related to OsmoTRX - Bug #2721: OsmoTRX build verification job for arm New 12/07/2017

History

#1 Updated by laforge 8 days ago

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.

#2 Updated by laforge 8 days ago

  • Related to Bug #2721: OsmoTRX build verification job for arm added

#3 Updated by o8ko8k 3 days ago

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 */

Also available in: Atom PDF