PR#1805: backend: do not rsync-copy permissions from builder - copr/copr

copr / copr

#1805 backend: do not rsync-copy permissions from builder

Closed 2 years ago by praiskup. Opened 2 years ago by praiskup.

Unknown source fix-rsync-permissions into main

backend: do not rsync-copy permissions from builder

Pavel Raiskup • 2 years ago

450a102

backend/copr_backend/sshcmd.py

file modified

+1 -1

		`@@ -197,7 +197,7 @@`
		`log_filepath = "/dev/null"`
		`if logfile:`
		`log_filepath = os.path.join(dest, logfile)`
		`- command = "/usr/bin/rsync -rlptDvH -e '{}' {} {}/ &> {}".format(`
		`+ command = "/usr/bin/rsync -rltDvH -e '{}' {} {}/ &> {}".format(`
		`ssh_opts, full_source_path, dest, log_filepath)`

		`try:`

praiskup commented 2 years ago

Some files that mock copies-out from the chroot for us contained some
weird sticky bits and permissions. It's good idea to keep the rsync's
default that respects just umask.

Relates: #1804