Code review comment for lp:~ahayzen/webbrowser-app/fix-1491279-depend-on-download-manager

Revision history for this message
Olivier Tilloy (osomon) wrote :

Please keep the list of dependencies alphabetically sorted.

Please add those two runtime deps to webapp-container as well. The fact that shared QML files are being installed by the webbrowser-app package is an implementation detail that might not hold true forever, so having the dependencies list explicitly is more future-proof.

review: Needs Fixing

« Back to merge proposal