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

    wide-energy-11069

    10/29/2018, 8:14 PM
    Copy code
    $ ./pants run examples/src/python/example/hello/main
...
$ ls -l dist
total 8992
-rwxr-xr-x  1 yic  wheel   496613 Oct 29 13:12 main.pex
  • e

    enough-analyst-54434

    10/29/2018, 8:14 PM
    It is if some other vertical needs binaries before runs
  • e

    enough-analyst-54434

    10/29/2018, 8:14 PM
    The current old goal system works that way
  • e

    enough-analyst-54434

    10/29/2018, 8:14 PM
    --explain should confirm this
  • w

    wide-energy-11069

    10/29/2018, 8:15 PM
    yeah, log says so as well
    Copy code
    13:12:48 00:03   [binary]
13:12:48 00:03     [binary-jvm-prep-command]
13:12:48 00:03       [jvm_prep_command]
13:12:48 00:03     [binary-prep-command]
13:12:48 00:03     [py]
                   created pex dist/main.pex
13:12:48 00:03     [py-wheels]
13:12:48 00:03     [jvm]
13:12:48 00:03     [dup]
13:12:48 00:03     [cpplib]
13:12:48 00:03       [cpp-library]
13:12:48 00:03     [cpp]
13:12:48 00:03       [cpp-binary]
13:12:48 00:03     [go]
13:12:48 00:03   [run]
13:12:48 00:03     [py]
                   Invalidated 2 targets.
13:12:49 00:04       [run]
Hello, world!
  • e

    enough-analyst-54434

    10/29/2018, 8:16 PM
    Most likely cpp->CppRun_run_cpp
  • e

    enough-analyst-54434

    10/29/2018, 8:16 PM
    That would need a binary
  • w

    wide-energy-11069

    10/29/2018, 8:16 PM
    interesting. cc @strong-crayon-70379 for the side effects
  • e

    enough-analyst-54434

    10/29/2018, 8:17 PM
    This is a very old truth fwiw
  • w

    wide-energy-11069

    10/29/2018, 8:17 PM
    iirc it didn’t use to be that way?
  • w

    wide-energy-11069

    10/29/2018, 8:17 PM
    depending on the definition of 
    old
    i guess 🙂
  • s

    strong-crayon-70379

    10/29/2018, 8:18 PM
    Also, those cpp tasks are from contrib and are in no way related to the native support work from me and Danny.
  • w

    wide-energy-11069

    10/29/2018, 8:18 PM
    ah good to know.
  • e

    enough-analyst-54434

    10/29/2018, 8:19 PM
    @wide-energy-11069 Pants has worked that way since you've been involved and I'm pretty darn sure always
  • w

    witty-crayon-22786

    10/29/2018, 8:53 PM
    i think that because twitter does not have the 
    cpp
    contrib plugin installed, 
    run
    does not have dependency on 
    binary
    for us.
  • w

    witty-crayon-22786

    10/29/2018, 8:55 PM
    (yea, confirmed.)
    👍 1
  • m

    mysterious-farmer-45668

    10/29/2018, 9:02 PM
    If there's a python 3rdparty that provides both py2 version and py3 version, we can enforce the py3 version by doing something like this, right? 
    apache-airflow==1.10.0 ; python_version<'3'
  • m

    mysterious-farmer-45668

    10/29/2018, 9:02 PM
    (in the requirements.txt file)
  • h

    hundreds-father-404

    10/29/2018, 9:09 PM
    @mysterious-farmer-45668 that means it will only install the dependency if you’re using Python 2. When using Python 3, it won’t do anything. Are you using both a Python 2 and 3 interpreter, or only Py3?
  • m

    mysterious-farmer-45668

    10/29/2018, 9:11 PM
    we're trying to make our whole codebase py3 compliant
  • m

    mysterious-farmer-45668

    10/29/2018, 9:12 PM
    so we put in our pants.ini something like 
    interpreter_constraints: ['CPython>=3']
  • m

    mysterious-farmer-45668

    10/29/2018, 9:12 PM
    and noticed that some dependencies are still pulling in (we're assuming) the py2 deps
  • m

    mysterious-farmer-45668

    10/29/2018, 9:13 PM
    so by that logic.... if 
    python_version>'3'
    it will only pull the dependency if it is above py3.0?
  • h

    hundreds-father-404

    10/29/2018, 9:14 PM
    Yes exactly, although I’m not sure that will actually change what gets downloaded. What leads you to think it’s downloading the Python 2 version? It should either be downloading a universal wheel that works for either interpreter, or a Py3 only wheel. Do you see where the dependency is being installed to?
  • m

    mysterious-farmer-45668

    10/29/2018, 9:15 PM
    unsure, which is why it's an assumption... hmm. is there a way to check where the dep is being installed to?
  • m

    mysterious-farmer-45668

    10/29/2018, 9:17 PM
    i removed the version constraint and am getting an error that's different from my original question
  • m

    mysterious-farmer-45668

    10/29/2018, 9:17 PM
    pyprep stage gives me this:
    Copy code
    **** Failed to install apache-airflow-1.10.0 (caused by: NonZeroExit("received exit code 1 during execution of `[u'/anaconda3/bin/python3.7', '-', 'bdist_wheel', '--dist-dir=/var/folders/7y/9l6fcjcn6pj__cgcnk3ggdxwzs6css/T/tmpR00Gqz']` while trying to execute `[u'/anaconda3/bin/python3.7', '-', 'bdist_wheel', '--dist-dir=/var/folders/7y/9l6fcjcn6pj__cgcnk3ggdxwzs6css/T/tmpR00Gqz']`",)
):
stdout:

stderr:
Traceback (most recent call last):
  File "<stdin>", line 9, in <module>
  File "setup.py", line 124
    async = [
          ^
SyntaxError: invalid syntax
  • m

    mysterious-farmer-45668

    10/29/2018, 9:18 PM
    i don't think this is a pants issue. but thanks eric for the answer!
  • m

    mysterious-farmer-45668

    10/29/2018, 10:17 PM
    what are some possible culprits when the same test/build command is producing different results across two osx machines?
  • m

    mysterious-farmer-45668

    10/29/2018, 10:21 PM
    same branch of the repo works for my machine and throws a 3rdparty transitive dependency problem for a python target for my coworker
1...207920802081...2420Latest