search

jean-airoldie / zeromq-src-rs

Source code and logic to build ZeroMQ from source
MIT License
11 stars 14 forks source link

DEP_ZMQ_INCLUDE output incorrect #25

Closed iamthad closed 1 year ago

iamthad commented 1 year ago

https://github.com/jean-airoldie/zeromq-src-rs/blob/e7dd3b2103c3e5d95376cbe9e11e0be39822367a/src/lib.rs#L422-L426

The header files are placed in out/source/include, but include_dir is defined as out/include (missing source).

jean-airoldie commented 1 year ago

Well this is embarrassing. I guess that's what i get for not testing the ENV variables.

I'll cook up a PR and then do a release.

jean-airoldie commented 1 year ago

Alright I pushed a commit. Should do the trick.

@iamthad Please tell me if this works properly, then I'll do a release.

jean-airoldie commented 1 year ago

Alright, I'm gonna assume this fixes your issue and close.

iamthad commented 1 year ago

I just tested your latest release and can confirm it fixes the issue. Thank you for the quick fix!