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

ruby: remove 1.9 (WIP) #18385

Closed
wants to merge 3 commits into from
Closed

ruby: remove 1.9 (WIP) #18385

wants to merge 3 commits into from

Conversation

vrthra
Copy link
Member

@vrthra vrthra commented Sep 7, 2016

Motivation for this change

Ruby 1.9 is EOLd by upstream

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • OS X
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Remove ruby 1.9 since it is EOLed upstream
@mention-bot
Copy link

@vrthra, thanks for your PR! By analyzing the annotation information on this pull request, we identified @globin, @pikajude and @Phreedom to be potential reviewers

@joachifm
Copy link
Contributor

joachifm commented Sep 8, 2016

cc @zimbatm

, name ? "sdformat-${version}"
, version ? "4.0.0" # versions known to work with this expression include 3.7.0
, srchash-sha256 ? "b0f94bb40b0d83e35ff250a7916fdfd6df5cdc1e60c47bc53dd2da5e2378163e"
, ...
}:

let
ruby = ruby_1_9;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I bet there is a reason why it had been set to ruby 1.9. Did you test this library?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe @pxc knows. He's the implicit maintainer as the gazebo package is the only one depending on this lib.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I only checked that it compiled fine. I will check this and get back.

@zimbatm
Copy link
Member

zimbatm commented Sep 8, 2016

Good idea. 👍

@vrthra vrthra changed the title ruby: remove 1.9 ruby: remove 1.9 (WIP) Sep 8, 2016
@fpletz
Copy link
Member

fpletz commented Sep 25, 2016

Any news here?

@vrthra
Copy link
Member Author

vrthra commented Sep 25, 2016

@fpletz I am currently on a slow and sporadic internet connection (this month and for the next month too). Hence I have been unable to move forward on this. I changed the title to WIP, and can close it for now and reopen it later if that is better.

@matthiasbeyer
Copy link
Contributor

Any progress?

@fpletz fpletz added this to the 17.03 milestone Jan 30, 2017
@zimbatm
Copy link
Member

zimbatm commented Jan 30, 2017

Rebased and merged into master. sdformat is marked as broken until @pxc fixes it.

@zimbatm zimbatm closed this Jan 30, 2017
@pxc
Copy link

pxc commented Jan 30, 2017

Not sure which user you want to notify @zimbatm but it's not me ;-)

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

Successfully merging this pull request may close these issues.

None yet

8 participants