#general
- b
bored-art-40741
09/08/2015, 6:32 PMIn that case, you need to depend on the buildgen contrib pypi package - b
bored-art-40741
09/08/2015, 6:32 PMAt which point you'll be able to importBuildFileManipulator - a
astonishing-jelly-60479
09/08/2015, 6:32 PMi'm using a hacked up version of this at the moment https://github.com/mitsuhiko/pyastutil/blob/master/astutil/codegen.py - b
bored-art-40741
09/08/2015, 6:33 PM
was super scope restricted because I couldn't find anything that did quite what I wantedBuildFileManipulator - b
bored-art-40741
09/08/2015, 6:33 PMIt's plausible there are better tools for this - a
astonishing-jelly-60479
09/08/2015, 6:33 PMi searched on pypi for a buildgen package - b
bored-art-40741
09/08/2015, 6:33 PMBFM is concerned about comments and certain style constraints though - a
astonishing-jelly-60479
09/08/2015, 6:33 PMand couldn't find it - b
bored-art-40741
09/08/2015, 6:33 PMHuh - a
astonishing-jelly-60479
09/08/2015, 6:33 PMyeah the style constraints are what prompted the modifications but unfortunately comments get stripped at the moment - b
bored-art-40741
09/08/2015, 6:34 PMI don't see it on pypi either, but I know we depend on it - a
astonishing-jelly-60479
09/08/2015, 6:34 PMweird, maybe it's a search issue - b
bored-art-40741
09/08/2015, 6:34 PMpantsbuild.pants.contrib.buildgen==0.0.45 - b
bored-art-40741
09/08/2015, 6:34 PMI think it is a search issue - b
bored-art-40741
09/08/2015, 6:34 PMBecause that's the package, and it works for us - a
astonishing-jelly-60479
09/08/2015, 6:35 PMyeah, that worked. thanks. - a
astonishing-jelly-60479
09/08/2015, 6:36 PMwhere is the right place to put that requirement so pants itself picks it up? is that something that happens at bootstrap time? - b
bored-art-40741
09/08/2015, 6:46 PMHm - b
bored-art-40741
09/08/2015, 6:46 PMDo you run pants from a virtualenv? - b
bored-art-40741
09/08/2015, 6:46 PMWe just install the extra contrib packages we need into the virtualenv that pants lives in - b
bored-art-40741
09/08/2015, 6:47 PMAs part of our pants bootstrapping script - b
bored-art-40741
09/08/2015, 6:47 PMYou also need to indicate within your pants.ini that you want the plugin loaded - b
bored-art-40741
09/08/2015, 6:47 PMThough in this case I don't think you need that, since buildgen doesn't register any targets or tasks - b
bored-art-40741
09/08/2015, 6:47 PMStu: I also got isolated + name_hashing to finish a run. Noticeably slower (23m vs 11) - b
bored-art-40741
09/08/2015, 6:48 PMThough that's a strictly clean recompile - b
bored-art-40741
09/08/2015, 6:48 PMThe gains might all come from incremental - w
witty-crayon-22786
09/08/2015, 7:17 PM@bored-art-40741: 23m with name hashing, 11 without? - b
bored-art-40741
09/08/2015, 7:17 PMYup - w
witty-crayon-22786
09/08/2015, 7:17 PMone thing to check: do a clean-all before enabling name hashing - b
bored-art-40741
09/08/2015, 7:17 PMAnd honestly incrementals seem worse too