RE: [PATCH] kbuild: Support clang-$ver builds

From: David Laight
Date: Fri Oct 29 2021 - 04:56:38 EST


From: Nick Desaulniers
> Sent: 29 October 2021 00:28
...
> > > 2. Update the documentation to describe using the
> > >
> > > $ PATH=/usr/lib/llvm-#/bin:$PATH make LLVM=1 ...
> > >
> > > trick. This has been the preferred method for using different
> > > versions of LLVM but it has never been documented anywhere. This
> > > would allow us to keep the current build infrastructure while giving
> > > people clear instructions for how to handle different versions of
> > > clang. As Peter has noted, this would require people who are not
> > > familiar with building with LLVM to be constantly looking at the
> > > documentation to remember the command to invoke, whereas with
> > > LLVM=-#, it is easy to remember.
> >
> > Right, this is no more than a crude hack and is super unfriendly. It
> > should never have been accepted.
>
> Lots of tools modify PATH in your `.bashrc` or `.zshrc` etc. I don't
> see how that's any different, other than one off commands in which you
> actually intend to use a specific version.

Nothing should be modifying the default $PATH, either by hacking
my .bashrc or any system default files that get run first.

It is as horrid and broken as setting LD_LIBRARY_PATH.
Even ldconfig is pretty much a broken idea.

David

-
Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK
Registration No: 1397386 (Wales)