On Tue, Apr 11, 2017 at 11:03:33AM +0100, Stuart Henderson wrote:
> > > > On Mon, Apr 10, 2017 at 01:35:57PM +0200, Mark Kettenis wrote:
> > > > > Looking into arm_neon.h. This header file is supposed to be generated
> > > > > using the clang_tablegen tool, which is why we missed it.
>
> $ tar tvzf comp61.tgz |grep ^d.*lib.clang
> drwxrwx--- 2 build wheel 0 Apr 10 13:01 ./usr/lib/clang
> drwxrwx--- 2 build wheel 0 Apr 10 13:01 ./usr/lib/clang/4.0.0
> drwxr-xr-x 2 root bin 0 Apr 10 13:01
> ./usr/lib/clang/4.0.0/include
>
> Is this the way to fix it, or do we need something different as it's only
> on some arch?
Isn't this the usual annoyance that install -d creates intermediate
directories without explicitly setting owner and permissions?
In gnu/usr.bin/cc/cc/Makefile natano worked around this problem by
incrementally creating the path with an ugly .for loop like this:
(untested)
Index: gnu/usr.bin/clang/include/clang/intrin/Makefile
===================================================================
RCS file: /var/cvs/src/gnu/usr.bin/clang/include/clang/intrin/Makefile,v
retrieving revision 1.7
diff -u -p -r1.7 Makefile
--- gnu/usr.bin/clang/include/clang/intrin/Makefile 10 Apr 2017 13:36:04
-0000 1.7
+++ gnu/usr.bin/clang/include/clang/intrin/Makefile 11 Apr 2017 10:32:42
-0000
@@ -86,8 +86,10 @@ clean cleandir:
rm -f ${GEN}
install includes: ${HEADERS}
+.for dir in /usr/lib/clang /usr/lib/clang/${LLVM_V}
/usr/lib/clang/${LLVM_V}/include
${INSTALL} -d -o ${SHAREOWN} -g ${SHAREGRP} -m ${DIRMODE} \
- ${DESTDIR}${CLANG_INTR_INCDIR}
+ ${DESTDIR}${dir}
+.endfor
${INSTALL} ${INSTALL_COPY} -m 444 ${.ALLSRC} \
${DESTDIR}${CLANG_INTR_INCDIR}
