pantsbuild #general

$ ./pants -ldebug test src/js/dashboard:unit
ERROR] Invalid scope [cache.resolve.npm] in pants-internal.ini
Exception caught: (<class 'pants.option.config.ConfigValidationError'>)
  File "/Users/pl/code/foursquare.web/.pantsenv/bin/pants", line 11, in <module>
    sys.exit(main())
  File "/Users/pl/code/foursquare.web/.pantsenv/lib/python2.7/site-packages/pants/bin/pants_exe.py", line 44, in main
    PantsRunner(exiter).run()
  File "/Users/pl/code/foursquare.web/.pantsenv/lib/python2.7/site-packages/pants/bin/pants_runner.py", line 53, in run
    options_bootstrapper=options_bootstrapper)
  File "/Users/pl/code/foursquare.web/.pantsenv/lib/python2.7/site-packages/pants/bin/pants_runner.py", line 43, in _run
    return LocalPantsRunner(exiter, args, env, options_bootstrapper=options_bootstrapper).run()
  File "/Users/pl/code/foursquare.web/.pantsenv/lib/python2.7/site-packages/pants/bin/local_pants_runner.py", line 49, in run
    self._maybe_profiled(self._run)
  File "/Users/pl/code/foursquare.web/.pantsenv/lib/python2.7/site-packages/pants/bin/local_pants_runner.py", line 46, in _maybe_profiled
    runner()
  File "/Users/pl/code/foursquare.web/.pantsenv/lib/python2.7/site-packages/pants/bin/local_pants_runner.py", line 67, in _run
    options_bootstrapper.verify_configs_against_options(options)
  File "/Users/pl/code/foursquare.web/.pantsenv/lib/python2.7/site-packages/pants/option/options_bootstrapper.py", line 186, in verify_configs_against_options
    raise Config.ConfigValidationError("Invalid config entries detected. "

Exception message: Invalid config entries detected. See log for details on which entries to update or remove.
(Specify --no-verify-config to disable this check.)

witty-crayon-22786

05/16/2016, 8:29 PM

well yea, it's old

witty-crayon-22786

05/16/2016, 8:29 PM

an old ticket

bored-art-40741

05/16/2016, 8:30 PM

Oh it's

node

now

witty-crayon-22786

05/16/2016, 8:30 PM

think the resolver is at

resolve.node

now

bored-art-40741

05/16/2016, 8:30 PM

In any case, disabling caching doesn't fix it for me unfortunately

bored-art-40741

05/16/2016, 8:31 PM

Just reading the code, it seems like

Copy code

def _can_resolve_target(self, target):
    """Returns whether this is a NodePackage and there a resolver registerd for its subtype.

    :param target: A Target
    :rtype: Boolean
    """
    return self.is_node_package(target) and self._resolver_for_target(target) != None

bored-art-40741

05/16/2016, 8:32 PM

Suggests that it's not going to a resolve for

node_module

targets

bored-art-40741

05/16/2016, 8:32 PM

Oh nevermind

bored-art-40741

05/16/2016, 8:32 PM

it's a subclass

witty-crayon-22786

05/16/2016, 8:32 PM

hm... looks like none of our internal usages have

devDependencies

witty-crayon-22786

05/16/2016, 8:32 PM

which is... scary

witty-crayon-22786

05/16/2016, 8:33 PM

and they all have karma in the

dependencies

list.

witty-crayon-22786

05/16/2016, 8:33 PM

(again, will wait for chris to chime in.)

bored-art-40741

05/16/2016, 8:34 PM

Moving everything to

dependencies

to iterate

bored-art-40741

05/16/2016, 8:34 PM

No dice though

bored-art-40741

05/16/2016, 8:41 PM

Copy code

$ ls /Users/pl/code/foursquare.web/.pants.d/resolve/node/252d64521cf9/src.js.dashboard.foo/current
node_modules  package.json  src  test  webpack.config.js

4sq [~]:
[!41440]$ ls /Users/pl/code/foursquare.web/.pants.d/resolve/node/252d64521cf9/src.js.dashboard.foo/current/node_modules/

4sq [~]:
[!41441]$ ls /Users/pl/code/foursquare.web/.pants.d/resolve/node/252d64521cf9/src.js.dashboard.foo/current/package.json
/Users/pl/code/foursquare.web/.pants.d/resolve/node/252d64521cf9/src.js.dashboard.foo/current/package.json

4sq [~]:
[!41442]$ cat /Users/pl/code/foursquare.web/.pants.d/resolve/node/252d64521cf9/src.js.dashboard.foo/current/package.json
{
  "name": "foo",
  "repository": "",
  "author": "Your name here",
  "private": true,
  "version": "0.0.1",
  "dependencies": {},
  "scripts": {
    "test:watch": "karma start --autoWatch=true --singleRun=false",
    "lint": "eslint ./src",
    "start": "webpack --watch",
    "build": "webpack; copyfiles -f dist/*.js ../../../dist",
    "clean": "rimraf dist/*",
    "test": "karma start"
  },
  "keywords": [],
  "main": "",
  "description": "foo"
}

bored-art-40741

05/16/2016, 8:41 PM

My claim is that it doesn't merge dependencies; rather it stomps them

witty-crayon-22786

05/16/2016, 8:42 PM

how many dependencies are in the list for that module? or is it all devDependencies

bored-art-40741

05/16/2016, 8:43 PM

I merged dev and regular dependencies into just

dependencies

bored-art-40741

05/16/2016, 8:43 PM

And "lots"

bored-art-40741

05/16/2016, 8:43 PM

Like ~20+

bored-art-40741

05/16/2016, 8:43 PM

Also, argh,

pushd

bored-art-40741

05/16/2016, 8:43 PM

We should not be changing the cwd of the pants process!

bored-art-40741

05/16/2016, 8:44 PM

It's shelling out anyway, if necessary it can change the

cwd

of the subproc