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

#1037 fixed ruby depdendencies #1038

Merged
merged 1 commit into from
Mar 4, 2016
Merged

Conversation

original-brownbear
Copy link
Contributor

#1037 is fixed by this.

  • Locked in all Gemfile dependencies
  • Locked in all related apt-get installed package dependencies
    • Removed apt-get upgrade run from Dockerfile to make the above have any meaning
  • Removed redundant bundle install from .rultor.yml install section given that this is also run from Maven (site target) anyhow.
    • Also no need for showing the Jekyll version in the logs, we can read this from the bundler output anyhow.

* Locked in all Gemfile dependencies
* Locked in all related apt-get installed package dependencies
 * Removed apt-get upgrade run from Dockerfile to make the above have any meaning
* Removed redundant bundle install from .rultor.yml install section given that this is also run from Maven (site target) anyhow.
@original-brownbear
Copy link
Contributor Author

@alex-palevsky this is invalid.

@original-brownbear
Copy link
Contributor Author

@rultor merge please.

@rultor
Copy link
Collaborator

rultor commented Mar 3, 2016

@rultor merge please.

@original-brownbear OK, I'll try to merge now. You can check the progress of the merge here

@rultor
Copy link
Collaborator

rultor commented Mar 3, 2016

@rultor merge please.

@original-brownbear Oops, I failed. You can see the full log here (spent 4min)

Using kramdown 1.10.0
Using liquid 2.6.3
Using mercenary 0.3.5
Using posix-spawn 0.3.11
Using yajl-ruby 1.2.1
Using redcarpet 3.3.4
Using safe_yaml 1.0.4
Using trollop 2.0
Using mime-types 2.99.1
Using mini_portile2 2.0.0
Using uuidtools 2.1.5
Using bundler 1.11.2
Using parslet 1.5.0
Using classifier-reborn 2.0.4
Using coffee-script 2.4.1
Using css_press 0.3.2
Using uglifier 2.7.2
Using faraday 0.9.2
Using rb-inotify 0.9.7
Using jekyll-sass-converter 1.4.0
Using pygments.rb 0.6.3
Using mail 2.6.3
Using nokogiri 1.6.7.2
Using toml 0.1.2
Using jekyll-coffeescript 1.0.1
Using multi_css 0.1.0
Using multi_js 0.1.0
Using sawyer 0.6.0
Using listen 3.0.6
Using html_press 0.8.2
Using octokit 4.2.0
Using jekyll-watch 1.3.1
Using jekyll-gist 1.4.0
Using jekyll 2.4.0
Using jekyll-plantuml 1.0.2
Using jekyll-press 0.2.1
Using jekyll-sass 1.2.2
Using jgd 1.3
\u001b[32mBundle updated!\u001b[0m
+ bundle show jekyll
/var/lib/gems/2.2.0/gems/jekyll-2.4.0
+ jekyll --version
/var/lib/gems/2.2.0/gems/safe_yaml-1.0.4/lib/safe_yaml/psych_resolver.rb:4:in `<class:PsychResolver>': uninitialized constant Psych::Nodes (NameError)
    from /var/lib/gems/2.2.0/gems/safe_yaml-1.0.4/lib/safe_yaml/psych_resolver.rb:2:in `<module:SafeYAML>'
    from /var/lib/gems/2.2.0/gems/safe_yaml-1.0.4/lib/safe_yaml/psych_resolver.rb:1:in `<top (required)>'
    from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:69:in `require'
    from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:69:in `require'
    from /var/lib/gems/2.2.0/gems/safe_yaml-1.0.4/lib/safe_yaml/load.rb:131:in `<module:SafeYAML>'
    from /var/lib/gems/2.2.0/gems/safe_yaml-1.0.4/lib/safe_yaml/load.rb:26:in `<top (required)>'
    from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:69:in `require'
    from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:69:in `require'
    from /var/lib/gems/2.2.0/gems/jekyll-3.1.2/lib/jekyll.rb:28:in `<top (required)>'
    from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:69:in `require'
    from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:69:in `require'
    from /var/lib/gems/2.2.0/gems/jekyll-3.1.2/bin/jekyll:6:in `<top (required)>'
    from /usr/local/bin/jekyll:23:in `load'
    from /usr/local/bin/jekyll:23:in `<main>'
container 6e05be7a7d49ed59863c860bc92b3010beca046d6e91237bf188bf8cbb8cdc56 is dead
Thu Mar  3 08:29:02 UTC 2016

@original-brownbear
Copy link
Contributor Author

@yegor256 I need you here to manually merge this. See #1039 for the detailed reason.

This currently is stuck otherwise and so are all other PRs in this project.

@alex-palevsky
Copy link
Contributor

@original-brownbear Thanks for your pull request, let me find someone who can review it

@original-brownbear
Copy link
Contributor Author

@alex-palevsky no review needed only @yegor256 's help! :)

@alex-palevsky
Copy link
Contributor

@HDouss could you review this pull request

@alex-palevsky
Copy link
Contributor

@alex-palevsky this is invalid.

@original-brownbear right, I added "invalid" label

@original-brownbear
Copy link
Contributor Author

@yegor256 sorry not meaning to be pushy here, just in case this got lost with all the messages above :)

You need to manually merge this one to master because of #1039 I'm afraid.

yegor256 pushed a commit that referenced this pull request Mar 4, 2016
@yegor256 yegor256 merged commit c610cb5 into yegor256:master Mar 4, 2016
@yegor256
Copy link
Owner

yegor256 commented Mar 4, 2016

@original-brownbear done

@original-brownbear
Copy link
Contributor Author

@yegor256 thanks!

@original-brownbear
Copy link
Contributor Author

@rultor deploy

@rultor
Copy link
Collaborator

rultor commented Mar 4, 2016

@rultor deploy

@original-brownbear OK, I'll try to deploy now. You can check the progress here

@rultor
Copy link
Collaborator

rultor commented Mar 4, 2016

@rultor deploy

@original-brownbear Done! FYI, the full log is here (took me 8min)

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

Successfully merging this pull request may close these issues.

4 participants