• Henrik Kjellander's avatar
    Use DEPS for all dependencies + add PRESUBMIT.py · f49fde79
    Henrik Kjellander authored
    This changes libyuv to use the DEPS file for pulling
    down all dependencies (thus no Chromium checkout is needed any more).
    
    Add tools_libyuv directory to contain libyuv-specific tools
    (needed to avoid name collision with the now DEPSed tools/ directory
    of Chromium, which is needed by the toolchain).
    Add tools_libyuv/autoroller/roll_deps.py script to automatically
    roll all entries in the DEPS file (copied from WebRTC).
    
    third_party/ is now DEPSed as well, including the gtest configuration
    headers that used to live inside the libyuv repo.
    
    Add PRESUBMIT.py with a few simple checks + execution of PyLint and
    Python unit tests. For PyLint a pylintrc file was also added.
    
    Valgrind in tools_libyuv/valgrind was updated to make PRESUBMIT.py pass
    and remove old tsan suppressions (not used).
    
    Removed util/android/test_runner.py since it's no longer needed.
    
    Buildbot changes in https://chromium-review.googlesource.com/436464 
    are needed for the Memcheck bot to go green.
    
    BUG=libyuv:676
    NOTRY=True
    
    Change-Id: Ib86fea2905a1656bba2933703ce5a59d29d8db6b
    Reviewed-on: https://chromium-review.googlesource.com/436264
    Commit-Queue: Henrik Kjellander <kjellander@chromium.org>
    Reviewed-by: 's avatarFrank Barchard <fbarchard@google.com>
    f49fde79
PRESUBMIT.py 3.83 KB
#!/usr/bin/env python
# Copyright (c) 2012 The LibYuv Project Authors. All rights reserved.
#
# Use of this source code is governed by a BSD-style license
# that can be found in the LICENSE file in the root of the source
# tree. An additional intellectual property rights grant can be found
# in the file PATENTS.  All contributing project authors may
# be found in the AUTHORS file in the root of the source tree.

"""
Copied from Chrome's src/tools/valgrind/memcheck/PRESUBMIT.py

See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
for more details on the presubmit API built into gcl.
"""

import os
import re
import sys

def CheckChange(input_api, output_api):
  """Checks the memcheck suppressions files for bad data."""

  # Add the path to the Chrome valgrind dir to the import path:
  tools_vg_path = os.path.join(input_api.PresubmitLocalPath(), '..', '..', '..',
                               'tools', 'valgrind')
  sys.path.append(tools_vg_path)
  import suppressions

  sup_regex = re.compile('suppressions.*\.txt$')
  suppressions = {}
  errors = []
  check_for_memcheck = False
  # skip_next_line has 3 possible values:
  # - False: don't skip the next line.
  # - 'skip_suppression_name': the next line is a suppression name, skip.
  # - 'skip_param': the next line is a system call parameter error, skip.
  skip_next_line = False
  for f in filter(lambda x: sup_regex.search(x.LocalPath()),
                  input_api.AffectedFiles()):
    for line, line_num in zip(f.NewContents(),
                              xrange(1, len(f.NewContents()) + 1)):
      line = line.lstrip()
      if line.startswith('#') or not line:
        continue

      if skip_next_line:
        if skip_next_line == 'skip_suppression_name':
          if 'insert_a_suppression_name_here' in line:
            errors.append('"insert_a_suppression_name_here" is not a valid '
                          'suppression name')
          if suppressions.has_key(line):
            if f.LocalPath() == suppressions[line][1]:
              errors.append('suppression with name "%s" at %s line %s '
                            'has already been defined at line %s' %
                            (line, f.LocalPath(), line_num,
                             suppressions[line][1]))
            else:
              errors.append('suppression with name "%s" at %s line %s '
                            'has already been defined at %s line %s' %
                            (line, f.LocalPath(), line_num,
                             suppressions[line][0], suppressions[line][1]))
          else:
            suppressions[line] = (f, line_num)
            check_for_memcheck = True;
        skip_next_line = False
        continue
      if check_for_memcheck:
        if not line.startswith('Memcheck:'):
          errors.append('"%s" should be "Memcheck:..." in %s line %s' %
                        (line, f.LocalPath(), line_num))
        check_for_memcheck = False;
      if line == '{':
        skip_next_line = 'skip_suppression_name'
        continue
      if line == "Memcheck:Param":
        skip_next_line = 'skip_param'
        continue

      if (line.startswith('fun:') or line.startswith('obj:') or
          line.startswith('Memcheck:') or line == '}' or
          line == '...'):
        continue
      errors.append('"%s" is probably wrong: %s line %s' % (line, f.LocalPath(),
                                                            line_num))
  if errors:
    return [output_api.PresubmitError('\n'.join(errors))]
  return []

def CheckChangeOnUpload(input_api, output_api):
  return CheckChange(input_api, output_api)

def CheckChangeOnCommit(input_api, output_api):
  return CheckChange(input_api, output_api)

def GetPreferredTrySlaves():
  # We don't have any memcheck slaves yet, so there's no use for this method.
  # When we have, the slave name(s) should be put into this list.
  return []