Code review comment for lp:~didrocks/autopilot/remove-circular-dep

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Indeed this is in the upstream code, but I think this would be silly to start forking our own upstream packaging and I think upstream should be a little bit more trustful on people making packaging for years. Also, they should request if they need any help on the packaging, especially for transition (that would have avoided all the Replaces: mismatch).

We shouldn't have to use mail, just merge request. However, do you remember how long some trivial branches sit idling last cycle before getting reviewed after a ping? I thought that for a trivial change only involving packaging changes, upstream can just take packagers into consideration and trust them to do the right thing… (or upstream need to be quite reactive on their MP).

We care about build-deps as well as deps, especially as autopilot is used as a release pre-requisite right now. Conceptually python-autopilot is a library for autopilot as it can never work without -desktop or -touch. So let me reiterate:
- it's either that the breakage isn't good because those 2 packages
- or, as we did on other packages, in both Debian and Ubuntu, downgrade as a recommends, as we install recommends by default.

Mind reconsidering this one and moving on?

« Back to merge proposal