[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: install.c: please set unlink_dest_before_opening=false
From: |
Robert Millan [ackstorm] |
Subject: |
Re: install.c: please set unlink_dest_before_opening=false |
Date: |
Fri, 23 Feb 2007 10:42:38 +0100 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
On Thu, Feb 22, 2007 at 12:42:12PM -0800, Paul Eggert wrote:
> "Robert Millan [ackstorm]" <address@hidden> writes:
>
> > How about attempting to write it, and if that fails then unlink ?
> > (preferably
> > checking the errno code for this condition)
>
> That doesn't sound right; it'd mean the result of installing (say)
> /bin/bash would differ depending on whether someone currently happened
> to be executing Bash. I think I'd rather have the unlink specified by
> a command-line option of some sort. (We can argue about what the
> default should be....).
Now that I think of it, overwriting the same inode isn't good either, since
we can't garantee that the change will be atomical (and datafiles currently
in used can have an intermediate "corrupted" state).
Can we have it remove the link, and then reset all other links from the old
inode to the new one, or is that completely insane?
--
Robert Millan
ACK STORM, S.L. - http://www.ackstorm.es/
- install.c: please set unlink_dest_before_opening=false, Robert Millan [ackstorm], 2007/02/20
- Re: install.c: please set unlink_dest_before_opening=false, Paul Eggert, 2007/02/20
- Re: install.c: please set unlink_dest_before_opening=false, Robert Millan [ackstorm], 2007/02/22
- Re: install.c: please set unlink_dest_before_opening=false, Paul Eggert, 2007/02/21
- Re: install.c: please set unlink_dest_before_opening=false, Robert Millan [ackstorm], 2007/02/22
- Re: install.c: please set unlink_dest_before_opening=false, Paul Eggert, 2007/02/22
- Re: install.c: please set unlink_dest_before_opening=false,
Robert Millan [ackstorm] <=
- Re: install.c: please set unlink_dest_before_opening=false, Andreas Schwab, 2007/02/23
- Re: install.c: please set unlink_dest_before_opening=false, Robert Millan [ackstorm], 2007/02/23
Re: install.c: please set unlink_dest_before_opening=false, James Youngman, 2007/02/25