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

fix(build): add missing AC_PROG_RANLIB #622

Merged
1 commit merged into from
Sep 26, 2024
Merged

fix(build): add missing AC_PROG_RANLIB #622

1 commit merged into from
Sep 26, 2024

Conversation

ghost
Copy link

@ghost ghost commented Sep 26, 2024

Automake manual:

> This is required if any libraries are built in the package. See
> Particular Program Checks in The Autoconf Manual.

add this for correctness.

Ranlib is already being used, but this just adds the automake hook to ensure
that it can be resolved..

By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license.

Automake manual:

> This is required if any libraries are built in the package. See
> Particular Program Checks in The Autoconf Manual.

add this for correctness.

Signed-off-by: Nicholas Sielicki <nslick@amazon.com>
@ghost ghost requested a review from bwbarrett as a code owner September 26, 2024 02:52
@ghost ghost merged commit b525063 into aws:master Sep 26, 2024
31 checks passed
@ghost ghost deleted the ranlib branch September 26, 2024 20:47
aws-ofiwg-bot pushed a commit to aws-ofiwg-bot/aws-ofi-nccl that referenced this pull request Oct 4, 2024
Automake manual:

> This is required if any libraries are built in the package. See
> Particular Program Checks in The Autoconf Manual.

add this for correctness.

Signed-off-by: Nicholas Sielicki <nslick@amazon.com>
This pull request was closed.
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.

1 participant