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

    enough-analyst-54434

    09/01/2015, 8:07 PM
    yeah
  • e

    enough-analyst-54434

    09/01/2015, 8:07 PM
    I guess you could do this:
  • e

    enough-analyst-54434

    09/01/2015, 8:08 PM
    If you really want dots, the alias strings already accept them just fine, but python eval will blow up, so in BuildConfiguration, for each alias with dots, split on dots and construct throwaway objects the support the python eval . traversal as maybe properties
  • j

    jolly-chef-92794

    09/01/2015, 8:09 PM
    Hmm, I’d be up for that. Sounds like a separate RB though
  • e

    enough-analyst-54434

    09/01/2015, 8:09 PM
    You'd want to float this though since its a hack around no lexical activation of names and its more complexity that may not be worth it
  • e

    enough-analyst-54434

    09/01/2015, 8:09 PM
    for just s/_/./ at the end of the day!
  • j

    jolly-chef-92794

    09/01/2015, 8:10 PM
    Yes. Maybe I will just use the 
    snake_case
    for now with my current changes, then investigate improving (or complicating) BuildConfiguration in a follow-up
  • e

    enough-analyst-54434

    09/01/2015, 8:11 PM
    sgtm
  • e

    enough-analyst-54434

    09/01/2015, 8:13 PM
    Tweeps - I'm hoping to hear that you're not using the Go plugin yet - is that true?
  • w

    witty-crayon-22786

    09/01/2015, 8:25 PM
    @enough-analyst-54434: that is true. we'll be poking one team to start migrating soon
  • e

    enough-analyst-54434

    09/01/2015, 8:27 PM
    cool, I may not put effort into a deprecation cycle for name='' in go targets then
  • w

    witty-crayon-22786

    09/01/2015, 8:27 PM
    +1
  • e

    enough-analyst-54434

    09/01/2015, 8:27 PM
    We can clean up the Medium BUILDs by hand easy enough
  • e

    enough-analyst-54434

    09/01/2015, 8:28 PM
    And that was a dangerous but actually valid use of the royal we! Since I'll be doing that cleanup
  • e

    enough-analyst-54434

    09/01/2015, 8:28 PM
    ..phew - Mr Farner would have taken me down if that leaked
  • w

    witty-crayon-22786

    09/01/2015, 8:29 PM
    heh
  • b

    bright-dinner-81701

    09/01/2015, 9:41 PM
    @enough-analyst-54434: @witty-crayon-22786 just opened my first issue 😛 https://github.com/pantsbuild/pants/issues/2115
  • b

    bright-dinner-81701

    09/01/2015, 9:42 PM
    so i just make a branch in my repo fork and issue a pull request?
  • b

    billowy-alarm-80641

    09/01/2015, 9:42 PM
    no,
  • b

    billowy-alarm-80641

    09/01/2015, 9:42 PM
    the doc says what to do
  • b

    billowy-alarm-80641

    09/01/2015, 9:42 PM
  • b

    billowy-alarm-80641

    09/01/2015, 9:42 PM
    make a branch
  • b

    billowy-alarm-80641

    09/01/2015, 9:43 PM
    and use "`./rbt-create`" to publish an RB for it
  • b

    bright-dinner-81701

    09/01/2015, 9:43 PM
    right ok
  • b

    bright-dinner-81701

    09/01/2015, 9:43 PM
    thanks
  • b

    billowy-alarm-80641

    09/01/2015, 9:43 PM
    sure, np
  • e

    enough-analyst-54434

    09/01/2015, 9:46 PM
    Luckily, http://pantsbuild.github.io/setup/ works - although I would never have thought to check it did
  • e

    enough-analyst-54434

    09/01/2015, 9:47 PM
    Good find Chris
  • a

    ancient-stone-50795

    09/01/2015, 11:52 PM
    I miss FAT
  • a

    ancient-stone-50795

    09/01/2015, 11:52 PM
    no space problems there.
1...269270271...2420Latest