Install scripts in /usr/bin with extensions

Bug #1454587 reported by Thomas Goirand
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Barbican
Fix Released
Medium
Dave Walker

Bug Description

It's generally forbidden in distributions (like Debian or RedHat) to install scripts in /usr/bin with the scripting language extension. Here's the errors from the Debian lintian tool:

W: barbican-common: script-with-language-extension usr/bin/barbican-db-manage.py
N:
N: When scripts are installed into a directory in the system PATH, the
N: script name should not include an extension such as .sh or .pl that
N: denotes the scripting language currently used to implement it. The
N: implementation language may change; if it does, leaving the name the
N: same would be confusing and changing it would be disruptive.
N:
N: Refer to Debian Policy Manual section 10.4 (Scripts) for details.
N:
N: Severity: normal, Certainty: certain
N:
N: Check: files, Type: binary, udeb
N:
W: barbican-common: script-with-language-extension usr/bin/barbican-keystone-listener.py
W: barbican-common: script-with-language-extension usr/bin/barbican-worker.py
W: barbican-common: script-with-language-extension usr/bin/barbican.sh

Please fix this.

Revision history for this message
Thomas Goirand (thomas-goirand) wrote :

Oh as well... this kind of things should be handled by PBR entry points, not manually creating a script in bin/*. Please have a look into other projects on how they do it.

Revision history for this message
Douglas Mendizábal (dougmendizabal) wrote :

I've read about removing the language extension from executable scripts, since it should not necessary to know what language they're written in when you execute them.

Entry points are a feature of setuptools [1] not PBR. It would definitely be interesting to move this script into the code tree and provide an entry point for it.

[1] http://pythonhosted.org//setuptools/pkg_resources.html#entry-points

Changed in barbican:
status: New → Confirmed
Revision history for this message
Thomas Goirand (thomas-goirand) wrote :

Hi. I know it's from setuptools, however it's even more easy to use with PBR... :)

Dave Walker (davewalker)
Changed in barbican:
assignee: nobody → Dave Walker (davewalker)
Changed in barbican:
status: Confirmed → Fix Committed
Thierry Carrez (ttx)
Changed in barbican:
milestone: none → liberty-2
status: Fix Committed → Fix Released
Changed in barbican:
importance: Undecided → Medium
Thierry Carrez (ttx)
Changed in barbican:
milestone: liberty-2 → 1.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.