Code review comment for lp:~ack/landscape-client/sources.list-preserve-old-permissions

Revision history for this message
Thomas Herve (therve) wrote :

This is the wrong approach. The package reporter should be run as the landscape user instead, not run. If you look at the package changer, it already does that before calling the reporter. Instead of doing it in the package changer, this code should be moved in the package reporter, and then the problem will be fixed for the aptsources plugin.

review: Needs Fixing

« Back to merge proposal