Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: releaseAutomation.py

Issue 29869563: Issue 6906 - pack source archives with the correct version (Closed)
Patch Set: Created Aug. 30, 2018, 1:10 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« ensure_dependencies.py ('K') | « ensure_dependencies.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: releaseAutomation.py
diff --git a/releaseAutomation.py b/releaseAutomation.py
index b346d5e1e4311f9fbfb14fbe5b5bcfaf8758a2b9..c7eaf6b77d8ed16b45ae5906d0a45b6daefa46dd 100644
--- a/releaseAutomation.py
+++ b/releaseAutomation.py
@@ -279,14 +279,15 @@ def run(baseDir, platforms, version, keyFile, downloads_repo):
downloads_repo,
'adblockplus-{}-source.tgz'.format(release_identifier),
)
- create_sourcearchive(baseDir, archive_path)
- downloads.append(archive_path)
try:
release_commit(baseDir, extension_name, version, target_platforms)
except subprocess.CalledProcessError as e:
if not (re_release and 'nothing changed' in e.output):
raise
+ create_sourcearchive(baseDir, archive_path)
+ downloads.append(archive_path)
+
release_tag(baseDir, release_identifier, extension_name)
# Now add the downloads and commit
« ensure_dependencies.py ('K') | « ensure_dependencies.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld