Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

-conditional instrument code without BISECT_ENABLE=YES #185

Closed
NathanReb opened this issue Jan 22, 2019 · 4 comments
Closed

-conditional instrument code without BISECT_ENABLE=YES #185

NathanReb opened this issue Jan 22, 2019 · 4 comments

Comments

@NathanReb
Copy link

It seems that since I upgraded to bisect_ppx v1.4.0, our ocaml code is always instrumented, even without setting BISECT_ENABLE=YES.

Downgrading it back to v1.3.4 fixes the issue.

I unfortunately didn't manage to reproduce this on a minimal example yet but I'm working on it, I'll get back to you as soon as I'm able to do so!

I think it was worth mentioning anyway in case you'd be aware of a recent change that might have somehow impacted -conditional behavior.

@NathanReb
Copy link
Author

NathanReb commented Jan 22, 2019

Hmm my bad, after running a few experiments it appears that the problem persists with v1.3.4.

I still can't reproduce it on a small project so it's a bit of a nightmare to debug.

@NathanReb
Copy link
Author

Finally found it. The issue arises when using bisect_ppx -conditional within a staged_pps stanza.

That happened because I was using both ppx_import and bisect_ppx in a library and I don't know of any way around that yet.

I'm not sure whether this requires fixing in dune or bisect_ppx but my guess would be the former. I'll report it there as well.

@NathanReb
Copy link
Author

It was indeed a bug in dune, sorry for the inconvenience!

@aantron
Copy link
Owner

aantron commented Jan 25, 2019

Thanks for looking into it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants