Code review comment for lp://staging/~fgimenez/autopilot/dev-dependencies

Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

Hi,

A better way to do this is to install the debian build deps for the package. The difference is subtle - your approach will install the runtime dependencies, which is (probably) a subset of the build-time dependencies. For example, we may have modules that we use to run our tests that aren't needed at runtime.

I think this command should be:

sudo mk-build-deps -i

note that in order to run that, you need 'devscripts' and 'equivs' packages installed first.

Also - if python3-autopilot does not install the dateutil module, _and_ it's needed at runtime, then that's another bug that needs to be fixed :D

Cheers,

review: Needs Fixing

« Back to merge proposal