Project

General

Profile

Refactor #26979

Remove ruby 1.x compatiblity code

Added by Ewoud Kohl van Wijngaarden 4 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Ewoud Kohl van Wijngaarden
Category:
Core
Target version:
Foreman - 1.23.0
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/smart-proxy/pull/660, https://github.com/theforeman/smart-proxy/pull/654
Fixed in Releases:
Foreman - 1.23.0
Found in Releases:

Description

We no longer support running on Ruby 1.x so we can remove the workarounds.

Associated revisions

Revision bf476d0a (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Fixes #26979 - Remove Ruby 1.x compatibility code

Revision 75589996 (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Refs #26979 - Remove left over require

bf476d0ad900e36f25865592538942a4b91610ad removed lib/daemon.rb since
it only contained Ruby 1.x compatibility but left the require causing it
to fail to start up.

Revision 862776b4 (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Refs #26979 - Require at least rack 1.3

This removes a workaround for an ancient rack version. EPEL7 contains
rack 1.6.4 so this should be safe.

Revision 403b79f9 (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Refs #26979 - Remove old platforms

History

#1 Updated by The Foreman Bot 4 months ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/654 added

#2 Updated by Tomer Brisker 4 months ago

  • Fixed in Releases 1.23.0 added

#3 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot 4 months ago

  • Pull request https://github.com/theforeman/smart-proxy/pull/660 added

Also available in: Atom PDF