log: PlatformFolders

Also slightly reworded the example cmake usage.

Mac no core

Added notes about Mac OS X and the drop of the Core Framework. The information is still included but will be removed after 4.0.0 has been released.

Cmake touchups

Since no dependencies are needed after compile-time (CoreServices is PRIVATE) we don't need its special logic.
This installs an auto-generated platform_foldersConfig.cmake file instead.

Add "Public" as an option.

XDG_ dirs check

This prevents trouble caused by future standards that may allow other kind of environments in the user-dirs.dirs-file.

We now allow trailing whitespace. Previously trailing whitespace would make the returned value longer.

We do not allow whitespace around the "=" (and never did). "/usr/bin/xdg-user-dir" fails on it too.

The check is very basic. I had to drop regex. gcc does not support it until 4.9. Current target is 4.8.4

Known folders2

C++11 & more

Pre-v3.8 Cmake didn't have cxx_std_11.
This sets the old setting to use C++11 when on an older version.
But on v3.8 and up it will use the newer way with cxx_std_11

PROJECT_NAME doesn't really work as expected. This can cause problems when trying to use it externally.
Things like git submodule's or Cmake's ExternalProject_Add could have weird behaviour, so this gets around that.
Removed some junk, and corrected the private includes to be public, so other projects correctly get the header.

Fix find_package not being seamless

Puts Config.cmake in Windows search path.
Different OS's use different Cmake search paths when find_package is called.

Because of how Cmake looks for the Config.cmake file, it previously required passing the path in the cmake command.
By moving where it's installed, we avoid that. This allows for Cmake to find_package(platform_folders) easily.

sago::platform_folders is created as an 'IMPORTED' library to be linked with target_link_libraries()

The used settings are what were referred to in Gitter

"I usually use astyle. Typically "astyle -t -j -y -c -k1 -z2 -A2 --pad-header "."

While it's used in the build script, for some reason it was fine without it.
It's better to be correct though :)

I tried to "nest" relevant content under headings, added notes about Cmake usage, and fixed some grammatical errors.
I added a section about installation, and made the build step system-agnostic.

Also changed the README's Cmake command to not build tests.
It's a lot quicker to not build them, and I doubt everyone wants to build/run tests.

Hopefully it's all an improvement :smile:

Also reworked the old test file into the 'integration.cpp' file.

Note that this increases build times a bit since there're both more code & more units to build.
You can pass -DBUILD_TESTING=OFF when running Cmake not not build them, which saves time.

WIN32_LEAN_AND_MEAN makes <windows.h> bring in a lot less junk.
SUCCEEDED macro in winerror.h is used.
Stringapiset.h is needed for WideCharToMultiByte.

Doesn't technically matter, but I figured why not.

No point having unused if's.

NULL can technically vary on some systems, while C++11's nullptr guarntees safety.

Fix for Linux cache dir

Change XDG_CONFIG_DIR to XDG_CACHE_DIR

Appveyor fix

Also switched to using the env var for the build folder, which is the clone_folder setting.

Adds AppVeyor CI support (Windows)

Compatible touchups

Removes the define used for strtok_s and replaces uses with std::strtok
Made the Windows errmsg const, and 'p' non-const, since it is changed.

Adds Cmake build process

Nitpicking1

Added gitter

Right path to the license file

Added more visible license badge.

Added note about UTF-8

* Linux can be configured with something besides UTF-8 but then the user is asking for trouble if he goes outside UTF-7.

Added link to releases

It is not important that this library is used. It is important to use a library that has taken this into account.

Build status is now visible.