Revision: 41547
          http://brlcad.svn.sourceforge.net/brlcad/?rev=41547&view=rev
Author:   brlcad
Date:     2010-12-07 21:24:10 +0000 (Tue, 07 Dec 2010)

Log Message:
-----------
shouldn't directly use path separators on paths.  use file join for 
portability, passing the buck down to the callers since we need to join 
individual command names.

Modified Paths:
--------------
    brlcad/trunk/src/tclscripts/mged/anim.tcl

Modified: brlcad/trunk/src/tclscripts/mged/anim.tcl
===================================================================
--- brlcad/trunk/src/tclscripts/mged/anim.tcl   2010-12-07 21:01:10 UTC (rev 
41546)
+++ brlcad/trunk/src/tclscripts/mged/anim.tcl   2010-12-07 21:24:10 UTC (rev 
41547)
@@ -620,7 +620,9 @@
     upvar #0 [format "mged_sketch_time_%s" [vdraw read n]] tlist
     set start [lindex $tlist 0]
     set end [lindex $tlist [expr $length - 1]]
-    set fo [open "| ${mged_sketch_tab_path}tabinterp -q > $mged_sketch_temp1" 
w]
+
+    set tabinterp [file join ${mged_sketch_tab_path} tabinterp]
+    set fo [open "| $tabinterp -q > $mged_sketch_temp1" w]
     puts $fo "file $mged_sketch_temp2 0 1 2;"
     puts $fo [concat times $start $end $mged_sketch_fps {;}]
     puts $fo "interp $cmdstr 0 1 2;"
@@ -1083,10 +1085,14 @@
     }
     text $buffer
     $text configure -state normal
+
+    set anim_orient [file join ${mged_sketch_anim_path} anim_orient]
+    set anim_lookat [file join ${mged_sketch_anim_path} anim_lookat]
+    set anim_cascade [file join ${mged_sketch_anim_path} anim_cascade]
+    set chan_permute [file join ${mged_sketch_anim_path} chan_permute]
     switch $oldlist {
        {size eye quat} {
-           set fd [open "| ${mged_sketch_anim_path}anim_orient qv y \
-                                       > $mged_sketch_temp1 " w]
+           set fd [open "| $anim_orient qv y > $mged_sketch_temp1 " w]
            sketch_text_to_fd $text $fd "5,6,7,8"
            catch {close $fd}
            set fd [open $mged_sketch_temp1 r]
@@ -1096,9 +1102,7 @@
            file delete $mged_sketch_temp1
        }
        {eye center} {
-
-           set fd [open "| ${mged_sketch_anim_path}anim_lookat -y -v \
-                                       > $mged_sketch_temp1" w]
+           set fd [open "| $anim_lookat -y -v > $mged_sketch_temp1" w]
            sketch_text_to_fd $text $fd all
            close $fd
            set fd [open $mged_sketch_temp1 r]
@@ -1107,7 +1111,7 @@
            file delete $mged_sketch_temp1
        }
        {size center ypr} {
-           set fd [open "| ${mged_sketch_anim_path}anim_cascade -ry 0 0 0 > 
$mged_sketch_temp1" w]
+           set fd [open "| $anim_cascade -ry 0 0 0 > $mged_sketch_temp1" w]
            sketch_text_do_script $buffer $text all {...@0 @2 @3 @4 @5 @6 @7 
{...@1/2.0} 0.0 0.0}
            sketch_text_to_fd $buffer $fd all
            close $fd
@@ -1119,11 +1123,11 @@
            file delete $mged_sketch_temp1
        }
        {size center quat} {
-           set fd [open "| ${mged_sketch_anim_path}anim_orient qv y > 
$mged_sketch_temp1" w]
+           set fd [open "| $anim_orient qv y > $mged_sketch_temp1" w]
            sketch_text_to_fd $text $fd "5,6,7,8"
            catch {close $fd}
            sketch_text_do_script $buffer $text all {...@0 @2 @3 @4 {...@1/2.0} 
0.0 0.0}
-           set fd [open "| ${mged_sketch_anim_path}chan_permute -i stdin 0 1 2 
3 4 5 6 -i $mged_sketch_temp1 8 9 10 -o stdout 0 1 2 3 8 9 10 4 5 6 | 
${mged_sketch_anim_path}anim_cascade -ry 0 0 0 > $mged_sketch_temp2" w]
+           set fd [open "| $chan_permute -i stdin 0 1 2 3 4 5 6 -i 
$mged_sketch_temp1 8 9 10 -o stdout 0 1 2 3 8 9 10 4 5 6 | $anim_cascade -ry 0 
0 0 > $mged_sketch_temp2" w]
            sketch_text_to_fd $buffer $fd all
            close $fd
            set fd [open $mged_sketch_temp2 r]
@@ -1142,8 +1146,7 @@
     #convert from {size eye ypr}
     switch $newlist {
        {size eye quat} {
-           set fd [open "| ${mged_sketch_anim_path}anim_orient y qv \
-                                       > $mged_sketch_temp1 " w]
+           set fd [open "| $anim_orient y qv > $mged_sketch_temp1 " w]
            sketch_text_to_fd $text $fd "5,6,7"
            catch {close $fd}
            set fd [open $mged_sketch_temp1 r]
@@ -1155,8 +1158,7 @@
        {eye center} {
            sketch_text_do_script $buffer $text all \
                {...@0 @2 @3 @4 @5 @6 @7 {...@1*0.5} 0.0 0.0 0.0 0.0 0.0}
-           set fd [open "| ${mged_sketch_anim_path}anim_cascade \
-                                       > $mged_sketch_temp2" w]
+           set fd [open "| $anim_cascade > $mged_sketch_temp2" w]
            sketch_text_to_fd $buffer $fd all
            close $fd
            sketch_text_col_arith $text all {...@0 @2 @3 @4}
@@ -1166,7 +1168,7 @@
            file delete $mged_sketch_temp2
        }
        {size center ypr} {
-           set fd [open "| ${mged_sketch_anim_path}anim_cascade -ry 0 0 0 > 
$mged_sketch_temp1" w]
+           set fd [open "| $anim_cascade -ry 0 0 0 > $mged_sketch_temp1" w]
            sketch_text_do_script $buffer $text all {...@0 @2 @3 @4 @5 @6 @7 
{...@1/2.0} 0.0 0.0}
            sketch_text_to_fd $buffer $fd all
            close $fd
@@ -1178,11 +1180,11 @@
            file delete $mged_sketch_temp1
        }
        {size center quat} {
-           set fd [open "| ${mged_sketch_anim_path}anim_cascade -ry 0 0 0 > 
$mged_sketch_temp1" w]
+           set fd [open "| $anim_cascade -ry 0 0 0 > $mged_sketch_temp1" w]
            sketch_text_do_script $buffer $text all {...@0 @2 @3 @4 @5 @6 @7 
{...@1/2.0} 0.0 0.0}
            sketch_text_to_fd $buffer $fd all
            close $fd
-           set fd [open "| ${mged_sketch_anim_path}chan_permute -i 
$mged_sketch_temp1 0 1 2 3 4 5 6 -o stdout 4 5 6 | 
${mged_sketch_anim_path}anim_orient y qv | ${mged_sketch_anim_path}chan_permute 
-i stdin 7 8 9 10 -i $mged_sketch_temp1 0 1 2 3 4 5 6 -o stdout 1 2 3 7 8 9 10" 
r]
+           set fd [open "| $chan_permute -i $mged_sketch_temp1 0 1 2 3 4 5 6 
-o stdout 4 5 6 | $anim_orient y qv | $chan_permute -i stdin 7 8 9 10 -i 
$mged_sketch_temp1 0 1 2 3 4 5 6 -o stdout 1 2 3 7 8 9 10" r]
            $text delete 1.0 end
            sketch_text_do_script $text $buffer all {...@0 {2...@7}}
            sketch_text_from_fd $text $fd all right
@@ -1843,7 +1845,8 @@
     #count number of lines, doesn't matter if a couple extra
     scan [$w index end] %d maxlen
     set arg2 "-m $maxlen"
-    set cmd "| ${mged_sketch_anim_path}anim_time $arg0 $arg1 $arg2 > 
$mged_sketch_temp1"
+    set anim_time [file join ${mged_sketch_anim_path} anim_time]
+    set cmd "| $anim_time $arg0 $arg1 $arg2 > $mged_sketch_temp1"
     #puts $cmd
     set f1 [open $cmd w]
     set mycols "0,$cols"
@@ -2248,7 +2251,8 @@
     }
     close $fd
 
-    set fd [open "| ${mged_sketch_tab_path}tabinterp -q < $mged_sketch_temp2 " 
r]
+    set tabinterp [file join ${mged_sketch_tab_path} tabinterp]
+    set fd [open "| $tabinterp -q < $mged_sketch_temp2 " r]
     sketch_text_from_fd $w $fd all replace
     #catch can be removed when -q option added to tabinterp
     catch {close $fd}
@@ -2710,6 +2714,12 @@
     set tmp [db get $mged_sketch_objname]
     if { $tmp == 0 } return
 
+    set anim_fly [file join ${mged_sketch_anim_path} anim_fly]
+    set anim_lookat [file join ${mged_sketch_anim_path} anim_lookat]
+    set anim_script [file join ${mged_sketch_anim_path} anim_script]
+    set anim_cascade [file join ${mged_sketch_anim_path} anim_cascade]
+    set chan_permute [file join ${mged_sketch_anim_path} chan_permute]
+
     #find the source
     switch $ltype {
        "curve:" {
@@ -2818,7 +2828,7 @@
                        append outcol " $i"
                    }
                }
-               set filecmd "${mged_sketch_anim_path}chan_permute -i $src 
$incol -o stdout $outcol"
+               set filecmd "$chan_permute -i $src $incol -o stdout $outcol"
            } elseif { $nsrc < $ncols } {
                tk_dialog ._sketch_msg {Insufficient columns} \
                    "The animation you requested requires $ncols \
@@ -2841,7 +2851,6 @@
        }
     }
 
-
     # build argument string
     if {$mged_sketch_objframe == ""} { set mged_sketch_objframe 0}
     set opts "-f $mged_sketch_objframe"
@@ -2856,8 +2865,7 @@
        if {$wcen == ""} { set wcen "0 0 0"; incr i }
        if {$wypr == ""} { set wypr "0 0 0"; incr i }
        if { $i < 4 } {
-           set fd [open "| ${mged_sketch_anim_path}anim_cascade \
-                          -or -fc $wcen -fy $wypr -ac $rcen -ay $rypr" r]
+           set fd [open "| $anim_cascade -or -fc $wcen -fy $wypr -ac $rcen -ay 
$rypr" r]
            gets $fd line
            close $fd
            set veye [lrange $line 1 3]
@@ -2885,7 +2893,6 @@
     #puts "anim_script options: $opts"
 
     if { $mged_sketch_objopt == "lookat" } {
-       set anim_lookat ${mged_sketch_anim_path}anim_lookat
        if { $type == "curve" } {
            #This shouldn't happen
            puts "sketch_objanim: Can't do lookat orientation \
@@ -2893,7 +2900,7 @@
            return
        } elseif { $type == "text" } {
            set fd [open "| $anim_lookat -y $lookat_v | \
-                         ${mged_sketch_anim_path}anim_script $opts $ovname > \
+                         $anim_script $opts $ovname > \
                          $mged_sketch_objscript" w]
            sketch_text_to_fd $w $fd $colsp
            catch {close $fd}
@@ -2901,11 +2908,11 @@
        } elseif { $type == "file" } {
            if { $filecmd == "" } {
                catch {eval exec $anim_lookat -y $lookat_v < $src | \
-                          ${mged_sketch_anim_path}anim_script $opts $ovname > \
+                          $anim_script $opts $ovname > \
                           $mged_sketch_objscript}
            } else {
                catch {eval exec $filecmd | $anim_lookat -y $lookat_v | \
-                          ${mged_sketch_anim_path}anim_script $opts $ovname > \
+                          $anim_script $opts $ovname > \
                           $mged_sketch_objscript}
            }
 
@@ -2921,7 +2928,6 @@
        } elseif { $mged_sketch_objmang < -89 } {
            set mged_sketch_objmang -89
        }
-       set do_bank ${mged_sketch_anim_path}anim_fly
        if { $type == "curve" } {
            set sfile $mged_sketch_temp1
            set fd [open $sfile w]
@@ -2941,9 +2947,8 @@
            }
        }
 
-       set factor [exec $do_bank -b $mged_sketch_objmang < $sfile]
-       eval exec $do_bank -f $factor < $sfile \
-           | ${mged_sketch_anim_path}anim_script $opts $ovname > 
$mged_sketch_objscript
+       set factor [exec $anim_fly -b $mged_sketch_objmang < $sfile]
+       eval exec $anim_fly -f $factor < $sfile | $anim_script $opts $ovname > 
$mged_sketch_objscript
 
        if { $type == "curve" } {
            sketch_open_curve $oldcurve
@@ -2974,22 +2979,22 @@
     if { $type == "file"} {
        #puts "filecmd = $filecmd src = $src"
        if { $filecmd == "" } {
-           eval exec ${mged_sketch_anim_path}anim_script $opts $ovname < $src 
> \
+           eval exec $anim_script $opts $ovname < $src > \
                $mged_sketch_objscript
        } else {
-           eval exec $filecmd | ${mged_sketch_anim_path}anim_script $opts 
$ovname | \
+           eval exec $filecmd | $anim_script $opts $ovname | \
                $mged_sketch_objscript
        }
     } elseif { $type == "curve" } {
        set fd [open \
-                   [concat | ${mged_sketch_anim_path}anim_script $opts $ovname 
> \
+                   [concat | $anim_script $opts $ovname > \
                         $mged_sketch_objscript] w ]
        sketch_write_to_fd $fd [vdraw read l]
        close $fd
        sketch_open_curve $oldcurve
     } elseif { $type == "text" } {
        set fd [open \
-                   [concat | ${mged_sketch_anim_path}anim_script $opts $ovname 
> \
+                   [concat | $anim_script $opts $ovname > \
                         $mged_sketch_objscript] w ]
        sketch_text_to_fd $w $fd $colsp
        close $fd
@@ -3179,11 +3184,13 @@
 
     set text [sketch_text_from_table $tid 4]
     if { $text == "" } {return 0}
+
     set fd [open $mged_sketch_temp1 w]
     sketch_text_to_fd $text $fd all
     close $fd
-    set fd [open "| ${mged_sketch_anim_path}anim_track \
-               -c $mged_sketch_temp1" r]
+
+    set anim_track [file join ${mged_sketch_anim_path} anim_track]
+    set fd [open "| $anim_track -c $mged_sketch_temp1" r]
     catch {flush $fd}
     gets $fd length
     catch {close $fd}
@@ -3210,6 +3217,7 @@
     upvar #0 mged_sketch_objscript outfile
     set ypr $mged_sketch_objori
     set center $mged_sketch_objcen
+    set anim_track [file join ${mged_sketch_anim_path} anim_track]
 
     #check for overwriting script file
     if {[file exists $outfile] } {
@@ -3321,8 +3329,7 @@
                -d $center $fcmd $wcmd $pcmd"
     #puts $myargs
 
-    set fd [ open "| ${mged_sketch_anim_path}anim_track \
-               $myargs $mged_sketch_temp1 > $outfile" w]
+    set fd [ open "| $anim_track $myargs $mged_sketch_temp1 > $outfile" w]
     sketch_text_to_fd $vtext $fd all
     close $fd
     file delete $mged_sketch_temp1
@@ -3449,9 +3456,10 @@
     foreach file [$list get 0 end] {
        append files "$file "
     }
-    set pid [eval exec cat $files | ${mged_sketch_tab_path}scriptsort \
-                -q -b 1 > $outfile &]
 
+    set scriptsort [file join ${mged_sketch_tab_path} scriptsort]
+    set pid [eval exec cat $files | $scriptsort -q -b 1 > $outfile &]
+
     frame $sortp.fa
     label $sortp.fa.l0 -text "Sorting $outfile ..."
     button $sortp.fa.b0 -text "Halt" -command "exec kill $pid"


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

------------------------------------------------------------------------------
What happens now with your Lotus Notes apps - do you make another costly 
upgrade, or settle for being marooned without product support? Time to move
off Lotus Notes and onto the cloud with Force.com, apps are easier to build,
use, and manage than apps on traditional platforms. Sign up for the Lotus 
Notes Migration Kit to learn more. http://p.sf.net/sfu/salesforce-d2d
_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to