Add libffi to support building chefdk on debian #164
Conversation
sethvargo
changed the title
| @@ -46,6 +46,7 @@ | |||
| override :yajl, version: "1.2.0" | |||
| override :zlib, version: "1.2.8" | |||
|
|
|||
| dependency "libffi" if platform == 'debian' | |||
There was a problem hiding this comment.
You can actually just say
dependency "libffi" if debian?on more recent versions of Omnibus 😄
|
Even tough we don't have plans of supporting chefdk on debian soon, I'm cool with merging this. 👍 /cc: @opscode/client-eng |
|
Hrmmmm actually I think we should rather have this dependency @ https://github.com/opscode/omnibus-software/blob/master/config/software/chefdk.rb |
|
Yeah it would be better to fix the software definition that is causing the libffi dep. We might be able to --disable-libffi in the software definition to make it go away entirely, and if we can't then the dependency should still definitely go in the software defn. The fact that it only happens on debian is also a bit weird, so all the context of the health_check failure would be useful. Regardless this isn't the right spot for the patch, needs to go into omnibus-software. |
On debian the health check fails on libffi, so add it as a dependency when building on debian.