Package: tecnoballz
Version: 0.91-cvs20050828-2
Severity: important

Hi Alexis,

this package has several packaging issues which are outlined below. The
problems are taken verbatim from the soon-to-come T&S part 2 reply.
Please fix them as soon as possible.

 * Does tecnoballz not compile with g++ < 4? If it does, please remove
   the build-dependency. Sources changed to compile with g++ 4.0 should
   also work with earlier versions. You should only target sid which has
   g++ 4.0 on all architectures now, an explicit dependency is not
   required. Apart from that `g++' has the version `4:4.0.2-1', so
   indeed it's higher, but the correct number would be `4:4.0.0'. Like
   this it's just useless, because it is satisfied in any case.
 * You should include the copyright notice taken verbatim from the
   source code.
 * There is `technoballz.substvars' in the diff, because the package name
   was once spelled wrong. Please remove it.
 * Shouldn't you ensure permissions with statoverride as specified above
   instead of overwriting them in any case in `postinst'?
 * The following part is wrong [cf. Policy 10.1]:
   | ifneq (,$(findstring nostrip,$(DEB_BUILD_OPTIONS)))
   |        CXXFLAGS += -g
   | else
   |        CXXFLAGS += -s
   | endif

Kind regards,
Philipp Kern
Debian Developer


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

Reply via email to