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

v2.0.2: build: Custom libmpi(_FOO) name option in configure #2138

Closed
wants to merge 1 commit into from

Conversation

jjhursey
Copy link
Member

  • Add a configure time option to rename libmpi(_FOO).*
    • --with-libmpi-name=STRING
  • This commit only impacts the installed libraries.
    Internal, temporary libraries have not been renamed to limit the
    scope of the patch to only what is needed.

For example:

shell$ ./configure --with-libmpi-name=wookie
...
shell$ find . -name "libmpi*"
shell$ find . -name "libwookie*"
./lib/libwookie.so.0.0.0
./lib/libwookie.so.0
./lib/libwookie.so
./lib/libwookie.la
./lib/libwookie_mpifh.so.0.0.0
./lib/libwookie_mpifh.so.0
./lib/libwookie_mpifh.so
./lib/libwookie_mpifh.la
./lib/libwookie_usempi.so.0.0.0
./lib/libwookie_usempi.so.0
./lib/libwookie_usempi.so
./lib/libwookie_usempi.la
shell$

(cherry picked from commit f6f24a4)
Signed-off-by: Joshua Hursey [email protected]

@jjhursey jjhursey added this to the v2.0.2 milestone Sep 30, 2016
@jjhursey
Copy link
Member Author

Refs PR #2059

 * Add a configure time option to rename libmpi(_FOO).*
   - `--with-libmpi-name=STRING`
 * This commit only impacts the installed libraries.
   Internal, temporary libraries have not been renamed to limit the
   scope of the patch to only what is needed.

For example:
```shell
shell$ ./configure --with-libmpi-name=wookie
...
shell$ find . -name "libmpi*"
shell$ find . -name "libwookie*"
./lib/libwookie.so.0.0.0
./lib/libwookie.so.0
./lib/libwookie.so
./lib/libwookie.la
./lib/libwookie_mpifh.so.0.0.0
./lib/libwookie_mpifh.so.0
./lib/libwookie_mpifh.so
./lib/libwookie_mpifh.la
./lib/libwookie_usempi.so.0.0.0
./lib/libwookie_usempi.so.0
./lib/libwookie_usempi.so
./lib/libwookie_usempi.la
shell$
```

(cherry picked from commit f6f24a4)
Signed-off-by: Joshua Hursey <[email protected]>
@jjhursey jjhursey force-pushed the cp-v2.0.x/custom-libmpi branch from 9f0bbd8 to 9f5a49c Compare September 30, 2016 04:01
@jsquyres jsquyres changed the title build: Custom libmpi(_FOO) name option in configure v2.0.2: build: Custom libmpi(_FOO) name option in configure Sep 30, 2016
@jsquyres
Copy link
Member

Per discussion with @jjhursey on the phone, this is not actually a bug fix. The proper place for it is v2.1.0, which is already PR'ed (#2139).

@jsquyres jsquyres closed this Sep 30, 2016
@jjhursey jjhursey deleted the cp-v2.0.x/custom-libmpi branch September 30, 2016 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants