[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Problems found by piuparts

On 20 Feb 2006, Lars Wirzenius stated:

> I added a Cc to Manoj since I would like to hear his
> comment. Whoever responds may want to remove the Cc to avoid
> stuffing his inbox unnecessarily.
> su, 2006-02-19 kello 23:42 -0800, Steve Langasek kirjoitti:
>> On Mon, Feb 20, 2006 at 08:24:53AM +0200, Lars Wirzenius wrote:
>>> * Use of ucf in postrm during a purge, without checking that ucf
>>> is installed. Since ucf is not an essential package, postrm
>>> during purge cannot rely on it. As it happens, I think it might
>>> be good to have ucf (or a subset of it) as an essential package,
>>> since this error happens a lot.
>> So assuming that we're stuck with the current implementation for a
>> bit where ucf is not part of essential, I do wonder if checking
>> whether ucf is installed is actually the correct thing to do in
>> postrm purge.  The state prior to purge is defined as "config
>> files"; the difference between config files state and "purged" is
>> whether there are still config files left on the system.  If the
>> package can't actually succeed in removing its config files because
>> ucf is not installed, isn't it *correct* for the postrm purge
>> command to fail?  I.e., I think it's more of a bug to allow dpkg to
>> put the package into "purged" state leaving orphaned config files
>> behind than it is for postrm purge to fail and leave the package in
>> "config files" state.
> Hm. I don't use ucf on my own packages (yet), so my understanding is
> a bit hazy, but if I have understood correctly, the actual config
> file is removed with rm anyway, and ucf is needed on purge only to
> remove the config file also from ucf's history data. Thus, only
> running ucf if it's there should be the right thing.
> Manoj, could you comment on this?

        Your analysis is correct.


"I'd love to go out with you, but I did my own thing and now I've got
to undo it."
Manoj Srivastava   <srivasta@debian.org>  <http://www.debian.org/%7Esrivasta/>
1024D/BF24424C print 4966 F272 D093 B493 410B  924B 21BA DABB BF24 424C

Reply to: