This is an archive of the discontinued LLVM Phabricator instance.

Differential D78074

Set the access permissions explicitly for published docs.
ClosedPublic

Authored by andreil99 on Apr 13 2020, 7:26 PM.

Details

Reviewers
gkistanova
Commits
rZORGae0085c0bc60: Set the access permissions explicitly for published docs. When the bot…
Summary

When the bot published the docs, the result should not depend on how permissions are set on the bot, but rather shall be set explicitly.

Diff Detail

Event Timeline

andreil99 created this revision.Apr 13 2020, 7:26 PM
gkistanova accepted this revision.Apr 13 2020, 10:15 PM

LGTM

This revision is now accepted and ready to land.Apr 13 2020, 10:15 PM
Closed by commit rZORGae0085c0bc60: Set the access permissions explicitly for published docs. When the bot… (authored by andreil99). · Explain WhyApr 13 2020, 10:47 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
zorg/
buildbot/
builders/
1 line

Diff 257202

zorg/buildbot/builders/SphinxDocsBuilder.py

Show First 20 Lines • Show All 229 Lines • ▼ Show 20 Lines for target, local_path, remote_path in docs:
"%s" % target, "%s" % target,
], ],
command=[ command=[
'rsync', 'rsync',
'-vrl', '-vrl',
'--delete', '--force', '--delay-updates', '--delete-delay', '--delete', '--force', '--delay-updates', '--delete-delay',
'--ignore-times', '--ignore-times',
'--checksum', '--checksum',
'-p', '--chmod=Du=rwx,Dg=rwx,Do=rx,Fu=rw,Fg=rw,Fo=r',
"%s" % local_path, "%s" % local_path,
"lists.llvm.org:web/%s" % remote_path, "lists.llvm.org:web/%s" % remote_path,
], ],
env=merged_env, env=merged_env,
) )
) )
return f return f