Heat: package improvement

Bug #1462253 reported by Thomas Goirand
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
Medium
Daniil Trishkin

Bug Description

This is a general review of the package

debian/control
--------------
- Missing build-depends on dh-python, python-sphinx, python-setuptools (they are needed by the clean target, therefore shouldn't be in build-depends-indep)
- Missing build-depends-indep: python-alembic -> alembic python-oslo.versionedobjects, subunit
- Useless build-depends: python-extras, python-lockfile, python-lxml, python-paste, python-testrepository,
- Advice: remove python-qpid, since nobody uses it and unit tests are skipped if not present
- Missing depends: python-babel
- Useless depends: python-coverage, python-extras, python-hacking, python-lockfile,
  python-qpid (same as above), python-setuptools, python-sphinx

debian/rules
------------
- Wrong useage of && instead of ;
- rm -rf pbr*.egg <--- That's old cruft that should we get rid of
- no generation of the docs

debian/changelog
----------------
- It's comming from my package, but my debian/changelog has been scrapped

debian/*.postrm
---------------
- Incorrectly purges log files (wrong filenames). Note: this was also present in the Debian package, which I have just fixed.

ruhe (ruhe)
Changed in mos:
milestone: none → 7.0
assignee: nobody → MOS Deployment Team (mos-deployment)
importance: Undecided → Medium
Changed in mos:
status: New → Confirmed
Changed in mos:
assignee: MOS Deployment Team (mos-deployment) → Daniil Trishkin (dtrishkin)
Changed in mos:
milestone: 7.0 → 8.0
Revision history for this message
Daniil Trishkin (dtrishkin) wrote :
Changed in mos:
status: Confirmed → Fix Released
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.