Skip to content

Conversation

@577fkj
Copy link
Contributor

@577fkj 577fkj commented May 14, 2025

@ubfx
Copy link
Owner

ubfx commented May 24, 2025

Thank you for the PR. I'd like to merge the bugfixes and namespace support but could you move the changes related to how BinDiff is called to another PR? We've had so many problems with this across different platforms that I'd like to be able to test that seperately. AFAIK, it should be working with the current solution (flushing stderr) on all platforms, or did you experience something different?

@577fkj
Copy link
Contributor Author

577fkj commented May 26, 2025

Thank you for the PR. I'd like to merge the bugfixes and namespace support but could you move the changes related to how BinDiff is called to another PR? We've had so many problems with this across different platforms that I'd like to be able to test that seperately. AFAIK, it should be working with the current solution (flushing stderr) on all platforms, or did you experience something different?

#39
#40

@577fkj 577fkj closed this May 26, 2025
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

Successfully merging this pull request may close these issues.

2 participants