[Pkg-nx-group] several small issues about nxcomp packaging

Stefan Huehner stefan at huehner.org
Sat Jul 23 17:43:08 UTC 2005


Hi,

testing out the nxcomp-packaing from svn i notives several small issues:

- lintian bails out with an error because the version number ends with
  -0 perhaps an suffix '.0' corrects this error. I've modified the
  changelog accordingly. As .0 is not larger than the existing -0 i had
  to force the change. But as the package isn't released yet this
  shouldn't be a problem.
- When compiling i noticed several compiler-warnings about
  -Wstrict-protoypes and -Wmissings-prototypes not being legal C++
  command line options. So in create a new dpatch which moves these 2
  extra warnings from cxxflags to cflags.
- I havent't touched the other two illegal c++ commandline options
  -Wmissings-declarations and -Wnested-extern as there are conditionally
  added by configure and the configure.in macros seem a bit weird to me
  in checking gcc 3.3* as version string...?
- There seem to be a lot inclusions of deprecated headers. One could
  silence these by adding -Wno-deprecated as an additional parameter but
  creating an patch for correction the includes would probably be better
  (have to check if upstream 1.5er tree has the same problems).

Patch for the addressed issues is attached. Please comment.

Regards,
Stefan

-------------- next part --------------
Index: debian/patches/00list
===================================================================
--- debian/patches/00list	(revision 16)
+++ debian/patches/00list	(working copy)
@@ -1,2 +1,3 @@
 gcc4-classdef
 build_with_fpic
+sanitize_cxxflags
Index: debian/changelog
===================================================================
--- debian/changelog	(revision 16)
+++ debian/changelog	(working copy)
@@ -1,5 +1,12 @@
+nxcomp (1.4.0.2-31.0) UNRELEASED; urgency=low
+
+  * adjust verision number to fix lintian error
+
+ -- Stefan Huehner <stefan at huehner.org>  Sat, 23 Jul 2005 19:36:39 +0200
+
 nxcomp (1.4.0.2-31-0) UNRELEASED; urgency=low
 
+  [ Stefan Lippers-Hollmann ]
   Changes by Tollef Fog Heen:
     - Initial upload to Debian
     - Split package into sensible source packages
@@ -11,8 +18,13 @@
     - switch to unpacked sources due to build issues.
     - bump standards version to 3.6.2
 
- -- Stefan Lippers-Hollmann <s.l-h at gmx.de>  Wed, 20 Jul 2005 12:16:08 +0200
+  [ Stefan Huehner ]
+  * create new patch to address that
+    - -Wmissing-declarations and -Wstrict-prototypes are
+      gcc flags and not g++ flags
 
+ -- Stefan Huehner <stefan at huehner.org>  Sat, 23 Jul 2005 19:11:25 +0200
+
 nx (1.4.0.2-0alpha6) experimental; urgency=low
 
   * add automake1.4 to the build dependencies. 
--- /dev/null	2005-03-29 14:09:30.000000000 +0200
+++ debian/patches/sanitize_cxxflags.dpatch	2005-07-23 19:10:45.000000000 +0200
@@ -0,0 +1,26 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+## sanitize-cxxflags.dpatch by  <Stefan Huehner <stefan at huehner.org>>
+##
+## All lines beginning with `## DP:' are a description of the patch.
+## DP: -Wstrict-prototypes and -Wmissing declarations are gcc flags not g++ flags.
+
+ at DPATCH@
+diff -urNad --exclude=CVS --exclude=.svn ./Makefile.in /tmp/dpep-work.22H3PC/trunk/Makefile.in
+--- ./Makefile.in	2005-07-23 19:09:32.000000000 +0200
++++ /tmp/dpep-work.22H3PC/trunk/Makefile.in	2005-07-23 19:09:59.000000000 +0200
+@@ -12,13 +12,13 @@
+ 
+ CXX         = @CXX@
+ CXXFLAGS    = @CXXFLAGS@ @X_CFLAGS@ @DEFS@ \
+-              -Wall -Wpointer-arith -Wstrict-prototypes -Wmissing-prototypes \
++              -Wall -Wpointer-arith \
+ 		-fPIC
+ CXXINCLUDES =
+ CXXDEFINES  =
+ 
+ CC          = @CC@
+-CCFLAGS     = $(CXXFLAGS)
++CCFLAGS     = $(CXXFLAGS) -Wstrict-prototypes -Wmissing-prototypes
+ CCINCLUDES  =
+ CCDEFINES   =
+ 


More information about the Pkg-nx-group mailing list