Add a pantsd.ini for development use of the daemon + watchman + buildgraph caching.

Review Request #3859 — Created May 11, 2016 and submitted

kwlzn
pants
kwlzn/pantsd/ini
3387
pants-reviews
jsirois, peiyu, stuhood, wisechengyi
Add a pantsd.ini for development use of the daemon + watchman + buildgraph caching.

Tested each step in the doc both with --config-override and with export PANTS_CONFIG_OVERRIDE.

  • 0
  • 0
  • 1
  • 0
  • 1
Description From Last Updated
  1. 
      
  2. pantsd.ini (Diff revision 1)
     
     

    The list 3rdparty:: examples scream bug to a dev. If you did not typo it may be worth calling out why :: may not be wise yet.

    1. ah, yes - this is indeed intentional. currently list :: does not fully work (see e.g. #3388) - but progress has been steadily made in recent reviews and should tip very soon.

      I think this is fairly well understood by the current engine-focused devs and the idea here being that this is just a quick working example usage, but folks are free to call it however they need as part of dev/test.

      still think this warrants a special callout? I just worry it'll quickly become obsoleted.

    2. Sounds fine to me.
  3. 
      
  1. Ship It!
  2. 
      
  1. 
      
    1. Also, convention would probably recommend: pants.d.ini to line up with the others.

    2. fixed.

  2. pantsd.ini (Diff revision 1)
     
     
     

    This should go in the [GLOBAL] section.

  3. 
      
Review request changed

Status: Closed (submitted)

Change Summary:

thanks gents! submitted @ a6a4bbdcc4d8d22535e21c69e70eca04eaf46ed2

Loading...