review ghostess

Jonas Smedegaard dr at
Thu Aug 26 08:38:44 UTC 2010

On Wed, Aug 25, 2010 at 10:29:00PM +0200, rosea grammostola wrote:
>As a not very experienced packager, I've added ghostess
>Comments are welcome.

A more useful URL than above is that of the Git browser for the project:

I see that you use CDBS.  Nice! :-D

Then I suggest to use auto-resolving of build-dependencies (currently 
they are not quite optimal):

  1) copy debian/control to debian/
  2) edit debian/ replacing cdbs build-dpendency with @cdbs@
  3) invoke the following command to regenerate debian/control:

     DEB_MAINTAINER_MODE=1 fakeroot debian/rules clean

  4) Check what changed in the now auto-updated debian/control
  5) Remove manual entries from debian/ now auto-resolved
  6) Repeat steps 3-5 until satisfied
  6) Commit debian/
  7) Commit debian/control (separately!)

I also suggest to enable tracking of copyright/licensing changes:

  1) Include /usr/share/cdbs/1/rules/
  2) touch debian/copyright_hints
  3) Invoke the following to generate debian/copyright_newhints:

     DEB_MAINTAINER_MODE=1 debian/rules pre-build

  4) Rename debian/copyright_newhints to debian/copyright_hints
  5) Commit debian/copyright_hints

After that, at each build the source is scanned for copyright and 
licensing hints, and changes compared to the saved hints file trigger a 
warning (or a failure if DEB_MAINTAINER_MODE=1).

I also suggest you enable upstream-tarball handling - i.e. a 
get-orig-source build target:

  1) Include /usr/share/cdbs/1/rules/
  2) Add hints to get upstream source (have a look inside the actual file, or look at e.g. jackd2 for an example)

Kind regards,

  - Jonas

  * Jonas Smedegaard - idealist & Internet-arkitekt
  * Tlf.: +45 40843136  Website:

  [x] quote me freely  [ ] ask before reusing  [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <>

More information about the pkg-multimedia-maintainers mailing list