I think my patch fulfills all your requirements. It has no side effect on non-musl platform.
Best Regards, Cheng On Nov 4, 2020, 4:08 PM -0800, Dmitry V. Levin <l...@altlinux.org>, wrote: > On Wed, Nov 04, 2020 at 02:57:08PM -0800, Cheng XU wrote: > > Please find the patch in the attachment. Thanks. > > > > Best Regards, > > > > Cheng > > If you're about to reintroduce the ldd check, please make sure the most > trustworthy of these checks (whatever it is) is attempted first and its > result is not discarded, please do not penalize non-musl platforms > unnecessarily. > > Thanks, > > > On Nov 4, 2020, 2:39 PM -0800, Ben Elliston <b...@air.net.au>, wrote: > > > On Wed, Nov 04, 2020 at 02:31:05PM -0800, Cheng XU wrote: > > > > > > > Thanks for the info. However, it seems that the referenced issue is > > > > only valid for the old version of Alpine. For the latest version, > > > > the `ldd --version` works just fine. As shown in the below, the old > > > > config.guess reports the correct result in a fresh alpine docker > > > > container, while the new config.guess does not. Therefore, I think > > > > we should just keep the original `ldd --version` check. There is no > > > > harm to do so. > > > > > > Please suggest a patch, then. > > > > > > Thanks, > > > Ben > > -- > ldv
