Project

Profile

Help

Issue #1383

closed

Convert shebangs to python2

Added by semyers over 8 years ago. Updated about 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
Platform Release:
2.8.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

Currently, most (maybe all?) of our shell script shebangs call the 'python' executable, either via /usr/bin/python, or /usr/bin/env python. Many of the platforms we'd like to support either point /usr/bin/python at the python 3 interpreter or do not provide the 'python' interpreter at all. To the best of my knowledge, all of those platforms do provide a 'python2' executable, so for our packages to work on all platforms we want to update our shebangs to be using the '/usr/bin/python2' executable, not '/usr/bin/python' (e.g. /usr/bin/env python2).

Actions #1

Updated by mhrivnak over 8 years ago

  • Project changed from Packaging to Pulp
  • Priority changed from Normal to High
  • Severity changed from 2. Medium to 1. Low
  • Platform Release set to 2.8.0
  • Triaged changed from No to Yes
  • Tags Easy Fix added
Actions #2

Updated by ipanova@redhat.com over 8 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com
Actions #4

Updated by semyers over 8 years ago

All the PRs look good.

@ipanova++ for also fixing the /usr/bin/python shebangs despite my rather unclear issue description. Also good call on removing the -tt from some shebangs because they were silly (flake8 for the win).

Actions #5

Updated by ipanova@redhat.com over 8 years ago

  • Status changed from POST to MODIFIED
Actions #6

Updated by dkliban@redhat.com about 8 years ago

  • Status changed from MODIFIED to 5
Actions #7

Updated by dkliban@redhat.com about 8 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #8

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF