Skip to content
This repository has been archived by the owner on Dec 7, 2018. It is now read-only.

Dcell on Heroku and lsb_release -d causes catastrophic failure #30

Merged

Conversation

halorgium
Copy link
Contributor

I have forked a copy of the ffi-rzmq gem and added in some Heroku (via bash) compiled zeromq libraries:

https://github.com/jsgoecke/ffi-rzmq/commit/ae88b6bc8f5b6f28017385777068a2c206065b37

But when I try to kick off the process on Heroku I am getting an error from this line:

https://github.com/celluloid/dcell/blob/master/lib/dcell/info_service.rb#L48

Here is the error itself:

https://gist.github.com/73cdfcd534700f073af2

This causes a catastrophic failure of the application. Would be great if it gave a warning instead.

@halorgium
Copy link
Contributor

@jsgoecke that gist seems to have evaporated.
I am guessing heroku didn't feel like installing this package: http://packages.ubuntu.com/lucid/lsb-release

Could confirm the error was probably related to this?

$ heroku run lsb_release -d
Running `lsb_release -d` attached to terminal... up, run.5369
bash: lsb_release: command not found

@tarcieri
Copy link
Member

tarcieri commented Apr 7, 2013

@halorgium it's probably related to the info service. I believe it may be fixed now?

@halorgium
Copy link
Contributor

@tarcieri definitely. but if lsb_release does not exist, the String matching will fail most likely.

@halorgium
Copy link
Contributor

@jsgoecke on a related note, i would recommend simply installing some .debs directly into your slug using a buildpack.

It would be awesome to have a DCell buildpack combined with my spiped buildpack/app for DCell inside @heroku!

@coveralls
Copy link

Coverage Status

Coverage increased (+39.72%) when pulling 434a5d0 on halorgium:safe-distribution-in-info-service into c2a70a8 on celluloid:master.

halorgium added a commit that referenced this pull request Apr 20, 2013
Dcell on Heroku and `lsb_release -d` causes catastrophic failure
@halorgium halorgium merged commit 7ca760a into celluloid:master Apr 20, 2013
@halorgium halorgium deleted the safe-distribution-in-info-service branch April 20, 2013 11:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants