← Back to team overview

dx-packages team mailing list archive

[Bug 1326788] [NEW] "debian/rules clean" does not clean adequately

 

Public bug reported:

This is a problem in the unity-lens-applications source package.

According to Ubuntu policy, the "clean" rule in debian/rules  "must undo
any effects that the build and binary targets may have had, except that
it should leave alone any output files created in the parent directory
by a run of a binary target."  That target in unity-lens-applications
does not do this:

root@chickatrice:/tmp# apt-get source unity-lens-applications=7.1.0+13.10.20131011-0ubuntu2 >/dev/null 2>&1
root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 | wc -l92
root@chickatrice:/tmp# ( cd unity-lens-applications-7.1.0+13.10.20131011 && debian/rules build && debian/rules clean ) >/dev/null 2>&1
root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 | wc -l103
root@chickatrice:/tmp# rm -r unity*
root@chickatrice:/tmp# apt-get source unity-lens-applications=7.1.0+13.10.20131011-0ubuntu2 >/dev/null 2>&1
root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 > files.before
root@chickatrice:/tmp# ( cd unity-lens-applications-7.1.0+13.10.20131011 && debian/rules build && debian/rules clean ) >/dev/null 2>&1
root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 > files.after
root@chickatrice:/tmp# diff files.before files.after | grep '^>'
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/aptd-client.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/software-center-data-cache.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/xapian-utils.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/schemas.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/software-center-utils.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/software-center-app-details.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/utils.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/purchase-info-helper.c
> unity-lens-applications-7.1.0+13.10.20131011/tests/unit/app-watcher.c
> unity-lens-applications-7.1.0+13.10.20131011/data/applications.scope
> unity-lens-applications-7.1.0+13.10.20131011/data/scopes.scope

It looks like some of the files generating while building are not
correctly removed while cleaning.  This is a problem if (as I do) you
want to convert the package to use the "3.0 (quilt)" source format.

The attached patch fixes the problem, but probably more crudely than is
acceptable.

** Affects: unity-lens-applications (Ubuntu)
     Importance: Undecided
         Status: New

** Patch added: "Crude fix for the problem"
   https://bugs.launchpad.net/bugs/1326788/+attachment/4125877/+files/mcs-clean

-- 
You received this bug notification because you are a member of DX
Packages, which is subscribed to unity-lens-applications in Ubuntu.
https://bugs.launchpad.net/bugs/1326788

Title:
  "debian/rules clean" does not clean adequately

Status in “unity-lens-applications” package in Ubuntu:
  New

Bug description:
  This is a problem in the unity-lens-applications source package.

  According to Ubuntu policy, the "clean" rule in debian/rules  "must
  undo any effects that the build and binary targets may have had,
  except that it should leave alone any output files created in the
  parent directory by a run of a binary target."  That target in unity-
  lens-applications does not do this:

  root@chickatrice:/tmp# apt-get source unity-lens-applications=7.1.0+13.10.20131011-0ubuntu2 >/dev/null 2>&1
  root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 | wc -l92
  root@chickatrice:/tmp# ( cd unity-lens-applications-7.1.0+13.10.20131011 && debian/rules build && debian/rules clean ) >/dev/null 2>&1
  root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 | wc -l103
  root@chickatrice:/tmp# rm -r unity*
  root@chickatrice:/tmp# apt-get source unity-lens-applications=7.1.0+13.10.20131011-0ubuntu2 >/dev/null 2>&1
  root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 > files.before
  root@chickatrice:/tmp# ( cd unity-lens-applications-7.1.0+13.10.20131011 && debian/rules build && debian/rules clean ) >/dev/null 2>&1
  root@chickatrice:/tmp# find unity-lens-applications-7.1.0+13.10.20131011 > files.after
  root@chickatrice:/tmp# diff files.before files.after | grep '^>'
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/aptd-client.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/software-center-data-cache.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/xapian-utils.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/schemas.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/software-center-utils.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/software-center-app-details.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/utils.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/purchase-info-helper.c
  > unity-lens-applications-7.1.0+13.10.20131011/tests/unit/app-watcher.c
  > unity-lens-applications-7.1.0+13.10.20131011/data/applications.scope
  > unity-lens-applications-7.1.0+13.10.20131011/data/scopes.scope

  It looks like some of the files generating while building are not
  correctly removed while cleaning.  This is a problem if (as I do) you
  want to convert the package to use the "3.0 (quilt)" source format.

  The attached patch fixes the problem, but probably more crudely than
  is acceptable.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/unity-lens-applications/+bug/1326788/+subscriptions


Follow ups

References