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

    enough-analyst-54434

    04/29/2016, 11:34 PM
    cokes clinking
  • w

    witty-crayon-22786

    04/29/2016, 11:35 PM
    Task.alternate_target_roots
  • h

    happy-kitchen-89482

    04/29/2016, 11:35 PM
    Hmm, I don’t want to modify the target roots though
  • h

    happy-kitchen-89482

    04/29/2016, 11:35 PM
    I want to take the existing build graph and augment it
  • h

    happy-kitchen-89482

    04/29/2016, 11:35 PM
    I guess that passes me the build graph
  • e

    enough-analyst-54434

    04/29/2016, 11:35 PM
    You can do that
  • h

    happy-kitchen-89482

    04/29/2016, 11:35 PM
    so I can tinker with it
  • h

    happy-kitchen-89482

    04/29/2016, 11:35 PM
    and then return None
  • w

    witty-crayon-22786

    04/29/2016, 11:36 PM
    there are examples of injecting synthetic targets from within targets
  • w

    witty-crayon-22786

    04/29/2016, 11:36 PM
    but it's all... scary =(
  • w

    witty-crayon-22786

    04/29/2016, 11:36 PM
    traversable_specs or traversable_dependency_specs on targets
  • w

    witty-crayon-22786

    04/29/2016, 11:37 PM
    are approximately "load these please" and "load these and make them implicit dependencies"
  • e

    enough-analyst-54434

    04/29/2016, 11:37 PM
    Well, those 2 are scary by design as hashed out in 2014 - thats just the way 1.0 is and will be
  • w

    witty-crayon-22786

    04/29/2016, 11:37 PM
    yea
  • w

    witty-crayon-22786

    04/29/2016, 11:37 PM
    @enough-analyst-54434: i have this theory that traversable_dependency_specs can be replaced with macros...?
  • h

    happy-kitchen-89482

    04/29/2016, 11:38 PM
    I can’t use traversable_dependency_specs because I don’t know what to add until I get task options
  • w

    witty-crayon-22786

    04/29/2016, 11:38 PM
    and i think in all cases traversable_specs are cycles, many of which can be fixed
  • e

    enough-analyst-54434

    04/29/2016, 11:38 PM
    Yeah - this was traditionally done in constructors IIRC Benjy
  • h

    happy-kitchen-89482

    04/29/2016, 11:38 PM
    Context: I want to add, to all java_library targets, dependencies on the set of targets that contain javac plugins
  • e

    enough-analyst-54434

    04/29/2016, 11:38 PM
    Task constructors
  • w

    witty-crayon-22786

    04/29/2016, 11:38 PM
    @happy-kitchen-89482: iirc, targets can have subsystem dependencies
  • e

    enough-analyst-54434

    04/29/2016, 11:38 PM
    Based on engine docs about what it does in what order
  • h

    happy-kitchen-89482

    04/29/2016, 11:39 PM
    So ZincCompile will have an option 
    --javac-plugin-deps
  • h

    happy-kitchen-89482

    04/29/2016, 11:39 PM
    which is a list of targets that contain plugins to build and stick on the command line
  • h

    happy-kitchen-89482

    04/29/2016, 11:39 PM
    in addition to any plugins you explicitly depend on
  • h

    happy-kitchen-89482

    04/29/2016, 11:39 PM
    this will allow you to run plugins without having to modify BUILD files
  • h

    happy-kitchen-89482

    04/29/2016, 11:40 PM
    I could create a javac-plugins subsystem I suppose
  • w

    witty-crayon-22786

    04/29/2016, 11:40 PM
    can the task just resolve those then? why does that require mutating the target?
  • h

    happy-kitchen-89482

    04/29/2016, 11:41 PM
    these might be in-repo targets
  • w

    witty-crayon-22786

    04/29/2016, 11:41 PM
    especially if it's compile-time-only
1...122812291230...2420Latest