Description of problem: Since the arm platform in general is bi-endian, pjproject config.h doesn't set either the PJ_IS_LITTLE_ENDIAN or PJ_IS_BIG_ENDIAN flags which means that at app compile time, you have to set them. Since Fedora is released only for little endian on arm, a patch to config.h to set PJ_IS_LITTLE_ENDIAN=1 and PJ_IS_BIG_ENDIAN=0 for arm would save a little work when compiling apps against pjproject. Version-Release number of selected component (if applicable): pjprokect-2.3-2.fc20 How reproducible: Continually Steps to Reproduce: 1. Install pjproject on an arm platform 2. Compile an app against pjproject 3. Actual results: You'll see /usr/include/pj/config.h:248:6: error: #error Endianness must be declared for this processor # error Endianness must be declared for this processor Expected results: No error Additional info:
In reality the pjproject should detect this at compile time and set it as appropriate like all other projects do. ARM isn't the only architecture that is endian agnostic. MIPs, PPC are as well.
True. Since you're never going to mix endianesses between pjproject and the app AC_C_BIGENDIAN in the pjproject configure is probably a better way to go.
(In reply to George Joseph from comment #2) > True. Since you're never going to mix endianesses between pjproject and the > app AC_C_BIGENDIAN in the pjproject configure is probably a better way to > go. It's not really possibly to mix endianess at the library/app level. You have to have one or the other so it should just be detected and dealt with, it's not something that should ever need to be in config files. Having dealt with ARM and PPC (both BE and LE in Fedora) there's no other package I'm aware of that doesn't just deal with this automatically at compile time
(In reply to Peter Robinson from comment #3) > (In reply to George Joseph from comment #2) > > True. Since you're never going to mix endianesses between pjproject and the > > app AC_C_BIGENDIAN in the pjproject configure is probably a better way to > > go. > > It's not really possibly to mix endianess at the library/app level. You have > to have one or the other so it should just be detected and dealt with, it's > not something that should ever need to be in config files. Having dealt with > ARM and PPC (both BE and LE in Fedora) there's no other package I'm aware of > that doesn't just deal with this automatically at compile time I *think* that's just what I said but when you say 'automatically' what do you mean? pjproject itself I believe needs to be endian-aware in some of the codecs.
Jared... The test build didn't work. You can tell by looking in /usr/include/pj/config.h. It still has... #elif defined (PJ_M_ARMV4) || defined(ARM) || defined(_ARM_) || \ defined(ARMV4) || defined(__arm__) /* * ARM, bi-endian, so raise error if endianness is not configured */ # undef PJ_M_ARMV4 # define PJ_M_ARMV4 1 # define PJ_M_NAME "armv4" # define PJ_HAS_PENTIUM 0 # if !PJ_IS_LITTLE_ENDIAN && !PJ_IS_BIG_ENDIAN # error Endianness must be declared for this processor # endif It should have the definitions instead of the test and error. # define PJ_IS_LITTLE_ENDIAN 1 # define PJ_IS_BIG_ENDIAN 0
Created attachment 948341 [details] Patch for arm endianess Here's a patch that modified config.h to automatically set the PJ flags for ARM if __BYTE_ORDER__ is set. If __BYTE_ORDER__ isn't set, the old behavior applies which gives you the error if you haven't explicitly set it. Yo you can probably push this upstream.
pjproject-2.3-5.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/pjproject-2.3-5.fc21
pjproject-2.3-5.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/pjproject-2.3-5.fc20
pjproject-2.3-5.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/pjproject-2.3-5.fc19
pjproject-2.3-5.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/pjproject-2.3-5.el6
Package pjproject-2.3-5.el6: * should fix your issue, * was pushed to the Fedora EPEL 6 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=epel-testing pjproject-2.3-5.el6' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-EPEL-2014-3991/pjproject-2.3-5.el6 then log in and leave karma (feedback).
pjproject-2.3-6.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/pjproject-2.3-6.fc21
pjproject-2.3-6.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/pjproject-2.3-6.fc20
pjproject-2.3-6.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report.
pjproject-2.3-6.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report.
pjproject-2.3-5.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report.