Closed q66 closed 3 years ago
compiler flags like this shoud never be hardcoded. they should be always up to end user to choose.
Sorry. I took out the import & call. The module file just being there should be innocuous. I can stamp a new version if you find it helpful.
would be helpful for everyone else compiling it, at least (not me, since i already worked around it)
Specifically this commit: 1f25ac7e19115a13a7fb5dcaca20b268629316c7
This:
1) breaks things for distributions, since
-march=native
will emit code for host CPU 2) breaks several platforms, since-march
is only provided on some (e.g. the powerpc family uses-mcpu
instead) 3) i don't think it behaves as intended; notably, this introduces-march=native
into build of every single package that depends oncligen
in any way; I ran across this when compilingchoosenim
onppc64le
, which then fails innimterop
:I then went to look for where that came from, and found this:
I forked this for our distribution for now, and pointed everything to https://github.com/q66/cligen/commit/deee68fdc8ec5f34de7054a029739d3933fc551b, but that's more of a temporary thing.