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

Prematurely dropped Ruby 1.8 compatibility #333

Closed
mislav opened this issue Sep 19, 2014 · 8 comments
Closed

Prematurely dropped Ruby 1.8 compatibility #333

mislav opened this issue Sep 19, 2014 · 8 comments

Comments

@mislav
Copy link

mislav commented Sep 19, 2014

Dir.home added in 34696d4 is not compatible with Ruby 1.8, which makes requiring simplecov in tests fail even though we were promised that no-op compatibility with 1.8 will be kept until 1.0.0:

Starting with SimpleCov 1.0.0, even no-op compatibility with Ruby <= 1.8 will be entirely dropped.
@sferik sferik closed this as completed in 9afdb46 Sep 20, 2014
@sferik
Copy link
Collaborator

sferik commented Sep 20, 2014

I don’t have gem push access.

@colszowka Can you please push a patch release to resolve this issue?

@bf4
Copy link
Collaborator

bf4 commented Sep 20, 2014

@sferik What's your rubygems.org email? I'll add you

@bf4
Copy link
Collaborator

bf4 commented Sep 20, 2014

n/m for some reason I thought I had gem push. Anyhoo, a simple gem owner t gave me the answer, so gem owner simplecov -a $(gem owner t | tail -1 | cut -d' ' -f2) would work

@colszowka
Copy link
Collaborator

@bf4 @sferik Sorry guys, must have forgotten that. Added you as owners on rubygems, updated the Changelog and will ship 0.9.1 within the next minutes.

@colszowka
Copy link
Collaborator

@xaviershay Please give me your e-mail address on rubygems so I can add you as an owner as well :)

@mislav
Copy link
Author

mislav commented Sep 21, 2014

On Sun, Sep 21, 2014 at 9:52 AM, Benjamin Fleischer <
notifications@github.com> wrote:

SimpleCov depends on the Coverage module in 1.9+, how was it ever used in
1.8? Was this documented somewhere and I missed it?

SimpleCov does not work on 1.8, however it prints out a warning on stderr
about this and doesn't crash the process. This PR is about fixing SimpleCov
to not crash the test suite if used on 1.8.

@xaviershay
Copy link
Collaborator

pretty sure it's notreal at rhnh dot net

@colszowka
Copy link
Collaborator

@bf4 At the time I initially wrote SimpleCov, Ruby 1.8 was still widely used for many projects, and especially considering the use of simplecov for gems and other libraries that should work across ruby versions, it made sense to avoid simplecov failing entirely on 1.8, instead printing a warning and bailing. You can still see that in the start method. For 1.0 I want to remove that though, since nowadays 1.8 is becoming very outdated.

@xaviershay Added you!

sferik added a commit that referenced this issue Jan 23, 2015
jsonn pushed a commit to jsonn/pkgsrc that referenced this issue Feb 3, 2015
0.9.1, 2014-09-21 ([changes](simplecov-ruby/simplecov@v0.9.0...v0.9.1))
====================

## Bugfixes

 * In 0.9.0, we introduced a regression that made SimpleCov no-op mode fail on Ruby 1.8, while
   dropping 1.8 support altogether is announced only for v1.0. This has been fixed.
   See [#333](simplecov-ruby/simplecov#333) (thanks (@sferik)


0.9.0, 2014-07-17 ([changes](simplecov-ruby/simplecov@v0.8.2...v0.9.0))
====================

**A warm welcome and big thank you to the new contributors [@xaviershay](https://github.com/xaviershay), [@sferik](https://github.com/sferik) and especially [@bf4](https://github.com/bf4) for tackling a whole lot of issues and pull requests for this release!**

## Enhancements

  * New interface to specify multiple formatters.
    See [#317](simplecov-ruby/simplecov#317) (thanks @sferik)
  * Document in the README how to exclude code from coverage reports,
    and that the feature shouldn't be abused for skipping untested
    private code.
    See [#304](simplecov-ruby/simplecov#304)
  * Clarify Ruby version support.
    See [#279](simplecov-ruby/simplecov#279) (thanks @deivid-rodriguez)

## Bugfixes

  * Ensure calculations return Floats, not Fixnum or Rational. Fixes segfaults with mathn.
    See [#245](simplecov-ruby/simplecov#245) (thanks to @bf4)
  * Using `Kernel.exit` instead of exit to avoid uncaught throw :IRB_EXIT when
    exiting irb sessions.
    See [#287](simplecov-ruby/simplecov#287) (thanks @wless1)
    See [#285](simplecov-ruby/simplecov#285)
  * Does not look for .simplecov in ~/ when $HOME is not set.
    See [#311](simplecov-ruby/simplecov#311) (thanks @lasseebert)
  * Exit with code only if it's Numeric > 0.
    See [#302](simplecov-ruby/simplecov#303 @hajder)
  * Make default filter case insensitive.
    See [#280](simplecov-ruby/simplecov#280) (thanks @ryanatball)
  * Improve regexp that matches functional tests.
    See [#276](simplecov-ruby/simplecov#276) (thanks @sferik)
  * Fix TravisCI [#272](simplecov-ruby/simplecov#272) [#278](simplecov-ruby/simplecov#278), [#302](simplecov-ruby/simplecov#302)
  * Fix global config load.
    See [#311](simplecov-ruby/simplecov#311) (thanks @lasseebert)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants