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

    witty-crayon-22786

    08/13/2015, 6:15 PM
    at which point it would be very easy to split out the classpath preparation into it's own tiny task
  • w

    witty-crayon-22786

    08/13/2015, 6:15 PM
    which export could depend on
  • w

    witty-crayon-22786

    08/13/2015, 6:16 PM
    (well, not grouptask, but the global strategy at least)
  • g

    gorgeous-fish-11040

    08/13/2015, 6:16 PM
    Agree. Now I need a workaround simple smile
  • a

    adventurous-twilight-44110

    08/13/2015, 6:49 PM
    does anyone know of a good strategy for getting CI to avoid resolving certain targets? right now CI runs 
    ./pants test contrib::
    and automatically tries to resolve some un-resolvable `go_remote_library`s sitting in a testprojects directory
  • a

    adventurous-twilight-44110

    08/13/2015, 7:23 PM
    yeah me neither
  • w

    witty-crayon-22786

    08/13/2015, 7:24 PM
    heh
  • w

    witty-crayon-22786

    08/13/2015, 7:25 PM
    @adventurous-twilight-44110: maybe make them real somewhere?
  • w

    witty-crayon-22786

    08/13/2015, 7:25 PM
    there is a github repo (which contains the go binaries, for example) which gets synced to bintray
  • w

    witty-crayon-22786

    08/13/2015, 7:26 PM
    mocking remote stuff out in tests is admirable, but none of the python or jvm tests are doing that with their remote deps sooo
  • w

    witty-crayon-22786

    08/13/2015, 7:26 PM
    (particularly "immutable" remote stuff)
  • e

    enough-analyst-54434

    08/13/2015, 7:28 PM
    Cody - which are those - the tmp zips?
  • a

    adventurous-twilight-44110

    08/13/2015, 7:28 PM
    yup — we’re generating zips on the fly during tests now
  • e

    enough-analyst-54434

    08/13/2015, 7:29 PM
    yeah - so the 2nd step would save you
  • e

    enough-analyst-54434

    08/13/2015, 7:29 PM
    Don't write down those BUILDs on dist
  • e

    enough-analyst-54434

    08/13/2015, 7:29 PM
    create the targets on the fly in the test
  • e

    enough-analyst-54434

    08/13/2015, 7:29 PM
    It also saves you introducing the otherwise un-needed --host in this RB
  • e

    enough-analyst-54434

    08/13/2015, 7:29 PM
    You might add it later for other reasons, but not needed anymore for this logical change
  • e

    enough-analyst-54434

    08/13/2015, 7:30 PM
    s/on dist/on disk/
  • a

    adventurous-twilight-44110

    08/13/2015, 7:31 PM
    okay. so I’ll have to generate these files from an integration test — is that sane?
  • e

    enough-analyst-54434

    08/13/2015, 7:31 PM
    why does the it use the on-the-fly zips?
  • e

    enough-analyst-54434

    08/13/2015, 7:31 PM
    It should be enough for the units to use on the fly zips and target
  • e

    enough-analyst-54434

    08/13/2015, 7:31 PM
    IT can hit github if you want
  • e

    enough-analyst-54434

    08/13/2015, 7:32 PM
    but keep them cleanly divided in 2
  • e

    enough-analyst-54434

    08/13/2015, 7:32 PM
    aka any go_remote_library targets on disk should be real
  • e

    enough-analyst-54434

    08/13/2015, 7:32 PM
    is that reasonable?
  • w

    witty-crayon-22786

    08/13/2015, 7:34 PM
    @adventurous-twilight-44110: there is infrastructure for writing actual BUILD files to disk in integration tests
  • e

    enough-analyst-54434

    08/13/2015, 7:34 PM
    but don't use it!
  • a

    adventurous-twilight-44110

    08/13/2015, 7:35 PM
    haha
  • w

    witty-crayon-22786

    08/13/2015, 7:35 PM
    it's lower level than the 
    make_target
    stuff
1...166167168...2420Latest