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

    enough-analyst-54434

    07/17/2015, 3:48 PM
    @benjy - I'm in a position to need Subsystem dependencies, so I think I'd like to wipe out your design doc / code TODO on this - sound good? or will this trample work you're doing?
  • w

    witty-crayon-22786

    07/17/2015, 3:49 PM
    interesting!
  • h

    happy-kitchen-89482

    07/17/2015, 3:57 PM
    @enough-analyst-54434: I have no pending work on this, and it sounds like a great idea, so go for it!
  • h

    happy-kitchen-89482

    07/17/2015, 3:57 PM
    Out of curiosity, what's the motivating example?
  • e

    enough-analyst-54434

    07/17/2015, 4:00 PM
    I'm shaving the yak of bumping aurora from apache thift 0.9.1 to 0.9.2 ... which is part of satisfying the Clover Health need for python3 compat codegen for thrift ... and I need a ThriftBinary subsystem that ApacheThriftGen and the python ThriftBuilder can share - this needs a BinaryUtil subsystem that knows the basics of fetching pre-built binaries (thrift, protobuf, ragel, ...)
  • e

    enough-analyst-54434

    07/17/2015, 4:00 PM
    several global options will die
  • e

    enough-analyst-54434

    07/17/2015, 4:01 PM
    well, move to subsystems
  • w

    witty-crayon-22786

    07/17/2015, 4:01 PM
    ah yes... i would also love that.
  • b

    bored-art-40741

    07/17/2015, 4:02 PM
    Remember the huge backwards incompatible change from 0.9.1 to 0.9.2!
  • w

    witty-crayon-22786

    07/17/2015, 4:02 PM
    (more binaries are coming in the form of node/npm/go* prolly)
  • e

    enough-analyst-54434

    07/17/2015, 4:05 PM
    Although I'm not affected - doing this for others, what is that incompat @bored-art-40741 ?
  • b

    bored-art-40741

    07/17/2015, 4:06 PM
    0.9.1 TCompact serialized and deserialized doubles with the wrong endianess
  • e

    enough-analyst-54434

    07/17/2015, 4:06 PM
    aha
  • e

    enough-analyst-54434

    07/17/2015, 4:06 PM
    k - compact not used by aurora
  • b

    bored-art-40741

    07/17/2015, 4:06 PM
    I fixed this, and they released the fix in 0.9.2 with basically no mention that it's a massively incompatible and dangerous change
  • e

    enough-analyst-54434

    07/17/2015, 4:10 PM
    https://issues.apache.org/jira/browse/THRIFT-1639 - well, I have a hard time imagining someone worked around this by continuing to use compact, but reverse engineering the correct double
  • e

    enough-analyst-54434

    07/17/2015, 4:10 PM
    So I'd guess it would break not many folks
  • b

    bored-art-40741

    07/17/2015, 4:12 PM
    It would be dangerous for shops that do serde out to a database with python only
  • b

    bored-art-40741

    07/17/2015, 4:12 PM
    Using TCompact
  • e

    enough-analyst-54434

    07/17/2015, 4:13 PM
    right - with a single encoder/decoder things used to work fine
  • w

    witty-crayon-22786

    07/17/2015, 4:48 PM
    about to start a release... anyone have anything in flight that they need?
  • w

    witty-crayon-22786

    07/17/2015, 4:58 PM
    correction... pausing until i get https://rbcommons.com/s/twitter/r/2489/ nailed down.
  • m

    mammoth-autumn-2551

    07/17/2015, 9:05 PM
    @enough-analyst-54434: I’ve opened an issue and attached a link to a demo project that showcases the problem I mentioned last week: https://github.com/pantsbuild/pants/issues/1831
  • m

    mammoth-autumn-2551

    07/17/2015, 9:06 PM
    To recap, I’m trying to configure a repo to just pull from local repo for wheels, but it seems that when bootstrapping whatever goes into .pants.d/python-setup/interpreters, it needs to fetch setuptools and wheel from an index, not a repo
  • e

    enough-analyst-54434

    07/17/2015, 9:07 PM
    K - I'll take a look. I'm sure this is a bug I introduced with the pex 1.0.1 upgrade
  • m

    mammoth-autumn-2551

    07/17/2015, 9:10 PM
    I’m not sure thats true… I had this problem in our Compass repo which is still on 0.0.32
  • e

    enough-analyst-54434

    07/17/2015, 9:11 PM
    Hrm, Twitter worked like this long before I left
  • e

    enough-analyst-54434

    07/17/2015, 9:12 PM
    Sometime back in 2012
  • m

    mammoth-autumn-2551

    07/17/2015, 9:13 PM
    This may just be a configuration tweak that I need to do… I just need to know the correct hook to tell the interpreter cache to resolve from the repo, or perhaps modify what I put in there so it can recognize the setuptools/wheel egg
  • m

    mammoth-autumn-2551

    07/17/2015, 9:44 PM
    @enough-analyst-54434: I think I resolved this issue… it seems that the python_interpreter_cache only looks for sdists, but then converts it to a bdist and will read that from the cache
1...585960...2420Latest