On Wed, 2013-10-23 20:21:50 -0400, David Edelsohn <dje....@gmail.com> wrote: > On Wed, Oct 23, 2013 at 6:21 PM, Jan-Benedict Glaw <jbg...@lug-owl.de> wrote: > > When building on gcc111, I tried to use different compilers for my > > build robot. This machine has IBM's XL C compiler installed, but not > > the C++ compiler. So I used CC=xlC, g++ was auto-detected for CXX. > > This led to depmode=aix, but g++ was actually used later on for > > calculating dependencies, totally fucking up the build directory > > (since it's output was wrongly parsed using aix-style.) > > Why do you say that the C++ compiler is not installed on gcc111? That > statement is incorrect. The AIX XL C++ compiler is xlC, which you > mention. > > Updating the dependencies to C++ makes sense.
Ah! I thought this would be `xlc++', which is not in $PATH, unlike `xlC', which is. Now I see that xlC is symlinked from /usr/vacpp/bin/xlC, where the `xlc++' also exists (as a symlink to the same file.) That's great, thanks for the notice! So I'll change one instance to use xlC/xlc++ throughout the build. However I still think that the patch itself is correct and thus would like to discuss it to gain final approval. Thanks, JBG -- Jan-Benedict Glaw jbg...@lug-owl.de +49-172-7608481 Signature of: The real problem with C++ for kernel modules is: the second : the language just sucks. -- Linus Torvalds
signature.asc
Description: Digital signature