Merge lp:~jontai/openvista-gtm-integration/bug375029 into lp:openvista-gtm-integration

Proposed by Jon Tai
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jontai/openvista-gtm-integration/bug375029
Merge into: lp:openvista-gtm-integration
Diff against target: None lines
To merge this branch: bzr merge lp:~jontai/openvista-gtm-integration/bug375029
Reviewer Review Type Date Requested Status
jeff.apple Approve
Review via email: mp+6497@code.launchpad.net
To post a comment you must log in.
Revision history for this message
jeff.apple (jeff-apple) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'scripts/usr/bin/ovimport'
2--- scripts/usr/bin/ovimport 2009-04-29 03:15:31 +0000
3+++ scripts/usr/bin/ovimport 2009-05-12 23:02:05 +0000
4@@ -219,6 +219,14 @@
5 umask 007
6 rsync -rl --include '*/' --include '*.m' --exclude '*' "$real_routines/" "$root/$instance/routines/"
7 ) || exit_with_error "ovimport" "Unable to import routines"
8+
9+ # the umask is not enough to guarantee that the routines are user
10+ # and group readable and writable; if the source routines are not
11+ # group writable, the destination routines won't be, either.
12+ #
13+ # if we're not running as root, we won't be able to set the
14+ # permissions for files we don't own, so just do our best
15+ chmod -R u+rw,g+rw "$root/$instance/routines/" 2> /dev/null
16 ;;
17 esac
18

Subscribers

People subscribed via source and target branches