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

have bin32/bin64 for 32/64 bit build #917

Closed
derekbruening opened this issue Nov 28, 2014 · 2 comments
Closed

have bin32/bin64 for 32/64 bit build #917

derekbruening opened this issue Nov 28, 2014 · 2 comments

Comments

@derekbruening
Copy link
Contributor

From zhao...@google.com on June 16, 2012 01:35:10

Now 32-bit build is put into bin and 64-bit build in bin64.
tests/CMakeLists.txt use bin as the path, which makes 64-bit build fail to find drmemory in testing.
The temp solution is to put 64-bit build into bin too, but should separate them into bin32 and bin64 for future package release.

Original issue: http://code.google.com/p/drmemory/issues/detail?id=917

@derekbruening
Copy link
Contributor Author

From bruen...@google.com on June 17, 2012 09:59:37

isn't it fairly simple to tweak the tests cmakelists? I think a recent
diff of yours does that.

an issue with changing bin to bin32 is that it breaks all the docs and
breaks any user who has scripts launching drmem. that's why it would probably be best
to stick with "bin" and "bin64" in perpetuity.

@derekbruening
Copy link
Contributor Author

From zhao...@google.com on June 20, 2012 09:05:21

This issue was closed by revision r904 .

Status: Fixed

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

No branches or pull requests

1 participant