Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Any 'bake' command fails when using log4r v1.1.11 (log4r v1.1.10 works) #12

Closed
carmstrong opened this issue Jan 5, 2014 · 10 comments
Closed

Comments

@carmstrong
Copy link
Contributor

carmstrong@linuxmint-pc ~ $ sudo gem install stove --pre
Fetching: multipart-post-1.2.0.gem (100%)
Successfully installed multipart-post-1.2.0
Fetching: faraday-0.8.8.gem (100%)
Successfully installed faraday-0.8.8
Fetching: faraday_middleware-0.9.0.gem (100%)
Successfully installed faraday_middleware-0.9.0
Fetching: i18n-0.6.9.gem (100%)
Successfully installed i18n-0.6.9
Fetching: log4r-1.1.11.gem (100%)
Successfully installed log4r-1.1.11
Fetching: minitar-0.5.4.gem (100%)
Successfully installed minitar-0.5.4
Fetching: mixlib-log-1.6.0.gem (100%)
Successfully installed mixlib-log-1.6.0
Fetching: mixlib-authentication-1.3.0.gem (100%)
Successfully installed mixlib-authentication-1.3.0
Fetching: addressable-2.3.5.gem (100%)
Successfully installed addressable-2.3.5
Fetching: sawyer-0.5.2.gem (100%)
Successfully installed sawyer-0.5.2
Fetching: octokit-2.7.0.gem (100%)
Successfully installed octokit-2.7.0
Fetching: solve-0.8.2.gem (100%)
Successfully installed solve-0.8.2
Fetching: stove-2.0.0.beta.1.gem (100%)
Successfully installed stove-2.0.0.beta.1
Parsing documentation for multipart-post-1.2.0
Installing ri documentation for multipart-post-1.2.0
Parsing documentation for faraday-0.8.8
Installing ri documentation for faraday-0.8.8
Parsing documentation for faraday_middleware-0.9.0
Installing ri documentation for faraday_middleware-0.9.0
Parsing documentation for i18n-0.6.9
Installing ri documentation for i18n-0.6.9
Parsing documentation for log4r-1.1.11
Installing ri documentation for log4r-1.1.11
Parsing documentation for minitar-0.5.4
Installing ri documentation for minitar-0.5.4
Parsing documentation for mixlib-log-1.6.0
Installing ri documentation for mixlib-log-1.6.0
Parsing documentation for mixlib-authentication-1.3.0
Installing ri documentation for mixlib-authentication-1.3.0
Parsing documentation for addressable-2.3.5
Installing ri documentation for addressable-2.3.5
Parsing documentation for sawyer-0.5.2
Installing ri documentation for sawyer-0.5.2
Parsing documentation for octokit-2.7.0
Installing ri documentation for octokit-2.7.0
Parsing documentation for solve-0.8.2
Installing ri documentation for solve-0.8.2
Parsing documentation for stove-2.0.0.beta.1
Installing ri documentation for stove-2.0.0.beta.1
13 gems installed

carmstrong@linuxmint-pc ~ $ bake
ERROR `warn' is not a valid Log Level!
/var/lib/gems/2.0.0/gems/stove-2.0.0.beta.1/lib/stove/mixins/loggable.rb:19:in `initialize': wrong number of arguments (1 for 0) (ArgumentError)
    from /var/lib/gems/2.0.0/gems/stove-2.0.0.beta.1/lib/stove/mixins/loggable.rb:19:in `new'
    from /var/lib/gems/2.0.0/gems/stove-2.0.0.beta.1/lib/stove/mixins/loggable.rb:19:in `log'
    from /var/lib/gems/2.0.0/gems/stove-2.0.0.beta.1/lib/stove/mixins/loggable.rb:28:in `log'
    from /var/lib/gems/2.0.0/gems/stove-2.0.0.beta.1/lib/stove/cli.rb:45:in `rescue in execute!'
    from /var/lib/gems/2.0.0/gems/stove-2.0.0.beta.1/lib/stove/cli.rb:53:in `execute!'
    from /var/lib/gems/2.0.0/gems/stove-2.0.0.beta.1/bin/bake:4:in `<top (required)>'
    from /usr/local/bin/bake:23:in `load'
    from /usr/local/bin/bake:23:in `<main>'

carmstrong@linuxmint-pc ~ $ sudo gem uninstall log4r

You have requested to uninstall the gem:
    log4r-1.1.11

stove-2.0.0.beta.1 depends on log4r (~> 1.1)
If you remove this gem, these dependencies will not be met.
Continue with Uninstall? [yN]  y
Successfully uninstalled log4r-1.1.11
.
carmstrong@linuxmint-pc ~ $ sudo gem install log4r -v 1.1.10
Fetching: log4r-1.1.10.gem (100%)
Successfully installed log4r-1.1.10
Parsing documentation for log4r-1.1.10
Installing ri documentation for log4r-1.1.10
1 gem installed

carmstrong@linuxmint-pc ~ $ bake --help
Usage: bake x.y.z

Actions:
        --[no-]bump                  Perform a version bump the local version automatically
        --[no-]changelog             Generate and prompt for a CHANGELOG
        --[no-]dev                   Bump a minor version release for development purposes
        --[no-]upload                Upload the cookbook to the community site

Plugins:
        --[no-]git                   Tag and push to a git remote
        --[no-]github                Publish the release to GitHub
        --[no-]jira                  Resolve JIRA issues

Global Options:
        --locale [LANGUAGE]          Change the language to output messages
        --log-level [LEVEL]          Set the log verbosity
        --category [CATEGORY]        Set category for the cookbook
        --path [PATH]                Change the path to a cookbook
        --remote [REMOTE]            The name of the git remote to push to
        --branch [BRANCH]            The name of the git branch to push to
    -h, --help                       Show this message
    -v, --version                    Show version
@carmstrong
Copy link
Contributor Author

Looks like others are seeing this too: colbygk/log4r#24 (comment)

@sethvargo
Copy link
Contributor

Sigh. Well, not only did they break semver, but they changed from GLP to BSD. Any suggestions for log4r alternatives?

@ghost
Copy link

ghost commented Jan 5, 2014

@sethvargo I been using Yell recently.

@sethvargo
Copy link
Contributor

@carmstrong after some rants, they've yanked the latest version, so stove should continue working. I'm going to leave this open an investigate switching to something else (like Yell).

@carmstrong
Copy link
Contributor Author

@sethvargo sounds good. Maybe it's worth adding a version constraint < 1.1.11 in case they break it again?

@carmstrong
Copy link
Contributor Author

@mitchellh did in hashicorp/vagrant@177bfc1

@sethvargo
Copy link
Contributor

@carmstrong if you'd like to issue a PR for that, I'd accept it.

@carmstrong
Copy link
Contributor Author

@sethvargo #13

@sethvargo
Copy link
Contributor

This was "fixed" in 892e5d4, but leaving open because reasons I said above.

@sethvargo
Copy link
Contributor

@carmstrong I thought about this more, and I really like log4r. I tried yell, and I wasn't super happy with the DSL. I'm going to stick to log4r.

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

No branches or pull requests

2 participants