search

pantoniou / libfyaml

Fully feature complete YAML parser and emitter, supporting the latest YAML spec and passing the full YAML testsuite.
MIT License
243 stars 74 forks source link

Statically-link `fy-tool` #105

Closed rahul-theorem closed 3 months ago

rahul-theorem commented 7 months ago

Issue

It'd be desirable to be able to build fy-tool as a standalone binary to consume in our build systems to format YAML files, vs the current state where other libs need to be installed in order for the tool to function

Summary

Statically-link fy-tool

rahul-theorem commented 7 months ago

@pantoniou this was a small change so figured it might be easier to just propose the diff & ping you instead of opening an issue, but curious if you'd be willing to accept this change? It's a lot more convenient to pull fy-tool into our build if it's available as a standalone binary

pantoniou commented 3 months ago

It was always possible to build it statically via CFLAGS at the time of configure. Unfortunately it is not obvious so I added a working --enable-static option for configure with 53b6e465c83f14154c82be812c19b4b55ed57722