from argparse import ArgumentParser
import logging
+from . import _
from . import common
from . import metadata
from .exception import BuildException, VCSException
scanignore_worked = set()
scandelete_worked = set()
- def toignore(fd):
+ def toignore(path_in_build_dir):
for k, paths in scanignore.items():
for p in paths:
- if fd.startswith(p):
+ if path_in_build_dir.startswith(p):
scanignore_worked.add(k)
return True
return False
- def todelete(fd):
+ def todelete(path_in_build_dir):
for k, paths in scandelete.items():
for p in paths:
- if fd.startswith(p):
+ if path_in_build_dir.startswith(p):
scandelete_worked.add(k)
return True
return False
- def ignoreproblem(what, fd):
- logging.info('Ignoring %s at %s' % (what, fd))
+ def ignoreproblem(what, path_in_build_dir):
+ logging.info('Ignoring %s at %s' % (what, path_in_build_dir))
return 0
- def removeproblem(what, fd, fp):
- logging.info('Removing %s at %s' % (what, fd))
- os.remove(fp)
+ def removeproblem(what, path_in_build_dir, filepath):
+ logging.info('Removing %s at %s' % (what, path_in_build_dir))
+ os.remove(filepath)
return 0
- def warnproblem(what, fd):
- if toignore(fd):
+ def warnproblem(what, path_in_build_dir):
+ if toignore(path_in_build_dir):
return
- logging.warn('Found %s at %s' % (what, fd))
-
- def handleproblem(what, fd, fp):
- if toignore(fd):
- return ignoreproblem(what, fd)
- if todelete(fd):
- return removeproblem(what, fd, fp)
- logging.error('Found %s at %s' % (what, fd))
+ logging.warn('Found %s at %s' % (what, path_in_build_dir))
+
+ def handleproblem(what, path_in_build_dir, filepath):
+ if toignore(path_in_build_dir):
+ return ignoreproblem(what, path_in_build_dir)
+ if todelete(path_in_build_dir):
+ return removeproblem(what, path_in_build_dir, filepath)
+ logging.error('Found %s at %s' % (what, path_in_build_dir))
return 1
def is_executable(path):
return any(command.match(line) for command in gradle_compile_commands)
# Iterate through all files in the source code
- for dirpath, dirnames, filenames in os.walk(build_dir, topdown=True):
+ for root, dirs, files in os.walk(build_dir, topdown=True):
# It's topdown, so checking the basename is enough
for ignoredir in ('.hg', '.git', '.svn', '.bzr'):
- if ignoredir in dirnames:
- dirnames.remove(ignoredir)
+ if ignoredir in dirs:
+ dirs.remove(ignoredir)
- for curfile in filenames:
+ for curfile in files:
if curfile in ['.DS_Store']:
continue
# Path (relative) to the file
- fp = os.path.join(dirpath, curfile)
+ filepath = os.path.join(root, curfile)
- if os.path.islink(fp):
+ if os.path.islink(filepath):
continue
- fd = fp[len(build_dir) + 1:]
- _, ext = common.get_extension(fd)
+ path_in_build_dir = os.path.relpath(filepath, build_dir)
+ _, ext = common.get_extension(path_in_build_dir)
if ext == 'so':
- count += handleproblem('shared library', fd, fp)
+ count += handleproblem('shared library', path_in_build_dir, filepath)
elif ext == 'a':
- count += handleproblem('static library', fd, fp)
+ count += handleproblem('static library', path_in_build_dir, filepath)
elif ext == 'class':
- count += handleproblem('Java compiled class', fd, fp)
+ count += handleproblem('Java compiled class', path_in_build_dir, filepath)
elif ext == 'apk':
- removeproblem('APK file', fd, fp)
+ removeproblem('APK file', path_in_build_dir, filepath)
elif ext == 'jar':
for name in suspects_found(curfile):
- count += handleproblem('usual supect \'%s\'' % name, fd, fp)
- warnproblem('JAR file', fd)
+ count += handleproblem('usual supect \'%s\'' % name, path_in_build_dir, filepath)
+ if curfile == 'gradle-wrapper.jar':
+ removeproblem('gradle-wrapper.jar', path_in_build_dir, filepath)
+ else:
+ count += handleproblem('JAR file', path_in_build_dir, filepath)
+
+ elif ext == 'aar':
+ count += handleproblem('AAR file', path_in_build_dir, filepath)
elif ext == 'java':
- if not os.path.isfile(fp):
+ if not os.path.isfile(filepath):
continue
- with open(fp, 'r', encoding='utf8', errors='replace') as f:
+ with open(filepath, 'r', encoding='utf8', errors='replace') as f:
for line in f:
if 'DexClassLoader' in line:
- count += handleproblem('DexClassLoader', fd, fp)
+ count += handleproblem('DexClassLoader', path_in_build_dir, filepath)
break
elif ext == 'gradle':
- if not os.path.isfile(fp):
+ if not os.path.isfile(filepath):
continue
- with open(fp, 'r', encoding='utf8', errors='replace') as f:
+ with open(filepath, 'r', encoding='utf8', errors='replace') as f:
lines = f.readlines()
for i, line in enumerate(lines):
if is_used_by_gradle(line):
for name in suspects_found(line):
- count += handleproblem('usual supect \'%s\' at line %d' % (name, i + 1), fd, fp)
+ count += handleproblem('usual supect \'%s\' at line %d' % (name, i + 1), path_in_build_dir, filepath)
noncomment_lines = [l for l in lines if not common.gradle_comment.match(l)]
joined = re.sub(r'[\n\r\s]+', ' ', ' '.join(noncomment_lines))
for m in gradle_mavenrepo.finditer(joined):
url = m.group(2)
if not any(r.match(url) for r in allowed_repos):
- count += handleproblem('unknown maven repo \'%s\'' % url, fd, fp)
+ count += handleproblem('unknown maven repo \'%s\'' % url, path_in_build_dir, filepath)
elif ext in ['', 'bin', 'out', 'exe']:
- if is_binary(fp):
- count += handleproblem('binary', fd, fp)
+ if is_binary(filepath):
+ count += handleproblem('binary', path_in_build_dir, filepath)
- elif is_executable(fp):
- if is_binary(fp) and not safe_path(fd):
- warnproblem('possible binary', fd)
+ elif is_executable(filepath):
+ if is_binary(filepath) and not safe_path(path_in_build_dir):
+ warnproblem('possible binary', path_in_build_dir)
for p in scanignore:
if p not in scanignore_worked:
# Parse command line...
parser = ArgumentParser(usage="%(prog)s [options] [APPID[:VERCODE] [APPID[:VERCODE] ...]]")
common.setup_global_opts(parser)
- parser.add_argument("appid", nargs='*', help="app-id with optional versionCode in the form APPID[:VERCODE]")
+ parser.add_argument("appid", nargs='*', help=_("applicationId with optional versionCode in the form APPID[:VERCODE]"))
metadata.add_metadata_arguments(parser)
options = parser.parse_args()
metadata.warnings_action = options.W
for appid, app in apps.items():
if app.Disabled:
- logging.info("Skipping %s: disabled" % appid)
+ logging.info(_("Skipping {appid}: disabled").format(appid=appid))
continue
if not app.builds:
- logging.info("Skipping %s: no builds specified" % appid)
+ logging.info(_("Skipping {appid}: no builds specified").format(appid=appid))
continue
- logging.info("Processing " + appid)
+ logging.info(_("Processing {appid}").format(appid=appid))
try:
appid, traceback.format_exc()))
probcount += 1
- logging.info("Finished:")
- print("%d problems found" % probcount)
+ logging.info(_("Finished"))
+ print(_("%d problems found") % probcount)
if __name__ == "__main__":
~ianmdlvl / fdroidserver.git / blobdiff