[PATCH] deb-pkg md5sums

[PATCH] deb-pkg md5sums

[FEJES Jozsef]

[-- Attachment #1: Type: text/plain, Size: 337 bytes --]

Hi,

I would like an md5sums file in the packages built by the deb-pkg 
script. It's a quite standard and useful thing to do.

Please consider my patch (or something similar) for inclusion.

My patch does the same thing as dh_md5sums, without any fancy options 
which are unnecessary in this case.

-- 
[ FEJES Jozsef ]
http://joco.name

[-- Attachment #2: debpkgmd5.patch --]
[-- Type: text/plain, Size: 565 bytes --]

diff -ur a/scripts/package/builddeb b/scripts/package/builddeb
--- a/scripts/package/builddeb	2010-02-09 13:57:19.000000000 +0100
+++ b/scripts/package/builddeb	2010-03-04 20:58:03.353788459 +0100
@@ -18,6 +18,7 @@
 	cp debian/copyright "$pdir/usr/share/doc/$pname/"
 	cp debian/changelog "$pdir/usr/share/doc/$pname/changelog.Debian"
 	gzip -9 "$pdir/usr/share/doc/$pname/changelog.Debian"
+	sh -c "cd '$pdir'; find . -type f ! -path './DEBIAN/*' -printf '%P\0' | xargs -r0 md5sum > DEBIAN/md5sums"
 
 	# Fix ownership and permissions
 	chown -R root:root "$pdir"

Re: [PATCH] deb-pkg md5sums

[Michal Marek]

On 4.3.2010 21:57, FEJES Jozsef wrote:
> I would like an md5sums file in the packages built by the deb-pkg 
> script. It's a quite standard and useful thing to do.
> 
> Please consider my patch (or something similar) for inclusion.
> 
> My patch does the same thing as dh_md5sums, without any fancy options 
> which are unnecessary in this case.

Hi,

Please submit a patch according to Documentation/SubmittingPatches, i.e.
inline it in the body of the email and include your Signed-off-by: line.
BTW:
sh -c "cd '$pdir'; find . -type f ! -path './DEBIAN/*' -printf '%P\0'
can be written as
find "$pdir" -path "$pdir/DEBIAN" -prune -o -type f -printf '%P\0'
(saves you some quoting).

Michal

Re: [PATCH] deb-pkg md5sums

[FEJES Jozsef]

On 2010.03.05. 14:13, Michal Marek wrote:
> On 4.3.2010 21:57, FEJES Jozsef wrote:
>> I would like an md5sums file in the packages built by the deb-pkg
>> script. It's a quite standard and useful thing to do.
>>
>> Please consider my patch (or something similar) for inclusion.
>>
>> My patch does the same thing as dh_md5sums, without any fancy options
>> which are unnecessary in this case.
>
> Hi,
>
> Please submit a patch according to Documentation/SubmittingPatches, i.e.
> inline it in the body of the email and include your Signed-off-by: line.
> BTW:
> sh -c "cd '$pdir'; find . -type f ! -path './DEBIAN/*' -printf '%P\0'
> can be written as
> find "$pdir" -path "$pdir/DEBIAN" -prune -o -type f -printf '%P\0'
> (saves you some quoting).
>
> Michal
>

Sorry, first time. (The directory change is needed because md5sum needs the correct base.) Here's the resubmission.

This patch creates the standard md5sums file for 'make deb-pkg' just like the dh_md5sums debhelper script.

Signed-off-by: Jozsef Fejes <fejes@joco.name>
-----
diff -upr a/scripts/package/builddeb b/scripts/package/builddeb
--- a/scripts/package/builddeb	2010-02-09 13:57:19.000000000 +0100
+++ b/scripts/package/builddeb	2010-03-04 20:58:03.353788459 +0100
@@ -18,6 +18,8 @@ create_package() {
 	cp debian/copyright "$pdir/usr/share/doc/$pname/"
 	cp debian/changelog "$pdir/usr/share/doc/$pname/changelog.Debian"
 	gzip -9 "$pdir/usr/share/doc/$pname/changelog.Debian"
+	sh -c "cd '$pdir'; find . -type f ! -path './DEBIAN/*' -printf '%P\0' \
+		| xargs -r0 md5sum > DEBIAN/md5sums"
 
 	# Fix ownership and permissions
 	chown -R root:root "$pdir"
-----

-- 
[ FEJES Jozsef ]
http://joco.name