Bug#464315: Intent to NMU

2008-02-21 Thread Margarita Manterola
Hola Ludovic Brenta!

> Thanks a lot for the patch and NMU.  I approve of your patch; however,
> why make a debian/compat file when the DH_COMPAT in debian/rules
> worked perfectly well?  Is there a Policy statement I overlooked?

I think it is due to Debhelper changing it's recommended way of setting the
value. From [0]:

As of debhelper version 4, the DH_COMPAT environment variable is only
to be used for temporarily overriding debian/compat. Any line in
debian/rules that sets it globally should be deleted and a separate
debian/compat file created if needed.

Refer to the debhelper(7) manual page for details. 

[0]: http://lintian.debian.org/reports/tags/debian-rules-sets-DH_COMPAT.html

I'm not sure what's the rationale behind the change though.  Maybe a
decision that it's better to have only one way of doing things?

-- 
 Bessos,
 Maggie.



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#464315: Intent to NMU

2008-02-20 Thread Ludovic Brenta
Thanks a lot for the patch and NMU.  I approve of your patch; however,
why make a debian/compat file when the DH_COMPAT in debian/rules
worked perfectly well?  Is there a Policy statement I overlooked?

Bises

-- 
Ludovic Brenta.




-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#464315: Intent to NMU

2008-02-19 Thread Margarita Manterola
Tags 464315 +pending
thanks.

I have prepared an NMU that fixes this FTBFS bug, and also gets the package
in better shape (new Standards Version, only one lintian warning, and it's
buildable twice in a row.

I'm attaching the debdiff output of the patch I have prepared.  Since this
is not only for the RC bug, I'll upload the package I've prepared to the
7-day delayed queue.  This means it should be entering unstable by February
26th.

-- 
Besos,  ,''`.
   Marga   : :' :
   `. `' 
 `-  
diff -u libtemplates-parser-10.0+20060522/debian/changelog libtemplates-parser-10.0+20060522/debian/changelog
--- libtemplates-parser-10.0+20060522/debian/changelog
+++ libtemplates-parser-10.0+20060522/debian/changelog
@@ -1,3 +1,18 @@
+libtemplates-parser (10.0+20060522-5.1) unstable; urgency=low
+
+  * Non-maintainer upload to fix Failure To Build From Source.
+  * Added patches/fix-tail-call.patch to use the new standard in the
+call to the tail command. Closes: #464315.
+  * Updated Standards-Version to 3.7.3:
+- Changed {Source-Version} for {binary:Version} in the -dev dependency
+  * Fixed lintian warnings:
+- Added Section: libs to the Source package
+- Added 'compat' file with a value of 4, removed it from debian/rules
+  * debian/rules: added a rule in the clean target to remove the .pc 
+directory generated by quilt.
+
+ -- Margarita Manterola <[EMAIL PROTECTED]>  Tue, 19 Feb 2008 18:05:52 +
+
 libtemplates-parser (10.0+20060522-5) unstable; urgency=low
   
   * debian/rules: add an empty binary-indep target.  Closes: #395713.
diff -u libtemplates-parser-10.0+20060522/debian/control libtemplates-parser-10.0+20060522/debian/control
--- libtemplates-parser-10.0+20060522/debian/control
+++ libtemplates-parser-10.0+20060522/debian/control
@@ -1,13 +1,14 @@
 Source: libtemplates-parser
+Section: libs
 Priority: optional
 Maintainer: Ludovic Brenta <[EMAIL PROTECTED]>
 Build-Depends: debhelper (>= 4.1.0), gnat (>= 4.1), texinfo, tetex-bin, tetex-extra, quilt
-Standards-Version: 3.7.2
+Standards-Version: 3.7.3
 
 Package: libtemplates-parser-dev
 Section: libdevel
 Architecture: alpha amd64 hppa i386 ia64 kfreebsd-i386 powerpc s390 sparc
-Depends: libtemplates-parser10 (= ${Source-Version}), gnat-4.1
+Depends: libtemplates-parser10 (= ${binary:Version}), gnat-4.1
 Description: Ada library to parse files and replace variables with their values
  The main goal is to ease the development of Web servers. In CGI
  (Common Gateway Interface) applications, you have to write  HTML pages in
diff -u libtemplates-parser-10.0+20060522/debian/rules libtemplates-parser-10.0+20060522/debian/rules
--- libtemplates-parser-10.0+20060522/debian/rules
+++ libtemplates-parser-10.0+20060522/debian/rules
@@ -24,7 +24,6 @@
 # gnatmake can do parallel builds; we don't want make to interfere.
 .NOTPARALLEL:
 
-export DH_COMPAT=4
 export SHELL=/bin/bash
 
 MAJOR := $(shell dpkg-parsechangelog | grep "^Version: " | sed 's/^Version: \(.*\)\.\(.*\)-\(.*\)/\1/')
@@ -85,6 +84,7 @@
 	dh_testdir
 	dh_testroot
 	-quilt pop -a
+	-rm -rf .pc
 	rm -f doc-stamp libtemplates_parser* build.adb
 	rm -rf obj-static obj-shared debian/files
 	$(MAKE) -C docs clean
diff -u libtemplates-parser-10.0+20060522/patches/series libtemplates-parser-10.0+20060522/patches/series
--- libtemplates-parser-10.0+20060522/patches/series
+++ libtemplates-parser-10.0+20060522/patches/series
@@ -1,2 +1,3 @@
+fix-tail-call.patch
 info-dir-section.patch
 strings_maps.patch
only in patch2:
unchanged:
--- libtemplates-parser-10.0+20060522.orig/debian/compat
+++ libtemplates-parser-10.0+20060522/debian/compat
@@ -0,0 +1 @@
+4
only in patch2:
unchanged:
--- libtemplates-parser-10.0+20060522.orig/patches/fix-tail-call.patch
+++ libtemplates-parser-10.0+20060522/patches/fix-tail-call.patch
@@ -0,0 +1,13 @@
+Index: libtemplates-parser-10.0+20060522/docs/makefile
+===
+--- libtemplates-parser-10.0+20060522.orig/docs/makefile	2008-02-19 12:21:17.0 +
 libtemplates-parser-10.0+20060522/docs/makefile	2008-02-19 12:21:28.0 +
+@@ -108,7 +108,7 @@
+ endif
+ 
+ $(ADBFILES) $(ADSFILES): all_sources.ada
+-	tail +3 all_sources.ada > sources.ada
++	tail --lines=+3 all_sources.ada > sources.ada
+ 	gnatchop -w sources.ada
+ 	rm sources.ada
+ 


signature.asc
Description: Digital signature