Merge lp://staging/~racb/ubuntu/oneiric/samba/862129 into lp://staging/ubuntu/oneiric/samba

Proposed by Robie Basak
Status: Work in progress
Proposed branch: lp://staging/~racb/ubuntu/oneiric/samba/862129
Merge into: lp://staging/ubuntu/oneiric/samba
Diff against target: 52 lines (+23/-4)
2 files modified
debian/changelog (+7/-0)
debian/samba.postrm (+16/-4)
To merge this branch: bzr merge lp://staging/~racb/ubuntu/oneiric/samba/862129
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+78862@code.staging.launchpad.net

Description of the change

This fixes my test case in bug 862129.

To post a comment you must log in.
Revision history for this message
Robie Basak (racb) wrote :

Outstanding questions I posted from that bug that I would like reviewed before merging:

1) Is this the right approach?
2) Is dpkg-query the right way to get the status of the update-inetd package?
3) What about triggers-awaiting and triggers-pending? What should the behaviour be in these cases? Is it OK to treat update-inetd as not available in these cases?

Thanks.

Revision history for this message
Colin Watson (cjwatson) wrote :

The effect of this change is to make the system non-deterministic. If
we want update-inetd to work from postrm scripts, don't we instead need
to make sure that it always works even if unconfigured, as if it were an
Essential package? This would be much safer and the resulting system
would be easier to understand.

I'm also unconvinced about the reliability of the data returned by
dpkg-query from a maintainer script. It may be better now, but
historically this didn't reliably reflect the state of a running dpkg
instance.

To be honest, I think there are too many questions about this kind of
thing for it to be safe to attempt a fix for this for oneiric. I'd
rather we got it right in precise, and then maybe backported the fix.

Revision history for this message
Adam Conrad (adconrad) wrote :

It might be worth pointing out that calling update-inetd in a postrm is vaguely crack anyway, as we're removing the service entries AFTER we've removed the binaries from the disk.

Moving the update-inetd call to prerm (where I'd argue it belongs) solves everything we're trying to solve with this complex branch, since the dependencies are still there during prerm.

Revision history for this message
Robie Basak (racb) wrote :

Thanks Colin and Adam.

I was trying to do the minimal change possible to get the fix into oneiric - as it's causing upgrade failures at the moment. If that's not going to happen, then I'll file a Debian bug as the postrm hasn't been modified by Ubuntu in any way.

Unmerged revisions

148. By Robie Basak

debian/samba.postrm: check that update-inetd is configured and in all code
paths. (LP: #862129)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: