Skip to content

Commit

Permalink
Fix some nits.
Browse files Browse the repository at this point in the history
  • Loading branch information
Benjy committed Jun 9, 2014
1 parent d9224ba commit 1b28767
Show file tree
Hide file tree
Showing 22 changed files with 11 additions and 61 deletions.
3 changes: 3 additions & 0 deletions src/python/pants/authentication/BUILD
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
# Copyright 2014 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

from pants.authentication.netrc_util import Netrc

netrc = Netrc()
Expand Down
2 changes: 0 additions & 2 deletions src/python/pants/authentication/__init__.py
Original file line number Diff line number Diff line change
@@ -1,2 +0,0 @@
# Copyright 2014 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).
3 changes: 2 additions & 1 deletion src/python/pants/authentication/netrc_util.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
# Copyright 2014 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

from __future__ import print_function
from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)

import collections
import os
Expand Down
5 changes: 0 additions & 5 deletions src/python/pants/backend/__init__.py
Original file line number Diff line number Diff line change
@@ -1,5 +0,0 @@
# Copyright 2014 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)
4 changes: 0 additions & 4 deletions src/python/pants/backend/jvm/ivy_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -252,10 +252,6 @@ def _generate_ivy(self, targets, jars, excludes, ivyxml, confs):
generator.write(output)

def _calculate_classpath(self, targets):

def is_jardependant(target):
return target.is_jar_library

jars = OrderedDict()
excludes = set()

Expand Down
2 changes: 0 additions & 2 deletions src/python/pants/backend/jvm/targets/artifact.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,9 @@
from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)

from twitter.common.collections import maybe_list
from twitter.common.lang import Compatibility

from pants.base.build_manual import manual
from pants.backend.jvm.targets.repository import Repository


@manual.builddict(tags=["jvm"])
Expand Down
4 changes: 2 additions & 2 deletions src/python/pants/backend/jvm/targets/jar_dependency.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,10 +176,10 @@ def with_artifact(self, name=None, type_=None, ext=None, url=None, configuration
extension, but not necessarily. For instance, ivy files are of type 'ivy' but have 'xml' as
their file extension.
:param ext: The file extension of the published artifact.
:param conf: The public configuration in which this artifact is published. The '*' wildcard can
be used to designate all public configurations.
:param url: The url at which this artifact can be found if it isn't located at the standard
location in the repository.
:param configuration: The public configuration in which this artifact is published. The '*' wildcard can
be used to designate all public configurations.
:param classifier: The maven classifier of this artifact.
"""
artifact = Artifact(name or self.name, type_=type_, ext=ext, url=url, conf=configuration,
Expand Down
5 changes: 0 additions & 5 deletions src/python/pants/backend/jvm/targets/jar_library.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,9 @@
from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)

from functools import partial

from twitter.common.collections import OrderedSet, maybe_list

from pants.base.build_manual import manual
from pants.base.payload import JarLibraryPayload
from pants.base.target import Target
from pants.backend.jvm.targets.jar_dependency import JarDependency


@manual.builddict(tags=["anylang"])
Expand Down
1 change: 0 additions & 1 deletion src/python/pants/backend/jvm/targets/java_library.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
print_function, unicode_literals)

from pants.base.build_manual import manual
from pants.base.exceptions import TargetDefinitionException
from pants.backend.jvm.targets.exportable_jvm_library import ExportableJvmLibrary


Expand Down
1 change: 0 additions & 1 deletion src/python/pants/backend/jvm/targets/jvm_target.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
from pants.base.target import Target
from pants.backend.jvm.targets.jar_library import JarLibrary
from pants.backend.jvm.targets.jarable import Jarable
from pants.backend.core.targets.resources import Resources


class JvmTarget(Target, Jarable):
Expand Down
5 changes: 0 additions & 5 deletions src/python/pants/backend/jvm/targets/scala_library.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,7 @@
from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)

from twitter.common.collections import maybe_list

from pants.base.address import SyntheticAddress
from pants.base.build_manual import manual
from pants.base.target import Target
from pants.base.exceptions import TargetDefinitionException
from pants.backend.jvm.targets.exportable_jvm_library import ExportableJvmLibrary


Expand Down
2 changes: 2 additions & 0 deletions src/python/pants/backend/jvm/tasks/depmap.py
Original file line number Diff line number Diff line change
Expand Up @@ -160,6 +160,8 @@ def output_deps(outputted, dep, parent=None):
if parent:
output.append(' "%s" -> "%s";' % (self._dep_id(parent)[0], self._dep_id(dep)[0]))

# TODO: This is broken. 'dependency' doesn't exist here, and we don't have
# internal_dependencies any more anyway.
if self._is_jvm(dependency):
for internal_dependency in dependency.internal_dependencies:
output += output_deps(outputted, internal_dependency, dependency)
Expand Down
1 change: 0 additions & 1 deletion src/python/pants/backend/jvm/tasks/filedeps.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
print_function, unicode_literals)

import itertools
import os

from pants.backend.jvm.targets.jvm_binary import JvmApp
from pants.backend.core.tasks.console_task import ConsoleTask
Expand Down
2 changes: 1 addition & 1 deletion src/python/pants/backend/jvm/tasks/jar_publish.py
Original file line number Diff line number Diff line change
Expand Up @@ -406,7 +406,7 @@ def parse_jarcoordinate(coordinate):
return org, name
else:
try:
address = Address.parse(get_buildroot(), coordinate)
address = Address.parse(get_buildroot(), coordinate) # TODO: This is broken.
try:
target = Target.get(address)
if not target:
Expand Down
1 change: 0 additions & 1 deletion src/python/pants/base/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)


try:
import ConfigParser
except ImportError:
Expand Down
1 change: 1 addition & 0 deletions src/python/pants/base/source_root.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
from pants.base.build_manual import manual
from pants.base.exceptions import TargetDefinitionException


@manual.builddict()
class SourceRoot(object):
"""Allows registration of a source root for a set of targets.
Expand Down
4 changes: 0 additions & 4 deletions src/python/pants/commands/command.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,9 @@
from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)

import os

from twitter.common.collections import OrderedSet

from pants.base.build_file import BuildFile
from pants.base.build_file_parser import BuildFileParser
from pants.base.build_graph import BuildGraph
from pants.base.config import Config
from pants.base.target import Target
from pants.base.workunit import WorkUnit
Expand Down
1 change: 0 additions & 1 deletion src/python/pants/goal/context.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
from collections import defaultdict
from contextlib import contextmanager

from twitter.common.collections import OrderedSet
from twitter.common.dirutil import Lock
from twitter.common.process import ProcessProviderFactory
from twitter.common.process.process_provider import ProcessProvider
Expand Down
Empty file removed src/python/pants/graph/__init__.py
Empty file.
1 change: 0 additions & 1 deletion src/python/pants/java/util.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
print_function, unicode_literals)

import os
import re

from pants.base.workunit import WorkUnit
from pants.java.executor import Executor, SubprocessExecutor
Expand Down
12 changes: 0 additions & 12 deletions src/python/pants/net/http/__init__.py
Original file line number Diff line number Diff line change
@@ -1,12 +0,0 @@
# Copyright 2014 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)

from pants.net.http.fetcher import Fetcher


__all__ = (
'Fetcher'
)
12 changes: 0 additions & 12 deletions src/python/pants/pants_doc/__init__.py
Original file line number Diff line number Diff line change
@@ -1,12 +0,0 @@
# Copyright 2014 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

from __future__ import (nested_scopes, generators, division, absolute_import, with_statement,
print_function, unicode_literals)

from pants.pants_doc.doc_builder import DocBuilder


__all__ = [
'DocBuilder',
]

0 comments on commit 1b28767

Please sign in to comment.