nginx-common should not depend on python

Bug #1480513 reported by Mark Constable on 2015-08-01
This bug affects 1 person
Affects Status Importance Assigned to Milestone
nginx (Ubuntu)
Thomas Ward

Bug Description

None of the dependencies for nginx-common (lsb-base and init-system-helpers) require python support so the nginx-common package should not force the "python" package to be installed. The current "python" dependency should be moved to Suggests or added to the other nginx-* packages that may need python support. The nginx-light package does not need python but because of the dependency in nginx-common it forces python to be installed.

Thomas Ward (teward) wrote :

The problem is that the intent in Debian is to ship an `ngx-conf` utility to help manage configuration files. Based off of commits to Debian, this was supposed to be installed as part of nginx-common. The Python dependency exists because that utility was written in Python.

I need to check with Debian to see if they still intend to ship this utility and add it to the install scripts. As such I am marking this as Wishlist importance and Triaged until I can reach out to Debian.

Changed in nginx (Ubuntu):
importance: Undecided → Wishlist
status: New → Triaged
assignee: nobody → Thomas Ward (teward)
Mark Constable (markc) wrote :

Thank you for considering this request and marking it appropriately.

Please kindly suggest to the Debian folks that this future ngx-conf utility should be in a separate package to isolate the need for python. I only noticed this because I routinely remove anything to do with python from my VPS and lxc containers to keep them as light as possible and I was puzzled why installing nginx(-light) tried to reinstall python-2.7.

Thomas Ward (teward) on 2016-01-27
Changed in nginx (Ubuntu):
status: Triaged → In Progress
Thomas Ward (teward) on 2016-01-27
Changed in nginx (Ubuntu):
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nginx - 1.9.10-1ubuntu1

nginx (1.9.10-1ubuntu1) xenial; urgency=low

  * Merge from Debian unstable. Remaining changes: (LP: #1538677)
    - debian/patches/ubuntu-branding.patch: add Ubuntu branding (refreshed)
    - d/{control,rules,nginx-core.*}: add new binary package for main,
      nginx-core, which contains only source-tarball-included modules
      and no third-party modules.
    - debian/tests/control: add nginx-core test.
    - debian/control:
      - drop luajit from Build-Depends as it is in universe.
      - Remove HTTP/2 references in package descriptions, per Ubuntu
        Security Team mandate to disable HTTP/2 support.
    - debian/rules:
      - Disable HTTP/2 module support in all flavors, per Ubuntu Security
        Team mandate.
    - debian/apport/ Add apport hooks for additional bug
      information gathering.
    - debian/nginx-common.install: Add install rule for apport hooks.
  * Additional bugs fixed by this merge:
    - nginx-common should not depend on python (LP: #1480513)

 -- Thomas Ward <email address hidden> Tue, 27 Jan 2016 12:52:00 -0500

Changed in nginx (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers