Error compiling EmulationStation

mitu

cmake 2.8.x is a 9 years old release, I guess since then it learned to support UTF8 for the generators/modules. Can't reproduce with a recent cmake version.

sushisan

This is exact but CMakeLists.txt say cmake_minimum_required(VERSION 2.8)
New versions of cmake don't have this error.
Converting FindRapidJSON.cmake to ascii like FindFreeImage.cmake FindlibCEC.cmake FindOpenGLES.cmake FindSDL2.cmake FindVLC.cmake MacroEnsureVersion.cmake can maintain compatibility without problem

mitu

Interesting. The problem seems to be from the UTF-8 BOM header present in the file.
Seems a bug has been logged here for this error, being fixed in the 3.0 release of cmake.

sushisan

Exact.
I see two solutions: update cmake_minimum_required or convert FindRapidJSON.cmake to ascii
The second solution can maintain compatibility and is very simple

sushisan

I have compiled EmulationStation on Centos 7 without problem with the second solution

mitu

@sushisan said in Error compiling EmulationStation:

I see two solutions: update cmake_minimum_required or convert FindRapidJSON.cmake to ascii

Probably both, at some point.

I have compiled EmulationStation on Centos 7 without problem with the second solution

Well, Centos7 is hardly a gaming platform :), probably that's why nobody complained so far. Thanks for reporting.

VictimRLSH

@mitu I used to run CentOS around 5.0 a long time ago. VERY maintenance heavy, switched to Knoppix. Maintenance headaches vanished. :)

sushisan

I want to use a long term supported distro and don't like Debian and derivates.
Centos 7 is still very stable but I will port to Centos 8

https://github.com/hiperiondev/RetroArcade-CentOS

mitu

@sushisan How about ArchLinux ? Never tried it, but I heard good things about it.

sushisan

I have not experience in Arch but I work with RedHat distros from '98.

BuZz

@sushisan without wanting to sound harsh, we are focussed on RetroPie. We can't be expected to support a distro that old (even if it's still supported by the vendor), with a cmake from 2015.

If you want to submit a fix that won't affect anything else feel free but for me it's a wontfix.

sushisan

@BuZz Are you correct!
I this case there are two errors: ascii encode and cmake_minimum_required
If you maintain minimum version of cmake in 2.8 then you have an error in encoding.
If you require newer version, and correct cmake_minimum_required, then there are no errors.

But I agree with you if not support oldedr versions. Is reasonable

BuZz

@sushisan I do agree with that. The code that fails was added later and I guess wasn't tested on older cmake. We can bump the cmake version but I don't mind if someone wants to make it work on older cmake too so long as it's easily maintainable. Eg Not having to adjusting the JSON cmake files if we upgrade that component to keep compatibility. Easier just to require a newer cmake.