Re: tmpdir problem
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
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
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
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
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
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
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
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
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
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
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
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