RFS: [QA upload] snowdrop

2011-06-30 Thread Arno Töll
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hello,

I'm seeking a sponsor for a QA upload of the package snowdrop. Snowdrop
is orphaned since over two years but in rather good shape and still
usable. I updated it to more recent Debian improvements (dh 8, quilt 3.0
package, ...), fixed some minor bugs and brought it up to date in general.

Upstream development stopped long time ago, but as it is still useful
I'd love to see this package well-kept.

I did:

* Switch to 3.0 (quilt) format, moved all changes to the upstream source
to a quilt patch
* Pushed debhelper to v8
* Switched debian/copyright to DEP-5
* Fixed some minor bugs (see changelog)
* Moved (Debian) man page out of the upstream source to debian/
* Made Lintian (mostly) happy, I didn't bother about a watch file, since
this makes no sense.

The full list of changes is attached in diff below. Joey, if you care
enough I can push you my changes to your git repository (or rather
export it somehow to you, since I have no write permissions to your
repository)

Finally I hope I didn't bother debian-qa@ too much with this request,
but last time I was told I shall CC you for QA-RFS mails.

The package can be found on mentors.debian.net:
- - URL: http://mentors.debian.net/debian/pool/main/s/snowdrop
- - Source repository: deb-src http://mentors.debian.net/debian unstable
main contrib non-free
- - dget
http://mentors.debian.net/debian/pool/main/s/snowdrop/snowdrop_0.02b-9.dsc

I'd be glad if someone could sponsor this QA upload.

- -- 
with kind regards,
Arno Töll
IRC: daemonkeeper on Freenode/OFTC
GnuPG Key-ID: 0x9D80F36D
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJODIKIAAoJEMcrUe6dgPNtigsP/jSX4v6RuHGM9DcRyjs8mMZM
4GTGA/V4aVXSGGY05nNvRsPEdYtlakbNEExCgnXF75iYdszlIoMG+e13DKz5pck5
q1afVCPthGF/W4pAn3+FTBdiRjI/xLwodxijmqE3SSIb8BRsb83+QUYXj+F+jbcK
lmLKX9BXLqA40b6PU7Yi9/k2DhhW94RsljT7cTu7WIChEskdYp/2ZwmW3gZA4STU
3Hdoo+ZUiWhldH++hxEQD6Ta9wBbE53+HR43EBmrNG0e/gTjYFInA1KKqthZGB54
RWv8UGEbKKOCIQ7cd/wPDjexG7ihrH6CYhC0ppHZcdl9ATxFpIuci1foGZATvZcA
W7BvsqyWLhXK5N/Pkq1JHczSAlvf1eBxgMQ1qbEHAuFrdve8gb6KEOgMTK9YEJGM
EaHooNLcpfYqFnGN4x6JCY2/jPxYFXCSUOHXLc3oKYhBhQxgH4dRSN+bdq2pCAhM
UaulUI9cKzaJR6bNPlqHpiNnTPDAMFY37lUrvmMDyWBfDTLp796bD/NC98GturQG
9V54q4yw/ZepHCINEkaE6Hw9V0beASThD3QAuJMVItjfX2DK57DGFew8LEtUpurg
LKnWFNFiWL1qjH1PUFehVeLgVuykwvOhTDeoDR5NHk9iZENTGMdTNvSsz09j+LHc
t7RKAhZvVDu08BL2GTjX
=rCO1
-END PGP SIGNATURE-
diff --git a/Makefile b/Makefile
index 9492a23..dfb0ed3 100644
--- a/Makefile
+++ b/Makefile
@@ -26,10 +26,9 @@ VER	= 0.02b
 LANG= eng engf c
 
 BINROOT = /usr/bin/
-DESTDIR = /
 
-CFLAGS  = -ggdb -O2 -fomit-frame-pointer -funroll-loops -fexpensive-optimizations \
-  -ffast-math -Wall -g
+CFLAGS  = -ggdb -O9 -fomit-frame-pointer -funroll-loops -fexpensive-optimizations \
+  -ffast-math -Wall 
 
 
 all: modules snowdrop toinstall
@@ -37,25 +36,25 @@ all: modules snowdrop toinstall
 modules: language.h
 	@echo ; \
 	echo [*] Compiling language modules: ; \
-	test -f /usr/include/gnutls/openssl.h  USEOPENSSL=1; \
-	test -f /usr/local/include/gnutls/openssl.h  USEOPENSSL=1; \
-	test $$USEOPENSSL =  || echo [+] Using GNU TLS MD5 modules. ; \
+	test -d /usr/include/openssl  USEOPENSSL=1; \
+	test -d /usr/local/include/openssl  USEOPENSSL=1; \
+	test $$USEOPENSSL =  || echo [+] Using OpenSSL MD5 modules. ; \
 	test $$USEOPENSSL =   echo [+] Trying to use RSA MD5 modules. ; \
 	for i in $(LANG); do \
 	  echo [+] Building language module for '$$i'...; \
-	  test $$USEOPENSSL =  || ADDME=-DUSE_OPENSSL -DUSE_GNUTLS ; \
+	  test $$USEOPENSSL =  || ADDME=-DUSE_OPENSSL ; \
 	  $(CC) $$ADDME $(CFLAGS) -c lang-$$i.c -o lang-$$i.o || exit 1; \
 	done; \
 	echo [*] Language modules compiled.
 
 snowdrop: snowdrop.c language.h 
 	@echo [*] Compiling main code:; \
-	test -f /usr/include/gnutls/openssl.h  USEOPENSSL=1; \
-	test -f /usr/local/include/gnutls/openssl.h  USEOPENSSL=1; \
+	test -d /usr/include/openssl  USEOPENSSL=1; \
+	test -d /usr/local/include/openssl  USEOPENSSL=1; \
 	for i in $(LANG); do \
 	  echo [+] Building 'sd-$$i'... ; \
 	  ADDME=-lmd5; \
-	  test $$USEOPENSSL =  || ADDME=-DUSE_OPENSSL -DUSE_GNUTLS -lgnutls-extra -lgnutls-openssl $(shell libgnutls-config --cflags --libs) ; \
+	  test $$USEOPENSSL =  || ADDME=-DUSE_OPENSSL -lcrypto ; \
 	  $(CC)  -DVER=\$(VER)\ $(CFLAGS) -DTARGETLANG=\$$i\ snowdrop.c lang-$$i.o -o sd-$$i $$ADDME || exit 1; \
 	done; \
 	echo [*] Main code compiled.
@@ -68,12 +67,12 @@ toinstall:
 clean:
 	rm -f sd-* *.o core core.* a.out
 
-install:
-	@echo [*] Installing binaries in $(DESTDIR)$(BINROOT)...
-	cp -f sd-* $(DESTDIR)$(BINROOT)
+install: modules snowdrop
+	@echo [*] Installing binaries in $(BINROOT)...
+	cp -f sd-* $(BINROOT)
 	@echo [*] Installing synonyms database...
-	@mkdir $(DESTDIR)/usr/share/snowdrop || true
-	cp synonyms $(DESTDIR)/usr/share/snowdrop/
+	@mkdir /usr/share/snowdrop || true
+	cp 

Re: RFS: [QA upload] snowdrop

2011-06-30 Thread Jan Hauke Rahm
On Thu, Jun 30, 2011 at 04:04:57PM +0200, Arno Töll wrote:
 I'm seeking a sponsor for a QA upload of the package snowdrop. Snowdrop
 is orphaned since over two years but in rather good shape and still
 usable. I updated it to more recent Debian improvements (dh 8, quilt 3.0
 package, ...), fixed some minor bugs and brought it up to date in general.
 
 Upstream development stopped long time ago, but as it is still useful
 I'd love to see this package well-kept.

I'll sponsor it. Thanks, Arno!

 The full list of changes is attached in diff below. Joey, if you care
 enough I can push you my changes to your git repository (or rather
 export it somehow to you, since I have no write permissions to your
 repository)

We should have a policy about all QA packages being in collab-maint or
something...

Hauke

-- 
 .''`.   Jan Hauke Rahm j...@debian.org   www.jhr-online.de
: :'  :  Debian Developer www.debian.org
`. `'`   Member of the Linux Foundationwww.linux.com
  `- Fellow of the Free Software Foundation Europe  www.fsfe.org


signature.asc
Description: Digital signature