YAML dependency? #502

Closed
oylenshpeegul opened this Issue May 5, 2011 · 6 comments

3 participants

@oylenshpeegul

I did

cpanm Dancer

and it wouldn't work until I also did

cpanm YAML

Does Dancer have a dependency on YAML, yet not insist on it being installed?

@xsawyerx
PerlDancer member

Dancer doesn't depend on YAML, nor does it insist on it being installed.

Could you show us your build.log file?

@bigpresh
PerlDancer member

By "it wouldn't work", do you mean that a scaffolded app didn't read its config, or that Dancer itself didn't install?

As @xsawyerx says, Dancer does not depend upon YAML; it is recommended though, so that config files can be used. (The reason it's not an outright dependency is that you can easily write Dancer apps as a single file with no config.yml file needed.)

If the problem was just that a scaffolded app couldn't read its config, I recently pushed 52a5dd4 which causes the scaffolding script (dancer -a) to show a prominent warning if YAML is not installed.

@xsawyerx
PerlDancer member

If you could provide a build.log that shows a problem with installing Dancer without YAML, and not running Dancer, we could investigate.

Otherwise, I suggest we close this ticket.

@oylenshpeegul

Yes, sorry for the lack of detail. In hindsight, I am embarrassed to have written "it wouldn't work" in a bug report. I guess I was thinking I had found a simple missing dependency and was not in bug report mode. Again, sorry.

Yes, I meant "dancer -a" neither complained nor produced a working app. I was following along at DotCloud, so I guess all that is needed (in addition to the warning bigpresh mentions above) is the insertion of

cpanm YAML

between the "cpanm Dancer" and the "dancer -a" in those instructions. I will contact them

Thanks! Dancer works great on DotCloud. Sorry for the crappy ticket!

@bigpresh
PerlDancer member

No problems at all, thanks for getting back to us!

The commit I mentioned earlier should be included in the next stable release of Dancer; it will cause the scaffolding script (dancer -a) to emit a clear warning message if YAML is not available.

@xsawyerx
PerlDancer member

@oylenshpeegul, no worries, and thanks for letting us know! :)

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