aws / aws-ofi-nccl

This is a plugin which lets EC2 developers use libfabric as network provider while running NCCL applications.
Apache License 2.0
147 stars 56 forks source link

fix(build): check features before mangling CFLAGS #589

Closed aws-nslick closed 1 month ago

aws-nslick commented 2 months ago

Stacked PRs:


fix(build): check features before mangling CFLAGS

Yet another autotools fix: If any warning/devel CFLAGS would invoke warnings in headers used to detect dependencies, those dependencies will fail with a highly misleading/confusing error message:

configure: Found .git directory. Adding -Werror to CFLAGS. checking if running on EC2 instance... yes checking if want AWS platform optimizations... yes checking for Libfabric 1.18.0 or later... no configure: error: On AWS platforms, Libfabric 1.18.0 or later is required

ie: it is not that Libfabric 1.18 was not found, it was that its headers produced warnings. Fix this by resolving all dependencies before modifying CFLAGS.

aws-nslick commented 1 month ago

bot:aws:retest