release-engineering/tito
1
0
Fork 0
mirror of https://github.com/rpm-software-management/tito.git synced 2025-02-24 04:32:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Don't upload patches to the lookaside cache

Browse source 
With @xsuchy we discovered that when doing `tito release`, into Fedora
DistGit, then .patch files are being uploaded to the lookaside
cache. That is not expected behavior, we should commit them to git
only.
This commit is contained in:
Jakub Kadlcik 2023-01-04 21:04:58 +01:00 • committed by Jakub Kadlčík
parent b78dc88fe5
commit 5d3df407f9
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/tito/release/distgit.py
View file

@ -327,13 +327,16 @@ class FedoraGitReleaser(Releaser):
Upload any tarballs to the lookaside directory. (if necessary) Upload any tarballs to the lookaside directory. (if necessary)
Uses the "fedpkg new-sources" command. Uses the "fedpkg new-sources" command.
""" """
if not self.builder.sources: sources = [x for x in self.builder.sources
if not x.endswith(self.copy_extensions)]
if not sources:
debug("No sources need to be uploaded.") debug("No sources need to be uploaded.")
return return
print("Uploading sources to lookaside:") print("Uploading sources to lookaside:")
os.chdir(project_checkout) os.chdir(project_checkout)
cmd = '%s new-sources %s' % (self.cli_tool, " ".join(self.builder.sources)) cmd = '%s new-sources %s' % (self.cli_tool, " ".join(sources))
debug(cmd) debug(cmd)
if self.dry_run: if self.dry_run:
@ -342,8 +345,8 @@ class FedoraGitReleaser(Releaser):
output = run_command(cmd) output = run_command(cmd)
debug(output) debug(output)
debug("Removing write-only permission on:") debug("Adding write permission for:")
for filename in self.builder.sources: for filename in sources:
run_command("chmod u+w %s" % filename) run_command("chmod u+w %s" % filename)
def _list_files_to_copy(self): def _list_files_to_copy(self):