Opened 13 years ago

Closed 13 years ago

#344 closed task (wontfix)

`svndigest --force` deleted protected files

Reported by: Peter Johansson Owned by: Jari Häkkinen
Priority: minor Milestone:
Component: core Version: 0.6.5
Keywords: Cc:

Description

There is a chmod in rmdirhier enabling deletion of files (non-dirs) that are write-protected.

// Make sure file is removable before removing it
chmod(entry->d_name,S_IWRITE);

Given the comment in the code, I guess there is some thought behind this, but I think it is bad behavior. I think svndigest should respect the write permission of the file. Moreover, I notice that we are not consistent because we do not chmod directories.

Change History (1)

comment:1 Changed 13 years ago by Peter Johansson

Milestone: svndigest 0.6.6
Resolution: wontfix
Status: newclosed

rm --force behaves in the same way, so I changed my mind here.

Note: See TracTickets for help on using tickets.