Bugs item #1645851, was opened at 2007-01-27 08:51
Message generated for change (Comment added) made by cstrauss
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=102435&aid=1645851&group_id=2435
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: msys
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Cesar Strauss (cstrauss)
Assigned to: Cesar Strauss (cstrauss)
Summary: rm -r ../testdir fails to remove testdir
Initial Comment:
The following recursive rm invocation fails to remove the top-level directory. The failure appears to be linked somehow with a '..' in the directory name.
$ mkdir test
$ cd test
$ mkdir ../testdir
$ touch ../testdir/testfile
$ du -a ..
0 ../test
0 ../testdir/testfile
0 ../testdir
0 ..
$ rm -r ../testdir
rm: cannot remove directory `../testdir': No such file or directory
$ du -a ..
0 ../test
0 ../testdir
0 ..
$ rm --version
rm (GNU coreutils) 5.97
$ /bin/ls -l /bin/rm
-rwxr-xr-x 1 cstrauss Administradores 37888 Jan 20 00:27 /bin/rm
System info:
Windows XP Home SP 2
MinGW installer: manual (tar -zxf)
$ uname -a
MINGW32_NT-5.1 CESAR 1.0.11(0.46/3/2) 2007-01-12 12:05 i686 Msys
MinGW runtime version: 3.11
w32api version: 3.8
----------------------------------------------------------------------
>Comment By: Cesar Strauss (cstrauss)
Date: 2007-01-27 18:53
Message:
Logged In: YES
user_id=1369729
Originator: YES
Hello, Earnie
You are right, the issues are related. The attached patch solves both
cases. I don't think your fix addressed the core of the matter, however, so
I also reverted it in my patch.
I suspect the MSYS version of fchdir may not be working correctly,
instead. It is used for changing to the original working directory, as the
final directory is removed. The fchdir call is in lib/save-cwd.c
(restore_cwd).
There was already a fallback using file names instead of descriptors, so I
used that.
The next thing to do would be making a test case against fchdir and
possibly filling a bug report. However, I will travel on vacation, starting
tomorrow, so expect a one week delay.
Cesar
File Added: rm-r.patch
----------------------------------------------------------------------
Comment By: Earnie Boyd (earnie)
Date: 2007-01-27 12:38
Message:
Logged In: YES
user_id=15438
Originator: NO
Cesar will you take a look please? The change I made before was to ensure
that the directory level had changed back to the starting point before
trying to remove the last directory entry. The recurse points didn't back
up to remove the last directory before my change so the directory was still
open and being windows it can't remove or rename an open directory. You
should be able to determine the diff from the CVS. TIA, Earnie.
----------------------------------------------------------------------
Comment By: Earnie Boyd (earnie)
Date: 2007-01-27 12:17
Message:
Logged In: YES
user_id=15438
Originator: NO
Sorry, this is different. The difference is the .. directory but is
probably related to the other issue.
----------------------------------------------------------------------
Comment By: Earnie Boyd (earnie)
Date: 2007-01-27 12:16
Message:
Logged In: YES
user_id=15438
Originator: NO
I thought I fixed that all ready. There is a closed ticket for this same
error.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=102435&aid=1645851&group_id=2435

Patches item #1443826, was opened at 2006-03-06 02:31
Message generated for change (Comment added) made by someone-guy
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=302435&aid=1443826&group_id=2435
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: runtime
Group: IINR - Include In Next Release
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Gianluigi Tiesi (sherpya)
Assigned to: Nobody/Anonymous (nobody)
Summary: lseek _FILE_OFFSET_BITS=64
Initial Comment:
currently mingw runtime ignores at all
_FILE_OFFSET_BITS directive, this sime patch handle
_FILE_OFFSET_BITS in lseek, the patch can be extended
also for other functions.
if _FILE_OFFSET_BITS=64 off_t will be 8bytes and lseek
will be replaced by _lseeki64.
sizeof results are compared to a ia32 linux
----------------------------------------------------------------------
Comment By: Reimar Döffinger (someone-guy)
Date: 2007-01-27 16:06
Message:
Logged In: YES
user_id=862448
Originator: NO
Any problem with it? Who can be bugged to get this included?
----------------------------------------------------------------------
Comment By: Gianluigi Tiesi (sherpya)
Date: 2006-03-18 02:33
Message:
Logged In: YES
user_id=38409
updated patch, this adds also stat function/struct
http://oss.netfarm.it/mplayer/misc/file64_mingw.diff
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=302435&aid=1443826&group_id=2435

Community

Help

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

I agree to receive quotes, newsletters and other information from sourceforge.net and its partners regarding IT services and products. I understand that I can withdraw my consent at any time. Please refer to our Privacy Policy or Contact Us for more details