pkg/43687: make package in devel/ruby-readline needs root privilege

[ryo_on%yk.rim.or.jp@localhost]

>Number:         43687
>Category:       pkg
>Synopsis:       make package in devel/ruby-readline needs root privilege
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Jul 30 20:50:00 +0000 2010
>Originator:     Ryo ONODERA
>Release:        NetBSD 5.99.38
>Organization:
        
>Environment:
        
        
System: NetBSD hydrogen.elements.tetera.org 5.99.38 NetBSD 5.99.38 (LEAFGIRL0) 
#1: Fri Jul 30 15:22:41 JST 2010 
root%hydrogen.elements.tetera.org@localhost:/usr/obj/sys/arch/i386/compile/LEAFGIRL0
 i386
Architecture: i386
Machine: i386
>Description:
make package in devel/ruby-readline needs root privilege.
In ext/readline/Makefile, install has "-o root -g wheel" options.
        
>How-To-Repeat:
% cd /usr/pkgsrc/devel/ruby-readline
% make package
        
>Fix:
        
The following patch fix this problem.
But I does not know build process of ruby module, there may be better solution.

diff -x CVS -upNr /usr/pkgsrc/devel/ruby-readline/Makefile 
ruby-readline/Makefile
--- /usr/pkgsrc/devel/ruby-readline/Makefile    2009-09-13 10:51:05.000000000 
+0900
+++ ruby-readline/Makefile      2010-07-31 05:32:56.000000000 +0900
@@ -22,6 +22,14 @@ WRKSRC=                      ${RUBY_WRKSRC}/ext/readline
 RUBY_DOCS=             README README.ja
 INSTALLATION_DIRS=     ${RUBY_DOC}/readline
 
+SUBST_CLASSES+=                        fix-install
+SUBST_STAGE.fix-install=       pre-build
+SUBST_MESSAGE.fix-install=     Fix install command options 
+SUBST_FILES.fix-install=       Makefile
+SUBST_SED.fix-install=         -e 's,^INSTALL =.*,INSTALL=${INSTALL},'
+SUBST_SED.fix-install+=                -e 's,^INSTALL_PROG =.*,INSTALL_PROG = 
${INSTALL_PROGRAM},'
+SUBST_SED.fix-install+=                -e 's,^INSTALL_DATA =.*,INSTALL_DATA = 
${INSTALL_DATA},'
+
 post-install:
 .for f in ${RUBY_DOCS}
        ${INSTALL_DATA} ${WRKSRC}/${f} ${DESTDIR}${PREFIX}/${RUBY_DOC}/readline


>Unformatted: