-
Notifications
You must be signed in to change notification settings - Fork 586
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
Handle METEOR_PACKAGE_DIRS #188
Comments
Did you see http://docs.vulcanjs.org/#Git-Based-Deploys ? Unless I'm wrong, Heroku deploys work by pushing your git code to the server first, the buildpack only takes over then. In that context |
Hi @SachaG , I think it is a bit more complex on Heroku, I think you don't simply push your app on the server and run it. E.g your app is built in a The submodule (here Vulcan libs) is correctly pushed to Heroku along with the app in I'll share a doc as soon as I get it working. But of course maybe it is not related to buildpack, I may miss something else. |
Don't you deploy with |
Yes I deploy |
Well if you deploy with |
Yep the submodules are indeed commited, Heroku loads them according to the It's simply that the I'll try with a relative path instead instead asap. |
Yes but what does |
The submodule is my fork of Vulcan, containing slightly patched libs (e.g to fix the CORS issue with Cordova). So Heroku clone this fork automatically along with my app, and |
Hi, I drafted a 2-repo deployment for Heroku. However, it does not work yet, I think that the `METEOR_PACKAGE_DIRS` is incorrect though I can't fix it, as [described in this issue](https://github.com/AdmitHub/meteor-buildpack-horse/issues/188](https://github.com/AdmitHub/meteor-buildpack-horse/issues/188 ) Also, I have no idea how you would handle a private repo for the custom Vulcan packages. However most of the time you'll use this kind of install either to enjoy the latest PR before they are merged into Vulcan or on the opposite to keep old versions of the packages, so I guess very few people will need to hide their packages code in a second private repo, they'll use the usual install process instead and put those private files directly into their app code.
@eric-burel I noticed you've brought up this issue on several buildpack repos. I have the exact same issue right now where I have a local version of a meteor package in my app under the packages directory. When I try deploying to Heroku, I keep getting an error that there is an unknown package in top-level dependencies which I'm guessing is coming from meteor not recognizing the local package. Had you found a solution to this issue? Alternatively, is there a platform to deploy to other than Heroku that handles this better? |
Type of problem
Summary
Contents of
.meteor/release
Buildpack version
Deployment target
Heroku
Hi,
I can't have
METEOR_PACKAGE_DIRS
to be taken into account during app build/run. It is set to/app/.submodules/my-repo/packages
and the folder exist, I checked out in the bash.However, it behaves as if ignored, I guess during build the path is different (e.g I should write
/tmp/.submodules/...
instead of/app
or simply./
, could not test yet) but I am not sure and I can't find a solution. Maybe the buildpack is a good place to handle this ?The text was updated successfully, but these errors were encountered: