Skip to content
This repository was archived by the owner on Jan 16, 2018. It is now read-only.

Conversation

@DaanVanVugt
Copy link

This fixes a bug with meteor-deploy on my machine, related to https://nodejs.org/api/child_process.html#child_process_event_exit

@xenolf
Copy link
Owner

xenolf commented Mar 11, 2015

Hey there,
thanks for the pull request!

Changing the exit signal is fine by me.
In the other change you are relying on meteor server to preprocess your scss for 10 seconds I'm guessing?

@DaanVanVugt
Copy link
Author

Hi Xenolf,

That is to work around a bug with ionicons-sass (in meteoric/meteor-ionic).
The build process always crashes the first time because those files are not
there, because they are placed there later in the build process. This works
around that, but should not be necessary once they change the dependency
handling in the build process. I linked to the bug reports in the source.

Best,
Daan

ps. Due to some other issues on debian I decided to move to a capistrano
script, which seems to work nicely as well

On Wed, Mar 11, 2015 at 9:14 PM, xenolf [email protected] wrote:

Hey there,
thanks for the pull request!

Changing the exit signal is fine by me.
In the other change you are relying on meteor server to preprocess your
scss for 10 seconds I'm guessing?


Reply to this email directly or view it on GitHub
#4 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants