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

Unified Diff: adblockplus/build.py

Issue 29626577: Issue 6108 - No filter list is selected after a migration failure (Closed)
Patch Set: Small adjustments Created Dec. 5, 2017, 3:51 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
« no previous file with comments | « adblockplus/Api.jsm ('k') | adblockplus/issue-6108.patch » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: adblockplus/build.py
===================================================================
--- a/adblockplus/build.py
+++ b/adblockplus/build.py
@@ -4,16 +4,17 @@
import os
import shutil
import subprocess
import sys
import tempfile
_BASE_DIR = os.path.dirname(os.path.abspath(__file__))
_ABP_DIR = "adblockplus"
+_ABP_CORE_DIR = "adblockpluscore"
def _build_abp(base_dir, xpi_path):
xpi_dir = os.path.dirname(os.path.abspath(xpi_path))
if not os.path.exists(xpi_dir):
os.makedirs(xpi_dir)
abp_dir = os.path.join(base_dir, _ABP_DIR)
@@ -26,20 +27,23 @@ def _build_abp(base_dir, xpi_path):
buildtools.packager.getMetadataPath = get_metadata_path
buildtools.build.processArgs(abp_dir, ["", "build", xpi_path])
def _patch_abp(patched_dir):
shutil.copytree(_BASE_DIR, patched_dir)
abp_dir = os.path.join(patched_dir, _ABP_DIR)
- subprocess.check_call(["hg", "import", "-q", "-R",
- os.path.join(abp_dir, "adblockpluscore"),
+ abp_core_dir = os.path.join(abp_dir, _ABP_CORE_DIR)
+ subprocess.check_call(["hg", "import", "-q", "-R", abp_core_dir,
os.path.join(patched_dir, "issue-6070.patch")])
+ subprocess.check_call(["hg", "import", "-q", "-R", abp_core_dir,
+ os.path.join(patched_dir, "issue-6108.patch")])
+
if __name__ == "__main__":
if len(sys.argv) < 2:
error_message = "Usage: %s XPI_PATH" % os.path.basename(sys.argv[0])
print >>sys.stderr, error_message
sys.exit(1)
xpi_path = sys.argv[1]
patched_dir = tempfile.NamedTemporaryFile().name
« no previous file with comments | « adblockplus/Api.jsm ('k') | adblockplus/issue-6108.patch » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld