Daniel Burrows <[EMAIL PROTECTED]> writes: > Yes, that's true, and as far as I'm concerned, making that the default would > be enough to close the bug. The main thing that bothered me is that you can > lose a lot of data with no warning just using the default behavior of the > program the way it's intended.
Um, you weren't using the program the way it's intended. It is not intended that you open backup files and start operating on them as regular files; it's intended that you copy them to some other name and use that. Does this intention suck? Yes. Is it not even documented? Yes. > I think that you're right, though. Tightening the backup-purging > function is enough to eliminate the most pathological parts of this > bug: right now it'll match <base>.<date>* (if I follow the code), > but it should only match <base>.<date>.xac. Probably just checking > that strcmp(res, "xac")==0 will do it... Well, ".xac" (as your patch indicates) actually, which is the fix I have already uploaded as gnucash-1.8.10-3. Thomas -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
