Bug #20918

Downloaded version of virt-who configure script is not compatible with rhel6

Added by Marek Hulán 12 days ago. Updated 3 days ago.

Status:Closed
Priority:Normal
Assigned To:Marek Hulán
Category:-
Target version:-
Difficulty: Pull request:https://github.com/theforeman/foreman_virt_who_configure/pull/70, https://github.com/theforeman/foreman_virt_who_configure/pull/65
Bugzilla link:1491450
Story points-
Velocity based estimate-
Releaseforeman_virt_who_configure-0.1.6Release relationshipAuto

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1491450

Description of problem:
When using the "Download the script" button the script using the shebang

"#! /usr/bin/bash"

This works on RHEL7 but not on RHEL6.

Version-Release number of selected component (if applicable):

How reproducible: Always

Steps to Reproduce:
1. Create virt-who config using plugin
2. Use "download the script" option to say the script to a file.
3. Copy the script to a RHEL6 box, chmod +x it and run it

Actual results:
'''
[root@ibm-x3250m4-04 ~]# ./deploy_virt_who_config_2.sh
-bash: ./deploy_virt_who_config_2.sh: /usr/bin/bash: bad interpreter: No such file or directory
'''

Expected results:
The script executes correctly

Additional info: In rhel7 /bin is a symlink to /usr/bin. So a simple fix would be to change the hash bang to /bin and this should work on both.

Associated revisions

Revision fab24745
Added by Marek Hulán 12 days ago

Fixes #20918 - use EL 6 compatible shebang

Revision 5a8e1285
Added by Marek Hulán about 14 hours ago

Refs #20918 - fix tests for shebang

History

#1 Updated by The Foreman Bot 12 days ago

  • Pull request https://github.com/theforeman/foreman_virt_who_configure/pull/65 added

#2 Updated by Marek Hulán 12 days ago

  • Subject changed from Downloaded version of virt-who configure script is not compatible with rhel6 to Downloaded version of virt-who configure script is not compatible with rhel6
  • Status changed from New to Ready For Testing

#3 Updated by Marek Hulán 12 days ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Marek Hulán 6 days ago

  • Release set to foreman_virt_who_configure-0.1.6

#5 Updated by The Foreman Bot 3 days ago

  • Pull request https://github.com/theforeman/foreman_virt_who_configure/pull/70 added

Also available in: Atom PDF