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

    mammoth-pharmacist-54215

    02/21/2016, 5:07 PM
    its a little odd
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:07 PM
    I’m also on El Capitan
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:07 PM
    python 2.7.9
  • a

    ancient-stone-50795

    02/21/2016, 5:07 PM
    Do you remember if you got the same 
    ImportError
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:07 PM
    nope, I got an assert error one second I’ll get the actual error
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:08 PM
    -.txt
    -.txt
  • a

    ancient-stone-50795

    02/21/2016, 5:08 PM
    Oh, I've gotten that before too.
  • f

    full-kilobyte-93429

    02/21/2016, 5:13 PM
    same assertion failure here as Matt
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:15 PM
    Doesn’t pants explicitly update sys.path to make sure contrib is on the path?
  • a

    ancient-stone-50795

    02/21/2016, 5:17 PM
    You think it might be a sourceroot problem?
  • a

    ancient-stone-50795

    02/21/2016, 5:17 PM
    https://rbcommons.com/s/twitter/r/3486/ will turn on self.maxDiff for that error we have all seen that seems to be intermittent
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:18 PM
    shipit
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:18 PM
    well the import error seems like it might be a sourceroot problem
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:19 PM
    the other one looks like a flaky test
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:25 PM
    Going to grab some breakfast. at mention me if you need a hand.
  • a

    ancient-stone-50795

    02/21/2016, 5:26 PM
    I agree, I don't think the two are related.
  • a

    ancient-stone-50795

    02/21/2016, 5:27 PM
    It seems to be guessing the module name incorrectly. It should be 
    pants_test.contrib.scalajs.tasks.test_scalajs_repl_integration
  • a

    ancient-stone-50795

    02/21/2016, 5:27 PM
    I think.
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:28 PM
    interesting
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:28 PM
    that is a little weird
  • a

    ancient-stone-50795

    02/21/2016, 5:31 PM
    What is also weird is that just 
    rm contrib/scalajs/tests/python/pants_test/contrib/__init__.py
    fixes the problem for me.
  • a

    ancient-stone-50795

    02/21/2016, 5:31 PM
    If I touch an empty 
    __init__.py
    in that place it fails again.
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:42 PM
    thats really weird
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:42 PM
    do you have 2.7.9 on your machine to try with that? See if its a 2.7.10 specific issue
  • m

    mammoth-pharmacist-54215

    02/21/2016, 5:43 PM
    also is there a init in pants_test?
  • r

    red-cpu-25099

    02/21/2016, 6:09 PM
    just for statistic - I had the same error as @mammoth-pharmacist-54215 and @full-kilobyte-93429 for the first run, and repeated success after
  • a

    ancient-stone-50795

    02/21/2016, 6:27 PM
    I don't have 2.7.9 on my machine. I did some debugging in this github issue: https://github.com/pantsbuild/pants/issues/2947
  • a

    ancient-stone-50795

    02/21/2016, 6:27 PM
    Yes, there is a non-empty init.py in pants_test
  • a

    ancient-stone-50795

    02/21/2016, 10:22 PM
    I cleared out some caches and things seem to have gotten better:
    Copy code
    $ rm -rf ~/.pex/install/py*
$ rm -rf ~/.cache/pants//python_cache
  • a

    ancient-stone-50795

    02/21/2016, 10:41 PM
    Yay! the release script finished successfully! https://pypi.python.org/pypi/pantsbuild.pants/0.0.73
1...994995996...2420Latest