Subject: Re: pkgtools/x11-links permissions
To: Todd Vierling <tv@duh.org>
From: Ian D. Leroux <ian_leroux@fastmail.fm>
List: tech-pkg
Date: 02/01/2005 07:44:33
On Mon, Jan 31, 2005 at 11:33:48PM -0500, Todd Vierling wrote:
> There's a variable for you to use, actually:  "-m ${PKGDIRMODE}"  (this
> reflects the fact that on at least one platform, the desired default is 775,
> not 755).

Thank you for the information. I've made the change and tested again,
make install, make clean, cd ../../wm/ratpoison && make.  Seems to work.
Any other comments or suggestions or should I file a PR?

Ian Leroux


--- Makefile.orig	2005-01-31 20:16:33.000000000 -0500
+++ Makefile.fixed	2005-02-01 07:34:05.000000000 -0500
@@ -52,12 +52,13 @@
 CREATE_X11LINK?=	${LN} -fs $$src $$dest
 
 do-build:
-	${MKDIR} ${X11_LINKS_BUILD_DIR}
+	${MKDIR} -m ${PKGDIRMODE} ${X11_LINKS_BUILD_DIR}
 	${RM} -f ${PLIST_SRC.dirs}
 	${FILES_LIST_CMD} | ${SED} -e "s,/[^/]*$$,," | ${SORT} -u |	\
 	while read dir; do						\
 		if [ -d ${X11BASE}/$$dir ]; then			\
-			${MKDIR} ${X11_LINKS_BUILD_DIR}/$$dir;		\
+			${MKDIR} -m ${PKGDIRMODE}			\
+				${X11_LINKS_BUILD_DIR}/$$dir;		\
 			${ECHO} "@dirrm ${X11_LINKS_SUBDIR}/$$dir"	\
 				>> ${PLIST_SRC.dirs};			\
 		fi;							\