Issue 6264 - Can't prepare libadblockplus dependencies

third_party/libadblockplus/download_ndk.py

 import os
 import urllib
 import zipfile
 import sys
+import shutil
+
 
 def main(argv):
-  # Download
-  ndk_src = 'https://dl.google.com/android/repository/android-ndk-r12b-linux-x86_64.zip'
+    # Download
+    ndk_src = 'https://dl.google.com/android/repository/android-ndk-r12b-linux-x86_64.zip'
 
-  cwd = os.getcwd()
-  libadblockplus_root = os.path.join(cwd, 'src', 'third_party', 'libadblockplus', 'third_party')
-  ndk_dst = os.path.join(libadblockplus_root, 'android-ndk-r12b-linux-x86_64.zip')
+    cwd = os.getcwd()
+    libadblockplus_root = os.path.join(cwd,

[sergei, 2018/01/18 09:16:51]

I think it's rather libadblockplus_third_party_dir than libadblockplus_root. If
you decide to rename the variable I would propose to `third_party_dir`.

[anton, 2018/01/18 12:20:26]

Done. See patch set #6

+                                       'src', 'third_party',
+                                       'libadblockplus', 'third_party')

[sergei, 2018/01/18 09:16:50]

Why not to put it into cwd/src/third_party?

[anton, 2018/01/18 09:49:12]

it's required for libadblockplus (and libadblockplus-android) first of all and
no other deps require it.
also there is third_party/android_tools/ndk dir so i'd like to keep it separate

+    ndk_dst = os.path.join(libadblockplus_root,
+                           'android-ndk-r12b-linux-x86_64.zip')
 
-  if os.path.exists(ndk_dst):
+    if os.path.exists(ndk_dst):
+        os.remove(ndk_dst)
+
+    print('Downloading {} to {}'.format(ndk_src, ndk_dst))
+    urllib.urlretrieve(ndk_src, ndk_dst)
+
+    # Delete existing NDK directory
+    ndk_dir = os.path.join(libadblockplus_root, 'android-ndk-r12b')

[anton, 2018/01/18 08:25:45]

Previously we deleted whole 'third_party' including ndk dir. Now we have to
delete it before extracting of zip. We could delete it with 'delete_dir.py' but
i'd prefer to keep the logics in one file here.

+    if os.path.exists(ndk_dir):
+        print('Deleting {}'.format(ndk_dir))
+        shutil.rmtree(ndk_dir)
+
+    # Extract zip (preserving file permissions)
+    print('Extracting {} to {}'.format(ndk_dst, libadblockplus_root))
+    with zipfile.ZipFile(ndk_dst, 'r') as zf:

[sergei, 2018/01/18 09:16:50]

Taking into account the number of files I would probably just call a subprocess
unzip for the sake of performance. BTW, is there a doc for info.external_attr?
I'm not sure that merely shifting is enough, perhaps we should also clear few
bits.

[anton, 2018/01/18 09:49:12]

zip python library does not keep permissions ("executable"), so i had to use
this workaround.

+        for info in zf.infolist():
+            zf.extract(info.filename, path=libadblockplus_root)
+            out_path = os.path.join(libadblockplus_root, info.filename)
+
+            perm = info.external_attr >> 16L
+            os.chmod(out_path, perm)
+
+    # Delete zip
     os.remove(ndk_dst)
 
-  print('Downloading %s to %s' % (ndk_src, ndk_dst))
-  urllib.urlretrieve(ndk_src, ndk_dst)
+    return 0
 
-  # Extract zip (preserving file permissions)
-  print('Extracting %s to %s' % (ndk_dst, libadblockplus_root))
-  with zipfile.ZipFile(ndk_dst, 'r') as zf:
-    for info in zf.infolist():
-      zf.extract(info.filename, path=libadblockplus_root)
-      out_path = os.path.join(libadblockplus_root, info.filename)
-
-      perm = info.external_attr >> 16L
-      os.chmod(out_path, perm)
-
-  # Delete zip
-  os.remove(ndk_dst)
-
-  return 0
 
 if '__main__' == __name__:
-  try:
-    sys.exit(main(sys.argv[1:]))
-  except KeyboardInterrupt:
-    sys.stderr.write('interrupted\n')
-    sys.exit(1)
+    try:
+        sys.exit(main(sys.argv[1:]))
+    except KeyboardInterrupt:
+        sys.stderr.write('interrupted\n')
+        sys.exit(1)