Attention is currently required from: fixeria, msuraev, osmith, pespin.

Patch set 8:Code-Review -1

View Change

2 comments:

• Patchset:

  • Patch Set #8:

    -1: Please add a warning for the user, because you're changing something what the user doesn't expect.
    It would be great if you could add this to the Debian Changelog (not sure how we're generating it).

• File debian/postinst:

  • Patch Set #8, Line 18: # Fix permissions of previous (root-owned) install (OS#4107)

    It would be great to show a warning or something, so the user know those file has been changed.

    Can't you detect if this is an upgrade and from which version you're upgrading from?

To view, visit change 30094. To unsubscribe, or for help writing mail filters, visit settings.