Re: CVS commit: wip/smc

["Jeremy C. Reed" <reed%reedmedia.net@localhost>]

Thanks for the info on smc.

The package has:

/usr/bin/install -c -o root -g wheel -m 444

${INSTALL_DATA} -m0555 ${WRKDIR}/smc-exec.sh ${PREFIX}/bin/smc

But INSTALL_DATA already has -m ${SHAREMODE}

If you want executable, use INSTALL_SCRIPT.

(And if ever needed, don't hard-code 0555, just use ${BINMODE}.)

I am not ready to try the package yet, due to all the  -m0666 and -m0777 
in the installation.

Maybe this should use the "games" group and run setgid if needed.

Or force it to do all in the user's home directory.

(Maybe this had been discussed already on some smc forum or list?)

 Jeremy C. Reed

                         BSD News, BSD tutorials, BSD links
                         http://www.bsdnewsletter.com/


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
pkgsrc-wip-discuss mailing list
pkgsrc-wip-discuss%lists.sourceforge.net@localhost
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-discuss