Project

General

Profile

Bug #33810

Hammer on Ubuntu 20.04 has a wrong shebang

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

Status:
Closed
Priority:
Normal
Category:
Hammer core
Target version:
Team Backlog:
Found in Releases:
In Kanboard:

Description

At least on Ubuntu 20.04 the shebang of hammer is

/usr/bin/env ruby
which can mean it's executed using the wrong Ruby.

Reproducer:

export PATH=/opt/puppetlabs/puppet/bin:$PATH
hammer -v

See https://github.com/theforeman/puppet-foreman/pull/981#issuecomment-954887717 as well.

Marking as Found in Releases 3.1.0 but really I checked with current nightly.

This prevents testing it in puppet-foreman.

Associated revisions

Revision c2105d8b (diff)
Added by Ewoud Kohl van Wijngaarden 3 months ago

Fixes #33810 - Remove a space in hammer's shebang

This space confuses Debian packaging on Ubuntu 20.04 and causes it to
not be replaced. The end result is that it may end up using the wrong
Ruby.

History

#1 Updated by Ewoud Kohl van Wijngaarden 3 months ago

  • Subject changed from Hammer has a wrong shebang to Hammer on Ubuntu 20.04 has a wrong shebang

#2 Updated by The Foreman Bot 3 months ago

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

#3 Updated by Ewoud Kohl van Wijngaarden 3 months ago

  • Target version changed from 3.1.0 to hammer-cli-3.1.0
  • Category changed from Debian/Ubuntu to Hammer core
  • Project changed from Packaging to Hammer CLI
  • Found in Releases hammer-cli-3.0.0 added
  • Found in Releases deleted (3.1.0)

Moving the bug to the Hammer project to fix it at the source.

#4 Updated by The Foreman Bot 3 months ago

  • Pull request https://github.com/theforeman/hammer-cli/pull/355 added

#5 Updated by The Foreman Bot 3 months ago

  • Fixed in Releases hammer-cli-3.1.0 added

#6 Updated by Ewoud Kohl van Wijngaarden 3 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by Oleh Fedorenko 3 months ago

  • Fixed in Releases hammer-cli-3.0.1 added

Also available in: Atom PDF