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

drop FreeBSD < 4 support #1339

Closed
wants to merge 1 commit into from
Closed

Conversation

cremno
Copy link
Contributor

@cremno cremno commented May 2, 2016

The most recent version affected by this is 3.5 and was released in 2000.

https://www.freebsd.org/releases/3.5R/announce.html
https://en.wikipedia.org/wiki/History_of_FreeBSD#Version_history

@nobu
Copy link
Member

nobu commented May 3, 2016

Should we drop the support of older FreeBSD first?

@cremno
Copy link
Contributor Author

cremno commented May 3, 2016

@nobu: What do you mean? Maybe the title is misleading and I should have written for FreeBSD < 4?

@nobu
Copy link
Member

nobu commented May 5, 2016

Removing these lines causes test failures.
That is we do not support the older versions anymore.

@knu
Copy link
Member

knu commented May 5, 2016

@nobu Can you point us to the exact test failures this would cause? I think defined(__FreeBSD__) && __FreeBSD__ < 4 is always false when it's been over a decade since FreeBSD 3 reached its EOL.

@cremno cremno force-pushed the remove-ifs-for-old-freebsd branch from fd4b845 to 4cdad2f Compare May 5, 2016 18:05
@cremno cremno changed the title remove #if directives for FreeBSD 3.5 and older drop FreeBSD < 4 support May 5, 2016
@cremno
Copy link
Contributor Author

cremno commented May 5, 2016

I've updated the title and added a NEWS entry. Hopefully my intent is more clear now. If these changes causes tests to fail, I'll fix them. But I believe @knu is correct.

@nobu
Copy link
Member

nobu commented May 6, 2016

No specific tests may exist.

Only I found is the ChangeLog for 1.4.0:

Sun Feb 14 12:47:48 1999  EGUCHI Osamu  <eguchi@shizuokanet.ne.jp>

       * numeric.c (Init_Numeric): allow divide by zero on FreeBSD.

nobu referenced this pull request May 14, 2016
* NEWS: drop FreeBSD < 4 support.
  The most recent version affected by this is 3.5 and was released
  in 2000.
  https://www.freebsd.org/releases/3.5R/announce.html
  https://en.wikipedia.org/wiki/History_of_FreeBSD#Version_history

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@54992 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
@nobu nobu closed this May 14, 2016
@cremno cremno deleted the remove-ifs-for-old-freebsd branch March 4, 2017 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants