On Mon, Aug 30, 2010 at 08:37:52AM +0000, ales...@users.alioth.debian.org wrote:
The following commit has been merged in the master branch:
commit fcd1948a6bfd99b638cb41f0aa6f1bd22d0994db
Author: Alessio Treglia <ales...@debian.org>
Date:   Mon Aug 30 10:37:20 2010 +0200

   Install NEWS file as upstream changelog.

diff --git a/debian/rules b/debian/rules
index 3aee747..e6b51c3 100755
--- a/debian/rules
+++ b/debian/rules
@@ -1,6 +1,7 @@
#!/usr/bin/make -f

LDFLAGS+=-Wl,--as-needed
+DEB_INSTALL_CHANGELOGS_ALL = NEWS

include /usr/share/cdbs/1/rules/debhelper.mk
include /usr/share/cdbs/1/class/autotools.mk


Please beware that NEWS is already included as upstream NEWS.


Also, please list inclusions _above_ variables whenever possible.

Ancient CDBS was picky more about ordering, even of inclusions themselves. Now only a few variables must[1] be declared before inclusions - please generally list variables below inclusions to distinguish the picky from the flexible.



Kind regards,

 - Jonas



[1] Picky variables declared too late should now get detected and cause the build to fail with an explicit error about it.

--
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

Attachment: signature.asc
Description: Digital signature

_______________________________________________
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to