libxo
libxo copied to clipboard
Add Cirrus-CI config
Awesome. I’ll pull it into the next release.
Thanks, Phil
On May 12, 2019, at 3:56 AM, Li-Wen Hsu [email protected] wrote:
You can view, comment on, or merge this pull request online at:
https://github.com/Juniper/libxo/pull/66 https://github.com/Juniper/libxo/pull/66 Commit Summary
Add Cirrus-CI config File Changes
A .cirrus.yml https://github.com/Juniper/libxo/pull/66/files#diff-0 (14) Patch Links:
https://github.com/Juniper/libxo/pull/66.patch https://github.com/Juniper/libxo/pull/66.patch https://github.com/Juniper/libxo/pull/66.diff https://github.com/Juniper/libxo/pull/66.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Juniper/libxo/pull/66, or mute the thread https://github.com/notifications/unsubscribe-auth/AAROIOGNMCDQTML4G2CFZCLPU7EUJANCNFSM4HMJZZSA.
Thanks, it would be nice if we can remove the GNU-make only syntax in warnings.mk
to drop gmake dependency.
On May 12, 2019, at 9:22 PM, Li-Wen Hsu [email protected] wrote:
Thanks, it would be nice if we can remove the GNU-make only syntax in warnings.mk to drop gmake dependency.
Unfortunately, gmake is the outcome of choosing to use autoconf, which helps make the code run in various environments. I have distinct Makefiles for building the library as part of FreeBSD’s “make world".
Thanks, Phil
Worth revisiting this now?
The Cirrus-CI config has been update but there are some error/warnings from the compiler, should we fix those first? https://github.com/lwhsu/libxo/runs/14633783929
there are some error/warnings from the compiler, should we fix those first
I'd say so. I guess we turned off the warning in FreeBSD?