Merge lp:~nuclearbob/utah/packaging-permissions into lp:utah

[Max Brustkern]

When we merged this before, a chmod on $DIR/* would fail if the directory was empty. Since we were recursing anyway, I just do $DIR instead of $DIR/* now.

[Andy Doan (doanac)]

review: Approve

[Javier Collado (javier.collado)]

The code looks good to me.

The only thing that I want to mention is that, since this is a fix over a
previous merge, I don't think a description should be added to the changelog
which I believe should be brief to keep it useful.

[lp:~nuclearbob/utah/packaging-permissions]

917. By Max Brustkern on 2013-05-21

Removed noisy changelog entry

[Max Brustkern (nuclearbob)]

I think Javier's comment regarding the changelog entry makes sense, so here's an update with that. If we don't have any further ideas, I can merge this.

review: Needs Resubmitting

[Javier Collado (javier.collado)]

review: Approve

=== modified file 'debian/utah-client.postinst'
--- debian/utah-client.postinst	2013-05-16 12:47:10 +0000
+++ debian/utah-client.postinst	2013-05-21 13:03:29 +0000
@@ -29,7 +29,7 @@
         then
           chown -R $SERVER_USER:$SERVER_GROUP $DIR
           chmod 2775 $DIR
-          chmod -R ug+rw $DIR/*
+          chmod -R ug+rw $DIR
         fi
       done
     }

=== modified file 'debian/utah.postinst'
--- debian/utah.postinst	2013-05-16 18:22:03 +0000
+++ debian/utah.postinst	2013-05-21 13:03:29 +0000
@@ -73,7 +73,7 @@
         then
           chown -R $SERVER_USER:$SERVER_GROUP $DIR
           chmod 2775 $DIR
-          chmod -R ug+rw $DIR/*
+          chmod -R ug+rw $DIR
         fi
       done
     }