Bug 1125258 - union _LITTLE_ENDIAN not correct
Summary: union _LITTLE_ENDIAN not correct
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: vnc-reflector
Version: 21
Hardware: ppc64le
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Petr Pisar
QA Contact: Fedora Extras Quality Assurance
URL: https://sourceforge.net/p/vnc-reflect...
Whiteboard:
Depends On:
Blocks: F-ExcludeArch-ppc64le, PPC64LETracker
TreeView+ depends on / blocked
 
Reported: 2014-07-31 12:48 UTC by Menanteau Guy
Modified: 2014-08-01 13:58 UTC (History)
1 user (show)

Fixed In Version: vnc-reflector-1.2.4-17.fc22
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-08-01 13:58:37 UTC


Attachments (Terms of Use)
patch to fix union _LITTLE_ENDIAN (181 bytes, patch)
2014-07-31 12:48 UTC, Menanteau Guy
no flags Details | Diff

Description Menanteau Guy 2014-07-31 12:48:02 UTC
Created attachment 922907 [details]
patch to fix union _LITTLE_ENDIAN

vnc-reflector build failed because of invalid declaration:
union _LITTLE_ENDIAN { ... } ...;

http://ppc.koji.fedoraproject.org/kojifiles/work/tasks/4455/1944455/build.log

vnc-reflector-1.2.4-16
gcc-4.9.0-14

Please consider this patch.

Comment 1 Petr Pisar 2014-08-01 12:24:06 UTC
Is that really invalid syntax? My C11 draft does not think so. It looks like the symbol _LITTLE_ENDIAN is defined on your platform to an integer literal. However GCC is not the source <https://gcc.gnu.org/onlinedocs/cpp/Common-Predefined-Macros.html>. Isn't it bug in some of the included header file?

I understand the identifier starting with underscore is really unfortunate and I can remove it as it's nowhere used. I just think that the real problem is somewhere else.

Comment 2 Petr Pisar 2014-08-01 13:58:37 UTC
Fixed as vnc-reflector-1.2.4-17.fc21 in F21.


Note You need to log in before you can comment on or make changes to this bug.