Merge lp:~ted/url-dispatcher/cache-permissions into lp:url-dispatcher/14.04

Proposed by Ted Gould
Status: Merged
Merged at revision: 50
Proposed branch: lp:~ted/url-dispatcher/cache-permissions
Merge into: lp:url-dispatcher/14.04
Diff against target: 56 lines (+25/-0)
3 files modified
data/url-dispatcher-refresh.conf.in (+6/-0)
data/url-dispatcher-update.conf.in (+6/-0)
data/url-dispatcher.conf.in (+13/-0)
To merge this branch: bzr merge lp:~ted/url-dispatcher/cache-permissions
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Indicator Applet Developers Pending
Review via email: mp+214067@code.launchpad.net

Commit message

Fix cache directory permissions if they got messed up

Description of the change

Our cache directory is getting weird permissions, not sure why yet, but this is a work around to fix it for most folks so they can live happy normal url dispatched lives.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
51. By Ted Gould

Putting in umasks for now

52. By Ted Gould

Marking this as 'fixed' for url-dispatcher

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/url-dispatcher-refresh.conf.in'
2--- data/url-dispatcher-refresh.conf.in 2014-01-13 19:47:11 +0000
3+++ data/url-dispatcher-refresh.conf.in 2014-04-03 19:16:11 +0000
4@@ -4,6 +4,12 @@
5
6 emits url-dispatcher-update
7
8+# Work around for: https://bugs.launchpad.net/upstart/+bug/1302117
9+
10+umask 0022
11+
12+# end work around
13+
14 pre-start script
15 # If we're starting with url-dispatcher let's let the rest of the
16 # system have a chance to settle.
17
18=== modified file 'data/url-dispatcher-update.conf.in'
19--- data/url-dispatcher-update.conf.in 2014-01-13 22:06:01 +0000
20+++ data/url-dispatcher-update.conf.in 2014-04-03 19:16:11 +0000
21@@ -1,5 +1,11 @@
22 description "URL Dispatcher Directory Watch"
23
24+# Work around for: https://bugs.launchpad.net/upstart/+bug/1302117
25+
26+umask 0022
27+
28+# end work around
29+
30 start on (file FILE=~/.config/url-dispatcher/urls/*.url-dispatcher) or (file FILE=@datadir@/url-dispatcher/urls/*.url-dispatcher) or url-dispatcher-update
31
32 instance $MATCH
33
34=== modified file 'data/url-dispatcher.conf.in'
35--- data/url-dispatcher.conf.in 2013-06-27 22:15:58 +0000
36+++ data/url-dispatcher.conf.in 2014-04-03 19:16:11 +0000
37@@ -6,6 +6,19 @@
38
39 respawn
40
41+# Work around for: https://bugs.launchpad.net/upstart/+bug/1302117
42+
43+umask 0022
44+
45+pre-start script
46+ if [ -d "$HOME/.cache/url-dispatcher" ] && [ ! -x "$HOME/.cache/url-dispatcher" ] ; then
47+ echo "Fixing cache directory permissions"
48+ chmod 700 "$HOME/.cache/url-dispatcher"
49+ fi
50+end script
51+
52+# end work around
53+
54 emits application-start
55
56 exec @pkglibexecdir@/url-dispatcher

Subscribers

People subscribed via source and target branches