04629674f5f9e65bd35565b329f8c892a16865a4,src/python/twitter/pants/tasks/jvm_compiler_dependencies.py,Dependencies,findclasses,#Dependencies#Any#,42

Before Change

    classes_by_target_by_source = defaultdict(lambda: defaultdict(set))
    if os.path.exists(self.depfile):
      with open(self.depfile, "r") as deps:
        for dep in deps.readlines():
          src, cls = dep.strip().split("->")
          sourcefile = os.path.relpath(os.path.join(self.outputdir, src.strip()), get_buildroot())
          if sourcefile in sources:
            classfile = os.path.relpath(os.path.join(self.outputdir, cls.strip()), self.outputdir)
            target = target_by_source[sourcefile]
            relsrc = os.path.relpath(sourcefile, target.target_base)
            classes_by_target_by_source[target][relsrc].add(classfile)
    return classes_by_target_by_source

After Change

        sources.add(src)

    classes_by_target_by_source = defaultdict(lambda: defaultdict(set))
    for sourcefile, classfiles in self.classes_by_source.items():
      if sourcefile in sources:
        target = target_by_source[sourcefile]
        relsrc = os.path.relpath(sourcefile, target.target_base)
        classes_by_target_by_source[target][relsrc] = classfiles
    return classes_by_target_by_source

Instances