search

SSBMTonberry / tileson

A modern and helpful cross-platform json-parser for C++, used for parsing Tiled maps.
BSD 2-Clause "Simplified" License
189 stars 29 forks source link

Add script to rebuild amalgamated headers on OSX #82

Closed dmlary closed 1 year ago

dmlary commented 1 year ago

Use https://github.com/Viatorus/quom (python utility) to construct the amalgamated header on platforms the original amalgamate binary won't run.

fixes #80

SSBMTonberry commented 1 year ago

I'll just do a quick test of this after work, but looks good :smile: Seems like the CI is not working properly right now, so don't mind those errors. I'll have to do a fix there.

SSBMTonberry commented 1 year ago

@dmlary : I've just tested this script on my Linux setup, just because there is no reason why it shouldn't work, and I think your solution is just great! Well done :smile:

I started off with not having quom installed, and I got the very descriptive message:

This script depends on the quom utility to combine the header files:
    https://github.com/Viatorus/quom

Quom is a python 3.6 (or above) utility that can be installed via pip:
    pip install quom

Or on OSX with python3 from homebrew installed:
    pip3 install quom

Ensure that the quom utility is in your PATH when running this script:
    PATH="~/Library/Python/3.10/bin:$PATH" ./amalgamate_osx.sh

So I ran pip3 install quom as the instruction says, which installed quom, and then it worked just fine.

Approved! :partying_face: