[innotop-issues] Issue 31 in innotop: location of configuration file on windows?

[inn...@googlecode.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 31 by ccurvey: location of configuration file on windows?
http://code.google.com/p/innotop/issues/detail?id=31

This is most likely "IBKAC" (Idiot Between Keyboard and Chair), but I can't
figure it out. Where is the innotop.ini file on Windows 7? Innotop seems
to have created a directory in c:\users\ccurvey\.innotop, but there are no
files in that directory.

If I run innotop with the -w flag, I get these errors on quitting:

Can't rename \Users\ccurvey\.innotop\innotopwrk4s.conf.tmp to
\Users\ccurvey/.innotop/innotop.conf: Permission denied at innotop line
7556.

Loaded config file on start-up, so ignoring -w (see --help)
Can't rename \Users\ccurvey\.innotop\innotopUlB0V.conf.tmp to
\Users\ccurvey/.innotop/innotop.conf: Permission denied at innotop line
7556.

Since I'm running innotop as myself, and have full permissions on that
directory, I'm wondering if the problem is that there's no drive letter on
those file paths?

--
You received this message because you are subscribed to the Google Groups "Innotop Issues" group.
To post to this group, send email to innotop...@googlegroups.com.
To unsubscribe from this group, send email to innotop-issue...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/innotop-issues?hl=en.

[inn...@googlecode.com]

Comment #1 on issue 31 by ccurvey: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

ack! did not mean to file this as a "defect", but I can't seem to change it
now.

[inn...@googlecode.com]

Updates:
Labels: -Type-Defect Type-Other

Comment #2 on issue 31 by baron.schwartz: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

I am not sure what's going on there. The directory in question is
determined by this
code:

# Find the home directory; it's different on different OSes.
our $homepath = $ENV{HOME} || $ENV{HOMEPATH} || $ENV{USERPROFILE} || '.';

Can you check these environment variables and see what's set?

[inn...@googlecode.com]

Comment #3 on issue 31 by ccurvey: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

HOMEPATH=\users\ccurvey
USERPROFILE=C:\users\ccurvey

So I tried hacking that line to make USERPROFILE come first, but I'm
getting the same permission problem. It's
got to be something simple.

[inn...@googlecode.com]

Comment #4 on issue 31 by baron.schwartz: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

Can you manually create the files that it complains it can't create?

[inn...@googlecode.com]

Comment #5 on issue 31 by ccurvey: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

Yep, I can create the files. It may be time for me to dust off my perl
skillz and see if I can duplicate the error in a
simpler program.

[inn...@googlecode.com]

Comment #6 on issue 31 by baron.schwartz: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

Sure. Can you check whether backslashes -vs- forwardslashes in the
filename makes a
difference?

[inn...@googlecode.com]

Comment #7 on issue 31 by ccurvey: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

Ah, found it. It looks like we can't copy the tempfile on windows 7
because it is
still open. So by fixing the code around line 7557, this works.

use File::Copy;
copy($tmpfile, $conf_file) or die("Can't rename $tmpfile to $conf_file:
$OS_ERROR");
#rename($tmpfile, $conf_file) or die("Can't rename $tmpfile to
$conf_file: $O

[inn...@googlecode.com]

Comment #8 on issue 31 by baron.schwartz: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

Is this problem still there in the latest code from SVN? I don't think it
is -- the file is closed before renaming.

[inn...@googlecode.com]

Updates:
Status: NotReproducible

Comment #9 on issue 31 by baron.schwartz: location of configuration file on
windows?
http://code.google.com/p/innotop/issues/detail?id=31

(No comment was entered for this change.)

[inn...@googlecode.com]

Comment #10 on issue 31 by sno...@emmis.com: location of configuration file
on windows?
https://code.google.com/p/innotop/issues/detail?id=31

I am still seeing this issue in 1.9.1, it's now at line 8367, and I had to
make the change suggested here. $file is closed before the rename, but the
rename refers to $tmpfile and $conf_file.

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings