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

Change default build type to 64-bits on 64-bit systems. #116

Closed
wants to merge 1 commit into from
Closed

Change default build type to 64-bits on 64-bit systems. #116

wants to merge 1 commit into from

Conversation

kpfleming
Copy link
Contributor

This patch changes the mechanism used to determine the system's processor type
from various mechanisms to Python's "platform.machine()" library function. It
uses this information to change the default for the --abi-bits parameter to
the 'configure' command.

In addition, it separates the code that determines this information from the code
that determines the compiler type and version, as those are unrelated operations.

This patch changes the mechanism used to determine the system's processor type
from various mechanisms to Python's "platform.machine()" library function. It
uses this information to change the default for the --abi-bits parameter to
the 'configure' command.

In addition, it separates the code that determines this information from the code
that determines the compiler type and version, as those are unrelated operations.
@kpfleming kpfleming mentioned this pull request Apr 2, 2014
@shayne-fletcher
Copy link

Worked nicely for my build. Thanks!

@abeels
Copy link
Contributor

abeels commented Jun 10, 2014

Internal ticket reference 53452398

@che2
Copy link
Contributor

che2 commented Jun 16, 2014

The build system has been moved to another repository: https://github.com/bloomberg/bde-tools

Please submit another pull request for that repository.

I recommend splitting the changeset into two parts, one for changing default abi bits depending on the platform, and another for updating the way the CPU name of the UPLID is acquired.

I believe the first part probably can be merged without further changes. The second part of the has some issues (on Windows), please see https://review.openbloomberg.com/D129 for more details.

@che2 che2 closed this Jun 16, 2014
abeels added a commit that referenced this pull request Feb 11, 2016
commit 1ad050d
Merge: 80cc5ab 28c27a6
Date:   Wed Jan 27 14:53:08 2016 -0500

    Merge pull request #116 from hrosen4/bdlmt_threadpool-dtor-doc_drqs-76512668

    bdlmt_threadpool: Correct/clarify destructor documentation.

commit 28c27a6
Date:   Tue Jan 19 18:01:19 2016 -0500

    Correct/clarify destructor documentation.
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.

4 participants