spox / batali

Light weight cookbook resolver
https://spox.github.io/batali
Other
32 stars 6 forks source link

The case of the missing DEBUG #89

Closed webframp closed 7 years ago

webframp commented 8 years ago

As discussed in irc, seen with batali 0.4.10 on Ubuntu 14.04 and Windows 10

$ export DEBUG=true
$ batali install
[Batali]: Readying installation destination... complete!
[Batali]: Installing cookbooks... complete!

and on Windows:

$ $env:DEBUG="true"
$ batali install
[Batali]: Readying installation destination... complete!
[Batali]: Installing cookbooks... complete!
webframp commented 8 years ago

Overrode the Gemfile to test out the changes,

Gemfile:

gem 'bogo-cli', git: 'https://github.com/spox/bogo-cli', ref: 'fix/ui-config'
...

bundle install:

Using bogo-cli 0.2.9 (was 0.2.8) from https://github.com/spox/bogo-cli (at fix/ui-@04f520a)

same commands as earlier.

Just to clarify what I think I would see:

https://github.com/spox/batali/blob/master/lib/batali/command/install.rb#L26

seems like it should report unit resolution and I should see the dep graph being walked when debug is enabled via $env:DEBUG='true'

so far testing with changes from spox/bogo-cli#10 I haven't seen that but let me know any other info I can provide.

webframp commented 7 years ago

Should have checked here first. I knew this seemed familiar.

Digging around here didn't help me too much: https://github.com/spox/bogo-ui/blob/master/lib/bogo-ui/ui.rb#L113

webframp commented 7 years ago

Can confirm from testing this is fixed when pointing at: https://github.com/spox/bogo-cli/tree/develop

chrisroberts commented 7 years ago

Fixed via #91. Thanks for confirmation @webframp !