Code review comment for lp://staging/~adamblackburn/wicd/loggroup

Revision history for this message
Dan O'Reilly (oreilldf) wrote :

In wpath.py.in log_group and log_perms are used. But in wicd-daemon.py wpath.loggroup and wpath.logperms are used, which won't work.

The try/except that the chmod/chown happen in should be split into one where the chmod happens and another one where the chown happens, so we can tell what actually failed if we get an exception doing one of them.

« Back to merge proposal