`site_cfg` is not defined

nanomsg / nnpy

cffi-based Python bindings for nanomsg

MIT License

117 stars 39 forks source link

`site_cfg` is not defined #38

Closed pschuprikov closed 6 years ago

pschuprikov commented 8 years ago

When I tried to use CPATH environment variable to pass include dierctory, I got this:

  File "generate.py", line 99, in create_module
    site_cfg['include_dirs'] += [os.path.join(p, 'nanomsg')
NameError: global name 'site_cfg' is not defined

The reason is this line. It should probably be set_source_args instead of site_cfg

djc commented 8 years ago

Ah, nice find. Would you be able to contribute a pull request for this?

djc commented 6 years ago

Fixed with #39.