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

METEOR_LOCAL_DIR. (Fixing #6532) #6760

Closed
wants to merge 3 commits into from
Closed

METEOR_LOCAL_DIR. (Fixing #6532) #6760

wants to merge 3 commits into from

Conversation

@laosb
Copy link
Collaborator

@laosb laosb commented Apr 8, 2016

#6532

I'm not sure if providing this env should also override the options.projectLocalDir, which I think is provided when use meteor test and fire an instance as a mirror. If I do override it in just like what I did, meteor test may not work as expected when METEOR_LOCAL_DIR is provided.

@laosb laosb changed the title METEOR_LOCAL_DIR. Fixing #6532 METEOR_LOCAL_DIR. (Fixing #6532) Apr 9, 2016
@laosb
Copy link
Collaborator Author

@laosb laosb commented Apr 13, 2016

Seems no one is looking at Pull Requests. Pinging @stubailo @tmeasday

@laosb
Copy link
Collaborator Author

@laosb laosb commented Apr 29, 2016

Is there anyone who can review? @meteor

@stubailo stubailo self-assigned this Apr 29, 2016
@stubailo
Copy link
Contributor

@stubailo stubailo commented Apr 29, 2016

@laosb yeah, as @tmeasday was talking about at Meteor Night last night, we are trying to switch to a more scalable process. Since I told you to make this PR, I should have looked at it - going to do that now!

@stubailo
Copy link
Contributor

@stubailo stubailo commented Apr 29, 2016

@laosb can you add entries about this to History.md? Otherwise this feature will be impossible to discover.

@laosb
Copy link
Collaborator Author

@laosb laosb commented Apr 30, 2016

Glad to see what I can help. Will do it soon on my computer 😀

Sent from Nylas N1, the extensible, open source
mail client.

On 4月 30 2016, at 1:14 凌晨, Sashko Stubailo <notifications@github.com>
wrote:

@laosb can you add entries about this to
History.md? Otherwise this feature will be impossible to discover.


You are receiving this because you were mentioned.
Reply to this email directly or [view it on GitHub](https://github.com/meteor/
/pull/6760#issuecomment-215818425)![](https://github.com/notifications/b
eacon/ACbWbYR5xh0O1Eq1jYAn-uuCjZ9x6z5jks5p8jyOgaJpZM4IDd0t.gif)

@laosb
Copy link
Collaborator Author

@laosb laosb commented Apr 30, 2016

@stubailo The History.md in this branch is not updated to devel. Would you like my rebase? Or you can add it then.

laosb added 2 commits May 2, 2016
@laosb
Copy link
Collaborator Author

@laosb laosb commented May 2, 2016

@stubailo
Copy link
Contributor

@stubailo stubailo commented May 2, 2016

Lol, in the History file you put a different env var name :] going to fix and merge

@stubailo
Copy link
Contributor

@stubailo stubailo commented May 2, 2016

Confirmed that this does indeed let you run multiple instances of the same app, and merged as one commit! bfdca2a

@stubailo stubailo closed this May 2, 2016
@aadamsx
Copy link

@aadamsx aadamsx commented Jul 6, 2017

On Meteor 1.5 I get an error when trying to run two instances of Meteor, on two different ports, out of the same directory.

// package.json:

{
  "name": "dev",
  "version": "1.0.0",
  "private": true,
  "scripts": {
    "admin": "meteor --settings .config/development/settings.json --port 8080",
    "client": "meteor --settings .config/development/settings.json --port 8081",
...

The mondo server is run from localhost:3001 as usual.

I run the command npm run admin and then npm run client in two different shells.

They both need to talk to the same DB.

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

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.