Re: tmpdir problem

2008-07-28 Thread rgheck

Bennett Helm wrote:

On Thu, Jul 24, 2008 at 5:20 PM, Abdelrazak Younes [EMAIL PROTECTED] wrote:
  
It seems this fixes bug 5085 (at least I'm not seeing the problem there

anymore):

http://bugzilla.lyx.org/show_bug.cgi?id=5085

Could you take a look and close it if appropriate?

  

Done.

rh



Re: tmpdir problem

2008-07-28 Thread rgheck

Bennett Helm wrote:

On Thu, Jul 24, 2008 at 5:20 PM, Abdelrazak Younes <[EMAIL PROTECTED]> wrote:
  
It seems this fixes bug 5085 (at least I'm not seeing the problem there

anymore):

http://bugzilla.lyx.org/show_bug.cgi?id=5085

Could you take a look and close it if appropriate?

  

Done.

rh



Re: tmpdir problem

2008-07-27 Thread Bennett Helm
On Thu, Jul 24, 2008 at 5:20 PM, Abdelrazak Younes [EMAIL PROTECTED] wrote:

 rgheck wrote:

 Enrico Forestieri wrote:

 With latest svn, LyX is not able to create the tmpdir:

 $ lyx
 ../../../src/support/filetools.cpp(328): LyX could not create temporary
 directory in /tmp'

 $ lyx -dbg files
 Setting debug level to files
 Debugging `files' (Files used by LyX)
 ../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
 createTmpDir:mask=`lyx_tmpdir'
 ../../../src/support/FileName.cpp(375): Unable to create temporary file
 with following template: /lyx_tmpdir
 ../../../src/support/filetools.cpp(328): LyX could not create temporary
 directory in /tmp'

 Abdel, I see that you recently touched that.

  I'm not seeing any problem on Linux. But mine looks like this:

 Debugging `files' (Files used by LyX)
 filetools.cpp(322): createTmpDir: tempdir=`/tmp/'
 createTmpDir: mask=`lyx_tmpdir'
 FileName.cpp(371): Temporary file `/tmp/lyx_tmpdir.J21987' created.

 Note the trailing slash: /tmp/. Also, the template is wrong. It should be:
 /tmp/lyx_tmpdir, not just /lyx_tmpdir.

 The problem is here:

 FileName FileName::tempName(FileName const  temp_dir, string const 
 mask)
 {
 QFileInfo tmp_fi(temp_dir.d-fi.absoluteDir(), toqstr(mask));
 return FileName(createTempFile(tmp_fi.absoluteFilePath()));
 }

 tmp_fi.absoluteFilePath() is doing the wrong thing with /tmp. It's
 giving us the path to where /tmp is---viz, /---not the path to /tmp itself,
 since the trailing slash is missing. Hence the /lyx_tmpdir, and the failure.

 Replacing absoluteDir() there with absoluteFilePath() may do the trick.


 That's what I did yes.


It seems this fixes bug 5085 (at least I'm not seeing the problem there
anymore):

http://bugzilla.lyx.org/show_bug.cgi?id=5085

Could you take a look and close it if appropriate?

Thanks.

Bennett


Re: tmpdir problem

2008-07-27 Thread Bennett Helm
On Thu, Jul 24, 2008 at 5:20 PM, Abdelrazak Younes <[EMAIL PROTECTED]> wrote:

> rgheck wrote:
>
>> Enrico Forestieri wrote:
>>
>>> With latest svn, LyX is not able to create the tmpdir:
>>>
>>> $ lyx
>>> ../../../src/support/filetools.cpp(328): LyX could not create temporary
>>> directory in /tmp'
>>>
>>> $ lyx -dbg files
>>> Setting debug level to files
>>> Debugging `files' (Files used by LyX)
>>> ../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
>>> createTmpDir:mask=`lyx_tmpdir'
>>> ../../../src/support/FileName.cpp(375): Unable to create temporary file
>>> with following template: /lyx_tmpdir
>>> ../../../src/support/filetools.cpp(328): LyX could not create temporary
>>> directory in /tmp'
>>>
>>> Abdel, I see that you recently touched that.
>>>
>>>  I'm not seeing any problem on Linux. But mine looks like this:
>>
>> Debugging `files' (Files used by LyX)
>> filetools.cpp(322): createTmpDir: tempdir=`/tmp/'
>> createTmpDir: mask=`lyx_tmpdir'
>> FileName.cpp(371): Temporary file `/tmp/lyx_tmpdir.J21987' created.
>>
>> Note the trailing slash: /tmp/. Also, the template is wrong. It should be:
>> /tmp/lyx_tmpdir, not just /lyx_tmpdir.
>>
>> The problem is here:
>>
>> FileName FileName::tempName(FileName const & temp_dir, string const &
>> mask)
>> {
>> QFileInfo tmp_fi(temp_dir.d->fi.absoluteDir(), toqstr(mask));
>> return FileName(createTempFile(tmp_fi.absoluteFilePath()));
>> }
>>
>> tmp_fi.absoluteFilePath() is doing the wrong thing with "/tmp". It's
>> giving us the path to where /tmp is---viz, /---not the path to /tmp itself,
>> since the trailing slash is missing. Hence the /lyx_tmpdir, and the failure.
>>
>> Replacing absoluteDir() there with absoluteFilePath() may do the trick.
>>
>
> That's what I did yes.


It seems this fixes bug 5085 (at least I'm not seeing the problem there
anymore):

http://bugzilla.lyx.org/show_bug.cgi?id=5085

Could you take a look and close it if appropriate?

Thanks.

Bennett


Re: tmpdir problem

2008-07-24 Thread Abdelrazak Younes

Enrico Forestieri wrote:

With latest svn, LyX is not able to create the tmpdir:

$ lyx
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

$ lyx -dbg files
Setting debug level to files
Debugging `files' (Files used by LyX)
../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
createTmpDir:mask=`lyx_tmpdir'
../../../src/support/FileName.cpp(375): Unable to create temporary file with 
following template: /lyx_tmpdir
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

Abdel, I see that you recently touched that.
   

Seems to. I'll fix that, sorry.

Abdel.




Re: tmpdir problem

2008-07-24 Thread rgheck

Enrico Forestieri wrote:

With latest svn, LyX is not able to create the tmpdir:

$ lyx
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

$ lyx -dbg files
Setting debug level to files
Debugging `files' (Files used by LyX)
../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
createTmpDir:mask=`lyx_tmpdir'
../../../src/support/FileName.cpp(375): Unable to create temporary file with 
following template: /lyx_tmpdir
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

Abdel, I see that you recently touched that.

  

I'm not seeing any problem on Linux. But mine looks like this:

Debugging `files' (Files used by LyX)
filetools.cpp(322): createTmpDir: tempdir=`/tmp/'
createTmpDir: mask=`lyx_tmpdir'
FileName.cpp(371): Temporary file `/tmp/lyx_tmpdir.J21987' created.

Note the trailing slash: /tmp/. Also, the template is wrong. It should 
be: /tmp/lyx_tmpdir, not just /lyx_tmpdir.


The problem is here:

FileName FileName::tempName(FileName const  temp_dir, string const  mask)
{
QFileInfo tmp_fi(temp_dir.d-fi.absoluteDir(), toqstr(mask));
return FileName(createTempFile(tmp_fi.absoluteFilePath()));
}

tmp_fi.absoluteFilePath() is doing the wrong thing with /tmp. It's 
giving us the path to where /tmp is---viz, /---not the path to /tmp 
itself, since the trailing slash is missing. Hence the /lyx_tmpdir, and 
the failure.


Replacing absoluteDir() there with absoluteFilePath() may do the trick.

rh




Re: tmpdir problem

2008-07-24 Thread Abdelrazak Younes

rgheck wrote:

Enrico Forestieri wrote:

With latest svn, LyX is not able to create the tmpdir:

$ lyx
../../../src/support/filetools.cpp(328): LyX could not create 
temporary directory in /tmp'


$ lyx -dbg files
Setting debug level to files
Debugging `files' (Files used by LyX)
../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
createTmpDir:mask=`lyx_tmpdir'
../../../src/support/FileName.cpp(375): Unable to create temporary 
file with following template: /lyx_tmpdir
../../../src/support/filetools.cpp(328): LyX could not create 
temporary directory in /tmp'


Abdel, I see that you recently touched that.


I'm not seeing any problem on Linux. But mine looks like this:

Debugging `files' (Files used by LyX)
filetools.cpp(322): createTmpDir: tempdir=`/tmp/'
createTmpDir: mask=`lyx_tmpdir'
FileName.cpp(371): Temporary file `/tmp/lyx_tmpdir.J21987' created.

Note the trailing slash: /tmp/. Also, the template is wrong. It should 
be: /tmp/lyx_tmpdir, not just /lyx_tmpdir.


The problem is here:

FileName FileName::tempName(FileName const  temp_dir, string const  
mask)

{
QFileInfo tmp_fi(temp_dir.d-fi.absoluteDir(), toqstr(mask));
return FileName(createTempFile(tmp_fi.absoluteFilePath()));
}

tmp_fi.absoluteFilePath() is doing the wrong thing with /tmp. It's 
giving us the path to where /tmp is---viz, /---not the path to /tmp 
itself, since the trailing slash is missing. Hence the /lyx_tmpdir, 
and the failure.


Replacing absoluteDir() there with absoluteFilePath() may do the trick.


That's what I did yes.

Abdel.



Re: tmpdir problem

2008-07-24 Thread Enrico Forestieri
On Thu, Jul 24, 2008 at 05:13:55PM -0400, rgheck wrote:

 I'm not seeing any problem on Linux. But mine looks like this:

Mmm. I see that on both Debian and Cygwin. Maybe you have write
permission in '/'?

-- 
Enrico


Re: tmpdir problem

2008-07-24 Thread Abdelrazak Younes

Enrico Forestieri wrote:

With latest svn, LyX is not able to create the tmpdir:

$ lyx
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

$ lyx -dbg files
Setting debug level to files
Debugging `files' (Files used by LyX)
../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
createTmpDir:mask=`lyx_tmpdir'
../../../src/support/FileName.cpp(375): Unable to create temporary file with 
following template: /lyx_tmpdir
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

Abdel, I see that you recently touched that.
   

Seems to. I'll fix that, sorry.

Abdel.




Re: tmpdir problem

2008-07-24 Thread rgheck

Enrico Forestieri wrote:

With latest svn, LyX is not able to create the tmpdir:

$ lyx
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

$ lyx -dbg files
Setting debug level to files
Debugging `files' (Files used by LyX)
../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
createTmpDir:mask=`lyx_tmpdir'
../../../src/support/FileName.cpp(375): Unable to create temporary file with 
following template: /lyx_tmpdir
../../../src/support/filetools.cpp(328): LyX could not create temporary 
directory in /tmp'

Abdel, I see that you recently touched that.

  

I'm not seeing any problem on Linux. But mine looks like this:

Debugging `files' (Files used by LyX)
filetools.cpp(322): createTmpDir: tempdir=`/tmp/'
createTmpDir: mask=`lyx_tmpdir'
FileName.cpp(371): Temporary file `/tmp/lyx_tmpdir.J21987' created.

Note the trailing slash: /tmp/. Also, the template is wrong. It should 
be: /tmp/lyx_tmpdir, not just /lyx_tmpdir.


The problem is here:

FileName FileName::tempName(FileName const & temp_dir, string const & mask)
{
QFileInfo tmp_fi(temp_dir.d->fi.absoluteDir(), toqstr(mask));
return FileName(createTempFile(tmp_fi.absoluteFilePath()));
}

tmp_fi.absoluteFilePath() is doing the wrong thing with "/tmp". It's 
giving us the path to where /tmp is---viz, /---not the path to /tmp 
itself, since the trailing slash is missing. Hence the /lyx_tmpdir, and 
the failure.


Replacing absoluteDir() there with absoluteFilePath() may do the trick.

rh




Re: tmpdir problem

2008-07-24 Thread Abdelrazak Younes

rgheck wrote:

Enrico Forestieri wrote:

With latest svn, LyX is not able to create the tmpdir:

$ lyx
../../../src/support/filetools.cpp(328): LyX could not create 
temporary directory in /tmp'


$ lyx -dbg files
Setting debug level to files
Debugging `files' (Files used by LyX)
../../../src/support/filetools.cpp(322): createTmpDir: tempdir=`/tmp'
createTmpDir:mask=`lyx_tmpdir'
../../../src/support/FileName.cpp(375): Unable to create temporary 
file with following template: /lyx_tmpdir
../../../src/support/filetools.cpp(328): LyX could not create 
temporary directory in /tmp'


Abdel, I see that you recently touched that.


I'm not seeing any problem on Linux. But mine looks like this:

Debugging `files' (Files used by LyX)
filetools.cpp(322): createTmpDir: tempdir=`/tmp/'
createTmpDir: mask=`lyx_tmpdir'
FileName.cpp(371): Temporary file `/tmp/lyx_tmpdir.J21987' created.

Note the trailing slash: /tmp/. Also, the template is wrong. It should 
be: /tmp/lyx_tmpdir, not just /lyx_tmpdir.


The problem is here:

FileName FileName::tempName(FileName const & temp_dir, string const & 
mask)

{
QFileInfo tmp_fi(temp_dir.d->fi.absoluteDir(), toqstr(mask));
return FileName(createTempFile(tmp_fi.absoluteFilePath()));
}

tmp_fi.absoluteFilePath() is doing the wrong thing with "/tmp". It's 
giving us the path to where /tmp is---viz, /---not the path to /tmp 
itself, since the trailing slash is missing. Hence the /lyx_tmpdir, 
and the failure.


Replacing absoluteDir() there with absoluteFilePath() may do the trick.


That's what I did yes.

Abdel.



Re: tmpdir problem

2008-07-24 Thread Enrico Forestieri
On Thu, Jul 24, 2008 at 05:13:55PM -0400, rgheck wrote:

> I'm not seeing any problem on Linux. But mine looks like this:

Mmm. I see that on both Debian and Cygwin. Maybe you have write
permission in '/'?

-- 
Enrico