Package: polipo
Severity: wishlist
Tags: patch

Hi,

The dpatch patch management system has been deprecated for some time. The
Lintian currently flags use of dpatch packages as an error. The new 3.0
packaging format is an improved version which, among other things, contains
patch management built-in. For more information, see:

    http://wiki.debian.org/Projects/DebSrc3.0

I had some free time; see attached patch to migrate to new package
format. Note that all files in debian/patches/* are canocalized to
*.patch.

Let me know if there is anything that needs adjusting or if it is ok
to upload this version in a NMU in case you are working on other
issues needing attention.

Thanks,
Jari

>From 2edd9b3e051e04286d471fcbd4826cb1c272b1ff Mon Sep 17 00:00:00 2001
From: Jari Aalto <jari.aa...@cante.net>
Date: Wed, 29 Feb 2012 09:13:04 -0500
Subject: [PATCH] format-3.0
Organization: Private
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Signed-off-by: Jari Aalto <jari.aa...@cante.net>
---
 debian/changelog                                   |   10 ++++
 debian/compat                                      |    2 +-
 debian/control                                     |    4 +-
 debian/patches/00list                              |    3 -
 .../{10_makefile.dpatch => 10_makefile.patch}      |   36 ++++------------
 .../patches/{20_config.dpatch => 20_config.patch}  |   26 +----------
 ...iously-ignore-CNAMEs-after-a-DNS-timeout.dpatch |   45 --------------------
 ...riously-ignore-CNAMEs-after-a-DNS-timeout.patch |   36 ++++++++++++++++
 debian/patches/series                              |    3 +
 debian/rules                                       |    1 -
 debian/source/format                               |    1 +
 11 files changed, 64 insertions(+), 103 deletions(-)
 delete mode 100644 debian/patches/00list
 rename debian/patches/{10_makefile.dpatch => 10_makefile.patch} (55%)
 rename debian/patches/{20_config.dpatch => 20_config.patch} (65%)
 delete mode 100644 debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.dpatch
 create mode 100644 debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.patch
 create mode 100644 debian/patches/series
 create mode 100644 debian/source/format

diff --git a/debian/changelog b/debian/changelog
index 09ec360..427e1a3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,13 @@
+polipo (1.0.4.1-1.2) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Remove deprecated dpatch and upgrade to packaging format "3.0 quilt".
+    Note: some patches were updated with "quilt refresh" to make
+    them apply cleanly (10, 30).
+  * Update to Standards-Version to 3.9.3 and debhelper to 9.
+
+ -- Jari Aalto <jari.aa...@cante.net>  Wed, 29 Feb 2012 09:10:41 -0500
+
 polipo (1.0.4.1-1.1) unstable; urgency=low
 
   * Non-maintainer upload.
diff --git a/debian/compat b/debian/compat
index 7ed6ff8..ec63514 100644
--- a/debian/compat
+++ b/debian/compat
@@ -1 +1 @@
-5
+9
diff --git a/debian/control b/debian/control
index 5152bb8..c3ddf46 100644
--- a/debian/control
+++ b/debian/control
@@ -2,8 +2,8 @@ Source: polipo
 Section: web
 Priority: optional
 Maintainer: Erinn Clark <er...@torproject.org>
-Build-Depends: debhelper (>= 5), cdbs, texinfo, dpatch
-Standards-Version: 3.8.4
+Build-Depends: debhelper (>= 9), cdbs, texinfo
+Standards-Version: 3.9.3
 Homepage: http://www.pps.jussieu.fr/~jch/software/polipo/
 
 Package: polipo
diff --git a/debian/patches/00list b/debian/patches/00list
deleted file mode 100644
index c7f6af3..0000000
--- a/debian/patches/00list
+++ /dev/null
@@ -1,3 +0,0 @@
-10_makefile.dpatch
-20_config.dpatch
-30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.dpatch
diff --git a/debian/patches/10_makefile.dpatch b/debian/patches/10_makefile.patch
similarity index 55%
rename from debian/patches/10_makefile.dpatch
rename to debian/patches/10_makefile.patch
index afa7b89..d37cef8 100644
--- a/debian/patches/10_makefile.dpatch
+++ b/debian/patches/10_makefile.patch
@@ -1,31 +1,13 @@
-#! /bin/sh /usr/share/dpatch/dpatch-run
-## 10_makefile.dpatch by Denis V. Sirotkin <fech...@gmail.com>
-## Original changes was done by Tom Huckstep <tom-debian-pol...@jaguarpaw.co.uk>
-##
-## DP: Change description
+From: Denis V. Sirotkin <fech...@gmail.com>
+Subject: Change description
 
-if [ $# -lt 1 ]; then
-    echo "`basename $0`: script expects -patch|-unpatch as argument" >&2
-    exit 1
-fi
 
-[ -f debian/patches/00patch-opts ] && . debian/patches/00patch-opts
-patch_opts="${patch_opts:--f --no-backup-if-mismatch} ${2:+-d $2}"
+---
+ Makefile |    8 ++++++--
+ 1 file changed, 6 insertions(+), 2 deletions(-)
 
-case "$1" in
-    -patch) patch -p1 ${patch_opts} < $0;;
-    -unpatch) patch -R -p1 ${patch_opts} < $0;;
-    *)
-	echo "`basename $0`: script expects -patch|-unpatch as argument" >&2
-	exit 1;;
-esac
-
-exit 0
-
-@DPATCH@
-
---- polipo-1.0.1.orig/Makefile
-+++ polipo-1.0.1/Makefile
+--- a/Makefile
++++ b/Makefile
 @@ -1,4 +1,5 @@
 -PREFIX = /usr/local
 +PREFIX = /usr
@@ -33,7 +15,7 @@ exit 0
  BINDIR = $(PREFIX)/bin
  MANDIR = $(PREFIX)/man
  INFODIR = $(PREFIX)/info
-@@ -62,7 +63,7 @@
+@@ -63,7 +64,7 @@
  
  DEFINES = $(FILE_DEFINES) $(PLATFORM_DEFINES)
  
@@ -42,7 +24,7 @@ exit 0
  
  SRCS = util.c event.c io.c chunk.c atom.c object.c log.c diskcache.c main.c \
         config.c local.c http.c client.c server.c auth.c tunnel.c \
-@@ -91,10 +92,13 @@
+@@ -92,10 +93,13 @@
  	mkdir -p $(TARGET)$(BINDIR)
  	mkdir -p $(TARGET)$(LOCAL_ROOT)
  	mkdir -p $(TARGET)$(LOCAL_ROOT)/doc
diff --git a/debian/patches/20_config.dpatch b/debian/patches/20_config.patch
similarity index 65%
rename from debian/patches/20_config.dpatch
rename to debian/patches/20_config.patch
index f621cdf..896f54d 100644
--- a/debian/patches/20_config.dpatch
+++ b/debian/patches/20_config.patch
@@ -1,28 +1,6 @@
-#! /bin/sh /usr/share/dpatch/dpatch-run
-## 20_config.dpatch by Denis V. Sirotkin <fech...@gmail.com>
-## Original changes was done by Tom Huckstep <tom-debian-pol...@jaguarpaw.co.uk>
-##
-## DP: Change description
+From: Denis V. Sirotkin <fech...@gmail.com>
+Subject: Change description
 
-if [ $# -lt 1 ]; then
-    echo "`basename $0`: script expects -patch|-unpatch as argument" >&2
-    exit 1
-fi
-
-[ -f debian/patches/00patch-opts ] && . debian/patches/00patch-opts
-patch_opts="${patch_opts:--f --no-backup-if-mismatch} ${2:+-d $2}"
-
-case "$1" in
-    -patch) patch -p1 ${patch_opts} < $0;;
-    -unpatch) patch -R -p1 ${patch_opts} < $0;;
-    *)
-	echo "`basename $0`: script expects -patch|-unpatch as argument" >&2
-	exit 1;;
-esac
-
-exit 0
-
-@DPATCH@
 
 --- polipo-1.0.1.orig/config.sample
 +++ polipo-1.0.1/config.sample
diff --git a/debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.dpatch b/debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.dpatch
deleted file mode 100644
index 343c74c..0000000
--- a/debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.dpatch
+++ /dev/null
@@ -1,45 +0,0 @@
-#! /bin/sh /usr/share/dpatch/dpatch-run
-## 30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.dpatch by Alan Jenkins
-##
-## All lines beginning with `## DP:' are a description of the patch.
-## DP: Don't spuriously ignore CNAMEs after a DNS timeout
-## DP: cherry-picked from 1144bc9a25fe685bf5a8d18a22d4fa79b9ade75b
-
-@DPATCH@
-
-From 1144bc9a25fe685bf5a8d18a22d4fa79b9ade75b Mon Sep 17 00:00:00 2001
-From: Juliusz Chroboczek <j...@pps.jussieu.fr>
-Date: Tue, 10 Nov 2009 16:48:04 -0800
-Subject: [PATCH] Don't spuriously ignore CNAMEs after a DNS timeout.
-
-Ignore-this: 60fbb7453699ecbff6961bf1622dfb3
-From Alan Jenkins.
-
-darcs-hash:20091111004804-4cc09-0d24709fbdebd5a7bdc5ce01a6a76aab5bc1241c.gz
----
- dns.c |    9 +++++++++
- 1 files changed, 9 insertions(+), 0 deletions(-)
-
-diff --git a/dns.c b/dns.c
-index 8d4ac75..1a5b39b 100644
---- a/dns.c
-+++ b/dns.c
-@@ -1181,6 +1181,15 @@ dnsReplyHandler(int abort, FdEventHandlerPtr event)
-         } else
-             releaseAtom(value);
-     } else if(af == 0) {
-+        /* Ignore errors in this case. */
-+        if(query->inet4 && query->inet4->length == 0) {
-+            releaseAtom(query->inet4);
-+            query->inet4 = NULL;
-+        }
-+        if(query->inet6 && query->inet6->length == 0) {
-+            releaseAtom(query->inet6);
-+            query->inet6 = NULL;
-+        }
-         if(query->inet4 || query->inet6) {
-             do_log(L_WARN, "Host %s has both %s and CNAME -- "
-                    "ignoring CNAME.\n", scrub(query->name->string),
--- 
-1.7.1
-
diff --git a/debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.patch b/debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.patch
new file mode 100644
index 0000000..4f5eb23
--- /dev/null
+++ b/debian/patches/30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.patch
@@ -0,0 +1,36 @@
+From: Juliusz Chroboczek <j...@pps.jussieu.fr>
+Subject: Don't spuriously ignore CNAMEs after a DNS timeout
+X-Comment:
+ cherry-picked from 1144bc9a25fe685bf5a8d18a22d4fa79b9ade75b
+ .
+ From 1144bc9a25fe685bf5a8d18a22d4fa79b9ade75b Mon Sep 17 00:00:00 2001
+ From: Juliusz Chroboczek <j...@pps.jussieu.fr>
+ Date: Tue, 10 Nov 2009 16:48:04 -0800
+ Subject: [PATCH] Don't spuriously ignore CNAMEs after a DNS timeout.
+ .
+ Ignore-this: 60fbb7453699ecbff6961bf1622dfb3
+ From Alan Jenkins.
+ .
+ darcs-hash:20091111004804-4cc09-0d24709fbdebd5a7bdc5ce01a6a76aab5bc1241c.gz
+ ---
+ dns.c |    9 +++++++++
+ 1 file changed, 9 insertions(+)
+
+--- a/dns.c
++++ b/dns.c
+@@ -1181,6 +1181,15 @@
+         } else
+             releaseAtom(value);
+     } else if(af == 0) {
++        /* Ignore errors in this case. */
++        if(query->inet4 && query->inet4->length == 0) {
++            releaseAtom(query->inet4);
++            query->inet4 = NULL;
++        }
++        if(query->inet6 && query->inet6->length == 0) {
++            releaseAtom(query->inet6);
++            query->inet6 = NULL;
++        }
+         if(query->inet4 || query->inet6) {
+             do_log(L_WARN, "Host %s has both %s and CNAME -- "
+                    "ignoring CNAME.\n", query->name->string,
diff --git a/debian/patches/series b/debian/patches/series
new file mode 100644
index 0000000..adfcc8a
--- /dev/null
+++ b/debian/patches/series
@@ -0,0 +1,3 @@
+10_makefile.patch
+20_config.patch
+30_Don-t-spuriously-ignore-CNAMEs-after-a-DNS-timeout.patch
diff --git a/debian/rules b/debian/rules
index d023ee3..22ba2ea 100755
--- a/debian/rules
+++ b/debian/rules
@@ -1,5 +1,4 @@
 #!/usr/bin/make -f
-include /usr/share/cdbs/1/rules/dpatch.mk
 include /usr/share/cdbs/1/rules/debhelper.mk
 include /usr/share/cdbs/1/class/makefile.mk
 include debian/cdbs/helper-scripts.mk
diff --git a/debian/source/format b/debian/source/format
new file mode 100644
index 0000000..163aaf8
--- /dev/null
+++ b/debian/source/format
@@ -0,0 +1 @@
+3.0 (quilt)
-- 
1.7.9

Reply via email to