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.

Sign up for GitHub

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

Jump to bottom

Change MCA component build style default to static #8132

Merged
merged 3 commits into from
Mar 25, 2021
Merged

Change MCA component build style default to static #8132

merged 3 commits into from
Mar 25, 2021

Commits on Mar 16, 2021

  1. build: clean up opal_mca / dlopen argument handling 

    opal_mca.m4 was setting enable_dlopen after it was processed, and
the dlopen argument checking was overriding configure arguments
that belong to opal_mca.  Clean up the late change in enable_dlopen
and handle the --disable-dlopen case in opal_mca instead of
overriding user options.

Signed-off-by: Brian Barrett <[email protected]>
    @bwbarrett
    bwbarrett committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    ca72bd4 View commit details
    Browse the repository at this point in the history

  2. build: Remove useless code in opal_mca.m4 

    Remove an environment variable that probably did something at one
point, but is dead code now.  Git history is hard to follow due
to some reverts in the SVN days.

Signed-off-by: Brian Barrett <[email protected]>
    @bwbarrett
    bwbarrett committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    76328f0 View commit details
    Browse the repository at this point in the history

  3. build: Default to building MCA components in the library 

    Default to building MCA components into the library, rather than
as dso objects, since most users are not taking advantage of
per-library packaging, and this results in less stat/open/etc. calls
during application startup.

Clean up the static/dso selection logic for componets at the same time.
Before, it was relatively hard to describe what would happen if
there was a mix of --enable-mca-dso and --enable-mca-static arguments.
Now, the code will first look for a component-specific argument, then
a framework-level argument, then a global argument.  If there is a tie,
static is preferred.

Update the NEWS and README.md to document the changes.  Jeff Squyres
wrote the original update to the README file, which was adopted to
the new README.md format.

Signed-off-by: Brian Barrett <[email protected]>
    @bwbarrett
    bwbarrett committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    930260c View commit details
    Browse the repository at this point in the history