On Sun, Oct 12, 2014 at 01:08:37PM -0400, Stuart Cassoff wrote: > On 09/26/14 10:46, Stuart Cassoff wrote: > > On 09/16/14 21:48, Stuart Cassoff wrote: > >> On 07/11/14 06:22, Stuart Henderson wrote: > >>> On 2014/07/10 23:44, Stuart Cassoff wrote: > >>>> eh? meh? > >>> > >>> You need a subst variable with the actual path not just the version > >>> number, otherwise you're still going to be hand modifying PLISTs all > >>> over the place. > >>> > >>> So something like: > >>> > >>> MODTCL_MINVERSION?= 8.5 > >>> MODTCL_MODPATH= lib/tcl/${MODTCL_MINVERSION} > >>> SUBST_VARS+= MODTCL_MODPATH MODTCL_VERSION MODTCL_BIN > >>> > >>> It would help clarify things to show a diff with a port that has > >>> been modified to use this method, too. > >>> > >> > >> Here are diffs for everything concerned. > >> Added missing modtcl vars and did a bit of rearranging in port-modules.5. > >> Described MODTCL_MINVERSION and MODTCL_MODPATH. > >> Removed the rest package from tcllib; it needs tDOM which we don't > >> currently have. > >> Saves installing 51 directories and 51 pkgIndex.tcl files. > >> > > > > > Another way to do this would be to change the Tcl module > > dirs to be like modules/85 instead of modules/8.5, then > > the MODTCL_MODPATH stuff wouldn't be needed. > > > > Here are the diffs for doint it this way. Simpler. > Also simplified the existing layout Tcl itself: > removes a couple of dirs, shortens module path. > Also a fix for TkCon.
Can you remind me why you hardcode 85 for tcllib/tklib/tkcon ? Other than that, you know better than anyone how this stuff should be laid, so just move ahead, dont wait for months for someone to okay that.. Landry