Suggestion: allow conditional distutils directives

[Antony Lee]

Something like

IF UNAME_SYSNAME == "Linux":
    #distutils: define_macros = FOOBAR=42
    pass # to avoid completely confusing the parser, this may help.
ELSE:
    #distutils: define_macros = FOOBAR=123
    pass

The goal being mostly to pass different compilation options/link options depending on the platform, without having to put these options in setup.py.

Thoughts?

Antony

[Nathaniel Smith]

I have no opinion on the proposal, but I think if you're going to add syntax for pragmas then you should spell it like PRAGMA or something, not use magic comments :-)

--

---
You received this message because you are subscribed to the Google Groups "cython-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to cython-users...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Antony Lee]

Well, #distutils: XXX and #cython: XXX already are magic comments.

[Nathaniel Smith]

Huh, didn't know that. Well, it's still true... :-)

[Robert Bradshaw]

By the time you're introducing actual logic, I think it'd be
preferable to put it in the setup.py rather than complicate these
declarations. However, you can things conditionally via aliases:

https://github.com/cython/cython/blob/master/tests/build/inline_distutils.srctree

(Not a huge fan of that name, bug "macros" was taken...)

[Antony Lee]

Thanks for the tip.  I don't think this feature is documented anywhere?  (not in the docstring of cythonize, at least.)

(My actual use-case is a change in the name of the library linked between Linux and Windows.)

Antony