marcelog / logger_logstash_backend

Logstash backend for the Elixir Logger
Apache License 2.0
73 stars 40 forks source link

:exjsx required in applications? #13

Open taktran opened 8 years ago

taktran commented 8 years ago

I tried deploying logger_logstash_backend, but I get the error message

-----> Generating release
using mix to generate release
Building release with MIX_ENV=prod.

You have dependencies (direct/transitive) which are not in :applications!
The following apps should be added to :applications in mix.exs:

        logger_logstash_backend => logger_logstash_backend is missing from cms

To fix it, I had to add

def application do
  [applications: [:logger, :logger_logstash_backend, :exjsx]]
end

along with the dep

def deps do
  [{:logger_logstash_backend, "~> 2.1.1"}]
end

It doesn't seem like tzdata (https://github.com/marcelog/logger_logstash_backend/issues/5) is needed anymore?

I can do a PR to fix this if you want

taktran commented 8 years ago

Note: :exjsx is missing from the mix.exs in logger_logstash_backend too

ahmadferdous commented 7 years ago

I faced the same issue while trying to prepare release package with Distillery. The problem is Elixir version requirement in logger_logstash_backend is still 1.3. It's only Elixir v1.4 and up that automatically infers :applications from deps. But being tied to v1.3 prevent that from happening.

marcelog commented 7 years ago

Can you guys try the new release? Elixir 1.5 is supported (1.3 is still the base requirement tho') and exjsx was added to the applications list