Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow two pig.json files to reference each other without Maximum call stack size exceeded error #23

Open
KylePalko opened this issue May 21, 2016 · 3 comments

Comments

@KylePalko
Copy link

I ran into this issue when I had an API Server that I wanted to spin up with my App Server (and vice versa)

/app/pig.json

{
  "include": {
    "../api/pig.json": "api"
  }
}

/api/pig.json

{
  "include": {
    "../app/pig.json": "app"
  }
}

This will cause the error below

RangeError: Maximum call stack size exceeded
    at fromJsonFile (/usr/local/lib/node_modules/docker-pig/lib/config.js:25:59)
    at Object.module.exports.fromFile (/usr/local/lib/node_modules/docker-pig/lib/config.js:11:16)
    at /usr/local/lib/node_modules/docker-pig/lib/config.js:49:38
    at Function.forOwn (/usr/local/lib/node_modules/docker-pig/node_modules/lodash/dist/lodash.js:2106:15)
    at resolveIncludes (/usr/local/lib/node_modules/docker-pig/lib/config.js:48:7)
    at Object.module.exports.fromFile (/usr/local/lib/node_modules/docker-pig/lib/config.js:18:12)
    at /usr/local/lib/node_modules/docker-pig/lib/config.js:49:38
@KylePalko
Copy link
Author

Btw, thanks for this. It's exactly what I needed! I may try to get involved in the project once I understand the context. I wouldn't mind making this change if you'd like.

@mnylen
Copy link
Owner

mnylen commented May 23, 2016

Hi!

Sorry, I'm not maintaining pig actively at the moment. You should try using docker-compose or similar tool instead. pig has last been tested with docker 1.4.0 (or was it 1.5.0).

@KylePalko
Copy link
Author

Ah, okay. Thanks. Was there a specific reason you stopped supporting it? It has some added features that Docker Compose doesn't yet have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants