digitalmars.D.learn - remove file access denied
- Josphe Brigmo (14/14) Sep 13 2018 I am trying to remove a file
- Norm (7/21) Sep 13 2018 Do you have the file open when you call remove? If so close the
- Josphe Brigmo (16/44) Sep 14 2018 No, I use read, there is no file handles. Pointless to post code
- bauss (12/59) Sep 14 2018 It's not necessarily a permission issue, but might be an issue
- Alex (19/35) Sep 14 2018 ´´´
- Adam D. Ruppe (4/5) Sep 14 2018 The source code for remove is DeleteFile(name), so not much room
- Josphe Brigmo (7/13) Sep 14 2018 It happens on a bunch. I do get errors or overlong file names but
- H. S. Teoh (8/16) Sep 14 2018 It would really help if you post a stripped-down version of the code
- Josphe Brigmo (21/35) Sep 14 2018 It woudln't help. I'm dealing with over a million files and you'd
- bauss (5/27) Sep 14 2018 Went ahead and did the following in case anyone comes across your
- Josphe Brigmo (9/42) Sep 15 2018 Thanks. The problem ultimately is MAX_PATH.
I am trying to remove a file remove(filename); and I get an access denied! I can remove it from explorer just fine. I am able to remove other files but there should be no reason why the file can't be removed in this case. All I am doing to mess with the file is reading it's contents right before to do a file compare(I am removing the file if it is a duplicate). Does read() lock the file at all? (maybe the lock is persisting just long enough to make the remove fail? Since I can delete the file outside the program and since the filename is valid(I copied and pasted it to remove it to check), This seems like a D problem.
Sep 13 2018
On Thursday, 13 September 2018 at 23:25:24 UTC, Josphe Brigmo wrote:I am trying to remove a file remove(filename); and I get an access denied! I can remove it from explorer just fine. I am able to remove other files but there should be no reason why the file can't be removed in this case. All I am doing to mess with the file is reading it's contents right before to do a file compare(I am removing the file if it is a duplicate). Does read() lock the file at all? (maybe the lock is persisting just long enough to make the remove fail? Since I can delete the file outside the program and since the filename is valid(I copied and pasted it to remove it to check), This seems like a D problem.Do you have the file open when you call remove? If so close the file handle before the remove call. If you can post a stripped down version of your code it would also help. bye, Norm
Sep 13 2018
On Friday, 14 September 2018 at 04:48:09 UTC, Norm wrote:On Thursday, 13 September 2018 at 23:25:24 UTC, Josphe Brigmo wrote:No, I use read, there is no file handles. Pointless to post code because it won't offer much. Also, I have security privileges. I simply read the file to compare it's contents then I try to remove the file if it had the same contents and it says it is invalid. I also, of course, check that it exist and is a file. This is all I'm doing that is related to file reading and I cannot remove the file(but can read it and such). So, I'm really wondering if read locks the file but doesn't release it in time. Using lockHunder shows the file isn't locked but the directory is(Probably because I'm iterating through it. Seems it is an error with remove, using executeShell works fine: auto ls = executeShell(`del /F /Q "`~fn~`"`); which does not give an error but remove(fn) does. Seems remove is broke.I am trying to remove a file remove(filename); and I get an access denied! I can remove it from explorer just fine. I am able to remove other files but there should be no reason why the file can't be removed in this case. All I am doing to mess with the file is reading it's contents right before to do a file compare(I am removing the file if it is a duplicate). Does read() lock the file at all? (maybe the lock is persisting just long enough to make the remove fail? Since I can delete the file outside the program and since the filename is valid(I copied and pasted it to remove it to check), This seems like a D problem.Do you have the file open when you call remove? If so close the file handle before the remove call. If you can post a stripped down version of your code it would also help. bye, Norm
Sep 14 2018
bauss <jj_1337 live.dk> On Friday, 14 September 2018 at 08:32:48 UTC, Josphe Brigmo wrote:On Friday, 14 September 2018 at 04:48:09 UTC, Norm wrote:It's not necessarily a permission issue, but might be an issue with the files attributes. If you're on Windows try this: setAttributes(filename, 0x80); 0x80 means normal. You can always try to retrieve the attributes using getAttributes(filename) and see what you retrieve and if there are anything odd going on. D really needs an enum of the file attributes on windows and the examples are poor, because they only show examples related to Posix lol.On Thursday, 13 September 2018 at 23:25:24 UTC, Josphe Brigmo wrote:No, I use read, there is no file handles. Pointless to post code because it won't offer much. Also, I have security privileges. I simply read the file to compare it's contents then I try to remove the file if it had the same contents and it says it is invalid. I also, of course, check that it exist and is a file. This is all I'm doing that is related to file reading and I cannot remove the file(but can read it and such). So, I'm really wondering if read locks the file but doesn't release it in time. Using lockHunder shows the file isn't locked but the directory is(Probably because I'm iterating through it. Seems it is an error with remove, using executeShell works fine: auto ls = executeShell(`del /F /Q "`~fn~`"`); which does not give an error but remove(fn) does. Seems remove is broke.I am trying to remove a file remove(filename); and I get an access denied! I can remove it from explorer just fine. I am able to remove other files but there should be no reason why the file can't be removed in this case. All I am doing to mess with the file is reading it's contents right before to do a file compare(I am removing the file if it is a duplicate). Does read() lock the file at all? (maybe the lock is persisting just long enough to make the remove fail? Since I can delete the file outside the program and since the filename is valid(I copied and pasted it to remove it to check), This seems like a D problem.Do you have the file open when you call remove? If so close the file handle before the remove call. If you can post a stripped down version of your code it would also help. bye, Norm
Sep 14 2018
On Friday, 14 September 2018 at 08:32:48 UTC, Josphe Brigmo wrote:No, I use read, there is no file handles. Pointless to post code because it won't offer much. Also, I have security privileges. I simply read the file to compare it's contents then I try to remove the file if it had the same contents and it says it is invalid. I also, of course, check that it exist and is a file. This is all I'm doing that is related to file reading and I cannot remove the file(but can read it and such). So, I'm really wondering if read locks the file but doesn't release it in time. Using lockHunder shows the file isn't locked but the directory is(Probably because I'm iterating through it. Seems it is an error with remove, using executeShell works fine: auto ls = executeShell(`del /F /Q "`~fn~`"`); which does not give an error but remove(fn) does. Seems remove is broke.´´´ void main() { import std.file : exists, read, remove; import std.stdio : File; import std.uuid : randomUUID; import std.file : tempDir; import std.path : dirSeparator; string tFileName = tempDir ~ dirSeparator ~ randomUUID.toString; File(tFileName, "w").close; assert(tFileName.exists); assert(tFileName.read.length == 0); tFileName.remove; assert(!tFileName.exists); } ´´´ This works for me. And what does not for you?
Sep 14 2018
On Friday, 14 September 2018 at 08:32:48 UTC, Josphe Brigmo wrote:Seems remove is broke.The source code for remove is DeleteFile(name), so not much room for bugs there, except maybe string conversion. What is the filename you are working on?
Sep 14 2018
On Friday, 14 September 2018 at 13:28:41 UTC, Adam D. Ruppe wrote:On Friday, 14 September 2018 at 08:32:48 UTC, Josphe Brigmo wrote:It happens on a bunch. I do get errors or overlong file names but this doesn't seem to be the case. The fact is, that simply using execute shell using the same file name works. So this is a D problem. It happens quite often and every time I can delete the files in file explorer.Seems remove is broke.The source code for remove is DeleteFile(name), so not much room for bugs there, except maybe string conversion. What is the filename you are working on?
Sep 14 2018
On Fri, Sep 14, 2018 at 02:36:34PM +0000, Josphe Brigmo via Digitalmars-d-learn wrote: [...]It happens on a bunch. I do get errors or overlong file names but this doesn't seem to be the case. The fact is, that simply using execute shell using the same file name works. So this is a D problem. It happens quite often and every time I can delete the files in file explorer.It would really help if you post a stripped-down version of the code that exhibits the same problem. Otherwise we're just guessing what the real problem is. T -- It only takes one twig to burn down a forest.
Sep 14 2018
On Friday, 14 September 2018 at 15:21:21 UTC, H. S. Teoh wrote:On Fri, Sep 14, 2018 at 02:36:34PM +0000, Josphe Brigmo via Digitalmars-d-learn wrote: [...]It woudln't help. I'm dealing with over a million files and you'd need those files too. But basically all I have done is created a new rename function: void removeFile(string fn) { if (!isDir(fn)) { // remove(fn) setAttributes(fn, 0x80); auto ls = executeShell(`del /F /Q "`~fn~`"`); if (ls.status != 0) throw new Exception("Cannot delete file `"~fn~"`!"); } } And this works and functions appropriately. The other code is basically just recursively going through the directory as standard practice using dirEntries and deleting certain files(it's a little more complex since there is some logic on the file name, but nothing touches the file except delete).It happens on a bunch. I do get errors or overlong file names but this doesn't seem to be the case. The fact is, that simply using execute shell using the same file name works. So this is a D problem. It happens quite often and every time I can delete the files in file explorer.It would really help if you post a stripped-down version of the code that exhibits the same problem. Otherwise we're just guessing what the real problem is. T
Sep 14 2018
On Friday, 14 September 2018 at 16:55:21 UTC, Josphe Brigmo wrote:On Friday, 14 September 2018 at 15:21:21 UTC, H. S. Teoh wrote:Went ahead and did the following in case anyone comes across your issue in the future: https://github.com/dlang/phobos/pull/6707 That way the documentation will at least be clear about it.[...]It woudln't help. I'm dealing with over a million files and you'd need those files too. But basically all I have done is created a new rename function: void removeFile(string fn) { if (!isDir(fn)) { // remove(fn) setAttributes(fn, 0x80); auto ls = executeShell(`del /F /Q "`~fn~`"`); if (ls.status != 0) throw new Exception("Cannot delete file `"~fn~"`!"); } } And this works and functions appropriately. The other code is basically just recursively going through the directory as standard practice using dirEntries and deleting certain files(it's a little more complex since there is some logic on the file name, but nothing touches the file except delete).
Sep 14 2018
On Saturday, 15 September 2018 at 06:13:29 UTC, bauss wrote:On Friday, 14 September 2018 at 16:55:21 UTC, Josphe Brigmo wrote:Thanks. The problem ultimately is MAX_PATH. Seems that phobo's does not detect windows 10 nor use unicode for such things as it should which would not break simple code(one expects file routines to be well used and the bugs fixed). Seems not to many people use D for windows with long paths and files and hence D for windows. The fix is relatively simple ("prepend \\?\ or use the wide functions).On Friday, 14 September 2018 at 15:21:21 UTC, H. S. Teoh wrote:Went ahead and did the following in case anyone comes across your issue in the future: https://github.com/dlang/phobos/pull/6707 That way the documentation will at least be clear about it.[...]It woudln't help. I'm dealing with over a million files and you'd need those files too. But basically all I have done is created a new rename function: void removeFile(string fn) { if (!isDir(fn)) { // remove(fn) setAttributes(fn, 0x80); auto ls = executeShell(`del /F /Q "`~fn~`"`); if (ls.status != 0) throw new Exception("Cannot delete file `"~fn~"`!"); } } And this works and functions appropriately. The other code is basically just recursively going through the directory as standard practice using dirEntries and deleting certain files(it's a little more complex since there is some logic on the file name, but nothing touches the file except delete).
Sep 15 2018