MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015564CMakeCMakepublic2015-05-12 11:152015-11-02 09:15
ReporterMathieu Malaterre 
Assigned ToBrad King 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake 3.3 
Target VersionCMake 3.3Fixed in VersionCMake 3.3 
Summary0015564: MIPS is a CPU (fix mipsel support)
DescriptionThe code assume that __MIPS__ imply big endian, which is incorrect. See attached patch for details.

Orig ref: https://bugs.debian.org/785126 [^]
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch mipsel.patch (527) 2015-05-12 11:15
https://public.kitware.com/Bug/file/5454/mipsel.patch
Issue History
Date ModifiedUsernameFieldChange
2015-05-12 11:15Mathieu MalaterreNew Issue
2015-05-12 11:15Mathieu MalaterreFile Added: mipsel.patch
2015-05-12 11:33Brad KingNote Added: 0038741
2015-05-12 11:50Brad KingNote Added: 0038742
2015-05-13 08:34Brad KingNote Added: 0038744
2015-05-13 08:34Brad KingAssigned To => Brad King
2015-05-13 08:34Brad KingStatusnew => resolved
2015-05-13 08:34Brad KingResolutionopen => fixed
2015-05-13 08:34Brad KingFixed in Version => CMake 3.3
2015-05-13 08:34Brad KingTarget Version => CMake 3.3
2015-11-02 09:15Robert MaynardNote Added: 0039809
2015-11-02 09:15Robert MaynardStatusresolved => closed

Notes
(0038741)
Brad King   
2015-05-12 11:33   
For reference, the same change was already made in KWIML:

 KWIML: Teach ABI.h that MIPS is biendian
 http://www.cmake.org/gitweb?p=cmake.git;a=commitdiff;h=aabf65a0 [^]

but not KWSys.
(0038742)
Brad King   
2015-05-12 11:50   
I've ported the change to KWSys upstream:

 http://review.source.kitware.com/19771 [^]
(0038744)
Brad King   
2015-05-13 08:34   
I've updated KWSys in CMake to integrate this change:

 KWSys 2015-05-12 (b1d560a0)
 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=3b815ed2 [^]
(0039809)
Robert Maynard   
2015-11-02 09:15   
Closing resolved issues that have not been updated in more than 4 months.