Great! This fixes the analogous problem I had with the VMware player mount a
few weeks ago. Thanks!
-David
Quoting Steve Smith <[log in to unmask]>:
> Aha! Good call - yes, I'm pretty sure that in fact the line:
>
> set report [ open ${pagename} w ]
>
> in $FSLDIR/tcl/featlib.tcl
>
> is wrong (and in fact unnecessary) - we'll remove it for the next
> patch but for the moment can you just delete it and see if that fixes
> things?
>
> Thanks, Steve.
>
>
>
> On 3 Oct 2007, at 21:45, Syam Gadde wrote:
>
>> I've tracked it down, I think. Essentially, the first
>> call to feat5:report_insert on a file works. But the
>> second and subsequent calls on that same file do not work.
>> On the second call, when it gets into report_insert_start,
>> the "mv" fails with this error:
>>
>> mv: cannot move `report_prestats.html' to
>> `tmpreport_prestats.html': Text file busy
>>
>> There is a global variable "report" defined in report_insert
>> and its subsidiary functions, which seems to hold a file handle
>> open for writing but doesn't close it. What is the function
>> of this variable? I suspect getting rid of the following line
>> in report_insert_start:
>>
>> set report [ open ${pagename} w ]
>>
>> (as well as the "global report" lines) might fix our issue.
>> Do you see any hidden side effect that might have?
>>
>> -syam
>>
>>
>> Matthew Webster wrote:
>>> Hi,
>>> We've tried setting up a smb mount on a linux box, but so far we've
>>> been unable to reproduce your error - are you able to access files from
>>> the tclsh command line e.g.:
>>> set fp [open "/path/to/fileonsmb.txt" r]
>>> or do all read attempts fail? Are there any other unusual messages in the
>>> feat log?
>>> Matthew
>>>> Any luck replicating the errors? This is about to be a large problem
>>>> for us, as we are about to open up a large grid.
>>>>
>>>> Thanks,
>>>>
>>>> chris
>>>>
>>>> -----Original Message-----
>>>> From: FSL - FMRIB's Software Library [mailto:[log in to unmask]] On
>>>> Behalf Of Matthew Webster
>>>> Sent: Friday, September 28, 2007 5:11 AM
>>>> To: [log in to unmask]
>>>> Subject: Re: [FSL] feat html erros
>>>>
>>>> Hi,
>>>> Can you let us know some more about your system and what kind of
>>>> mountpoint is being used ( NFS,SMB,VMPlayers connection to windows disk
>>>> )
>>>>
>>>> Thanks
>>>>
>>>> Matthew
>>>>> We'll try to reproduce this locally and see if we can find a
>>>>> workaround (seems like it's a VMPlayer networking bug).
>>>>> Cheers.
>>>>>
>>>>>
>>>>> On 27 Sep 2007, at 21:27, Chris Petty wrote:
>>>>>
>>>>>> With FSL 4.0.1 we've been running into errors in out feat reports.
>>>>>> Everything runs, however the results never get put into the actual
>>>>>> htmls.
>>>>>>
>>>>>> These happen when sending feat output across a mountpoint.
>>>>>>
>>>>>> You can substitute prestats with poststats,reg or stats, it happens
>>>>>> whenever feat tries to use a tmp report. Any ideas or help? A lot
>>>>>> of people here have to output across a mount, some can write locally
>>>>>> (me
>>>>>> included) and
>>>>>> locally I haven't had a problem.
>>>>>>
>>>>>> couldn't open "tmpreport_prestats.html": no such file or directory
>>>>>> while executing
>>>>>> "open tmp${pagename} r "
>>>>>> (procedure "feat5:report_insert_start" line 5)
>>>>>> invoked from within
>>>>>> "feat5:report_insert_start $pagename $sectionlabel"
>>>>>> (procedure "feat5:report_insert" line 4)
>>>>>> invoked from within
>>>>>> "feat5:report_insert report_prestats.html prestatsrs $rs"
>>>>>> (procedure "feat5:proc_prestats" line 455)
>>>>>> invoked from within
>>>>>> "feat5:proc_prestats $session "
>>>>>> ("-prestats" arm line 2)
>>>>>> invoked from within
>>>>>> "switch -- [ lindex $argv $argindex ] {
>>>>>>
>>>>>> -I {
>>>>>> incr argindex 1
>>>>>> set session [ lindex $argv $argindex ]
>>>>>> }
>>>>>>
>>>>>> -D {
>>>>>> incr argindex 1
>>>>>> set..."
>>>>>> ("for" body line 2)
>>>>>> invoked from within
>>>>>> "for { set argindex 1 } { $argindex < $argc } { incr argindex 1 } {
>>>>>> switch -- [ lindex $argv $argindex ] {
>>>>>>
>>>>>> -I {
>>>>>> incr argindex 1
>>>>>> set ses..."
>>>>>> (file "/usr/local/fsl/bin/feat" line 137)
>>>>>>
>>>>>> ####################stats
>>>>>>
>>>>>> couldn't open "tmpreport_stats.html": no such file or directory
>>>>>> while executing
>>>>>> "open tmp${pagename} r "
>>>>>> (procedure "feat5:report_insert_start" line 5)
>>>>>> invoked from within
>>>>>> "feat5:report_insert_start $pagename $sectionlabel"
>>>>>> (procedure "feat5:report_insert" line 4)
>>>>>> invoked from within
>>>>>> "feat5:report_insert report_stats.html statsrs $rs"
>>>>>> (procedure "feat5:proc_film" line 83)
>>>>>> invoked from within
>>>>>> "feat5:proc_film "
>>>>>> ("-film" arm line 2)
>>>>>> invoked from within
>>>>>> "switch -- [ lindex $argv $argindex ] {
>>>>>>
>>>>>> -I {
>>>>>> incr argindex 1
>>>>>> set session [ lindex $argv $argindex ]
>>>>>> }
>>>>>>
>>>>>> -D {
>>>>>> incr argindex 1
>>>>>> set..."
>>>>>> ("for" body line 2)
>>>>>> invoked from within
>>>>>> "for { set argindex 1 } { $argindex < $argc } { incr argindex 1 } {
>>>>>> switch -- [ lindex $argv $argindex ] {
>>>>>>
>>>>>> -I {
>>>>>> incr argindex 1
>>>>>> set ses..."
>>>>>> (file "/usr/local/fsl/bin/feat" line 137)
>>>>>>
>>>>>> #################post-stats
>>>>>>
>>>>>> couldn't open "tmpreport_poststats.html": no such file or directory
>>>>>> while executing
>>>>>> "open tmp${pagename} r "
>>>>>> (procedure "feat5:report_insert_start" line 5)
>>>>>> invoked from within
>>>>>> "feat5:report_insert_start $pagename $sectionlabel"
>>>>>> (procedure "feat5:report_insert" line 4)
>>>>>> invoked from within
>>>>>> "feat5:report_insert report_poststats.html poststatsrs $rs"
>>>>>> (procedure "feat5:proc_poststats" line 357)
>>>>>> invoked from within
>>>>>> "feat5:proc_poststats $RERUNNING $STDSPACE "
>>>>>> ("-poststats" arm line 6)
>>>>>> invoked from within
>>>>>> "switch -- [ lindex $argv $argindex ] {
>>>>>>
>>>>>> -I {
>>>>>> incr argindex 1
>>>>>> set session [ lindex $argv $argindex ]
>>>>>> }
>>>>>>
>>>>>> -D {
>>>>>> incr argindex 1
>>>>>> set..."
>>>>>> ("for" body line 2)
>>>>>> invoked from within
>>>>>> "for { set argindex 1 } { $argindex < $argc } { incr argindex 1 } {
>>>>>> switch -- [ lindex $argv $argindex ] {
>>>>>>
>>>>>> -I {
>>>>>> incr argindex 1
>>>>>> set ses..."
>>>>>> (file "/usr/local/fsl/bin/feat" line 137)
>>>>>
>>>>> ------------------------------------------------------------------- ---
>>>>> -----
>>>>> Stephen M. Smith, Professor of Biomedical Engineering Associate
>>>>> Director, Oxford University FMRIB Centre
>>>>>
>>>>> FMRIB, JR Hospital, Headington, Oxford OX3 9DU, UK
>>>>> +44 (0) 1865 222726 (fax 222717)
>>>>> [log in to unmask] http://www.fmrib.ox.ac.uk/~steve
>>>>> ------------------------------------------------------------------- ---
>>>>> -----
>
>
> ------------------------------------------------------------------------ ---
> Stephen M. Smith, Professor of Biomedical Engineering
> Associate Director, Oxford University FMRIB Centre
>
> FMRIB, JR Hospital, Headington, Oxford OX3 9DU, UK
> +44 (0) 1865 222726 (fax 222717)
> [log in to unmask] http://www.fmrib.ox.ac.uk/~steve
> ------------------------------------------------------------------------ ---
>
|