https://pantsbuild.org/ logo
Powered by
#general
  • b

    bored-art-40741

    10/07/2015, 4:25 PM
    To be clear I have no issue with the warning staying
  • b

    bored-art-40741

    10/07/2015, 4:25 PM
    There's no clear way to warn about that and then know when to stop, and once you know why the warning is there it's easy to skim past
  • f

    fast-megabyte-33579

    10/07/2015, 4:26 PM
    patrick.lawson: in this instance though, it feels like it should be checking the right-hand-side of the declaration and going 'this isn't a list'
  • f

    fast-megabyte-33579

    10/07/2015, 4:26 PM
    *it feels to me
  • b

    bored-art-40741

    10/07/2015, 4:26 PM
    It serves a dual purpose of also saying "you probably don't need this at all"
  • b

    bored-art-40741

    10/07/2015, 4:26 PM
    We happen to override a couple of those because we actually do override, but I think that's fairly rare and we know how to interpret the warning
  • b

    bored-art-40741

    10/07/2015, 4:27 PM
    Maybe it could be split into two warnings
  • b

    bored-art-40741

    10/07/2015, 4:27 PM
    One is an error (this thing must be a single spec) and the other is a light warning (heads up, you might not need this at all if you just want to track upstream pants' default for this tool)
  • e

    enough-analyst-54434

    10/07/2015, 4:29 PM
    The error is there - at runtime
  • e

    enough-analyst-54434

    10/07/2015, 4:29 PM
    You hit a bug in that error IIRC!
  • b

    bored-art-40741

    10/07/2015, 4:29 PM
    Hm that sounds familiar
  • e

    enough-analyst-54434

    10/07/2015, 4:55 PM
    Yeah: ```
  • e

    enough-analyst-54434

    10/07/2015, 4:55 PM
    ```
  • e

    enough-analyst-54434

    10/07/2015, 4:55 PM
    Copy code
    $ git diff -U1
diff --git a/pants.ini b/pants.ini
index a0b697d..96c2027 100644
--- a/pants.ini
+++ b/pants.ini
@@ -109,3 +109,3 @@ deps: ["3rdparty:thrift-0.9.2"]
 [compile.checkstyle]
-bootstrap_tools: ["//:checkstyle"]
+checkstyle: ["//:checkstyle"]
 configuration: %(pants_supportdir)s/checkstyle/coding_style.xml
$ ./pants compile src/java/::
...
Exception message: JVM tool configuration now expects a single target address, use the
following in pants.ini:

[compile.checkstyle]
checkstyle: //tool/classpath:address
  • e

    enough-analyst-54434

    10/07/2015, 4:56 PM
    So the error appears to work correctly at runtime
  • a

    ancient-stone-50795

    10/07/2015, 5:08 PM
    I am getting a mysterious error when running python tests in my repo that just cropped up:
    Copy code
    Exception message: Ambiguous resolvable: six>=1.4
  • a

    ancient-stone-50795

    10/07/2015, 5:08 PM
    More context:
    Copy code
    09:35:11 00:43   [test]
09:35:11 00:43     [run_prep_command]
09:35:11 00:43     [test]
09:35:11 00:43     [pytest]
09:35:11 00:43       [run]
                     
09:35:14 00:46         [chroot]
               Waiting for background workers to finish.
09:35:25 00:57   [complete]
               FAILURE
Exception message: Ambiguous resolvable: six>=1.4
  • a

    ancient-stone-50795

    10/07/2015, 5:09 PM
    Its suddenly broken in our CI
  • a

    ancient-stone-50795

    10/07/2015, 5:09 PM
    Presumably because someone just published six version 1.1.0 today
  • a

    ancient-stone-50795

    10/07/2015, 5:12 PM
    It is not failing on my laptop. For the life of me I can’t find any reference like this in the Pants codebase.
  • f

    fast-megabyte-33579

    10/07/2015, 5:14 PM
    Back to the pants.ini stuff - we have a java platform specified in it but not one for scala platform. It looks like it takes it from the contents of BUILD.tools - is that right?
  • f

    fast-megabyte-33579

    10/07/2015, 5:14 PM
    zundel: we have
    Copy code
    six==1.9.0
    in our requirements.txt
  • f

    fast-megabyte-33579

    10/07/2015, 5:14 PM
    but I don't know what it is
  • a

    ancient-stone-50795

    10/07/2015, 5:15 PM
    Yes, and I am running a pants build from Friday 0.0.51
  • f

    fast-megabyte-33579

    10/07/2015, 5:15 PM
    that message is coming from pex
  • a

    ancient-stone-50795

    10/07/2015, 5:24 PM
    @fast-megabyte-33579: To my knowledge, all your scala versioning stuff is set in BUILD.tools
  • a

    ancient-stone-50795

    10/07/2015, 5:26 PM
    Partial stacktrace:
    Copy code
    ...
0.0.51_square_20151002_01-py2-none-any.whl/pants/backend/python/tasks/python_task.py", line 151, in cached_chroot
    extra_requirements, executable_file_content)
  File "/Users/square/.pex/install/pantsbuild.pants-0.0.51_square_20151002_01-py2-none-any.whl.b3f6b875b9e02db4b11b549db9cac13efaabdade/pantsbuild.pants-0.0.51_square_20151002_01-py2-none-any.whl/pants/backend/python/tasks/python_task.py", line 186, in _build_chroot
    chroot.dump()
  File "/Users/square/.pex/install/pantsbuild.pants-0.0.51_square_20151002_01-py2-none-any.whl.b3f6b875b9e02db4b11b549db9cac13efaabdade/pantsbuild.pants-0.0.51_square_20151002_01-py2-none-any.whl/pants/backend/python/python_chroot.py", line 226, in dump
    distributions = self._resolve_multi(reqs_to_build, find_links)
  File "/Users/square/.pex/install/pantsbuild.pants-0.0.51_square_20151002_01-py2-none-any.whl.b3f6b875b9e02db4b11b549db9cac13efaabdade/pantsbuild.pants-0.0.51_square_20151002_01-py2-none-any.whl/pants/backend/python/python_chroot.py", line 265, in _resolve_multi
    cache_ttl=self._python_setup.resolver_cache_ttl)
  File "/Users/square/.pex/install/pex-1.0.3-py2.py3-none-any.whl.9b479c2967fcb9866cb8b019740e265c9bc00da2/pex-1.0.3-py2.py3-none-any.whl/pex/resolver.py", line 337, in resolve
    return resolver.resolve(resolvables_from_iterable(requirements, builder))
  File "/Users/square/.pex/install/pex-1.0.3-py2.py3-none-any.whl.9b479c2967fcb9866cb8b019740e265c9bc00da2/pex-1.0.3-py2.py3-none-any.whl/pex/resolver.py", line 188, in resolve
    raise self.Error('Ambiguous resolvable: %s' % resolvable)

Exception message: Ambiguous resolvable: six>=1.4
  • a

    ancient-stone-50795

    10/07/2015, 5:28 PM
    You know, the whole reason I bother with building a pex file is so that I can distribute a pre-resolved bundle of stuff for everyone. Why is it consulting the internet for anything!?!
  • e

    enough-analyst-54434

    10/07/2015, 5:30 PM
    The stack trace shows pants running a python backend task
  • e

    enough-analyst-54434

    10/07/2015, 5:30 PM
    Its resolving to implement the task, not to run pants itself
1...508509510...2420Latest