I'm not planning to sponsor this but here is a review:

I wonder if the low-level nvram reading code should be replaced with a
Linux driver?

Please run tagpending from devscripts before uploading to mentors.

Please mention why debian/tpb.devfs is being dropped in the changelog.

fix-spelling-error-in-manpage.patch contains an invalid Bug URL.

Please send the patches upstream or mark them as forwarded using DEP-3:

http://dep.debian.net/deps/dep3/

The watch file doesn't account for the possibility of upstream
changing tarball formats. There is at least one other mistake in the
watch file:

https://wiki.debian.org/debian/watch#Common_mistakes

You may want to switch to debhelper compat 9 and dh instead of manual
dh_* sequencing for debian/rules.

You may want to use dh-autoreconf instead of autotools-dev.

The package fails to build twice in a row.

Various files have the incorrect FSF address, please to ask upstream
to fix them.

intl/plural.c is a generated file, please ensure that it is
regenerated at build time.

You might want to help review other packages:

http://mentors.debian.net/intro-reviewers

If you contact upstream, please mention our upstream guide:

http://wiki.debian.org/UpstreamGuide

Automatic checks:

https://wiki.debian.org/HowToPackageForDebian#Check_points_for_any_package

gcc:

cfg.c: In function 'override_cfg':
cfg.c:965:21: warning: comparison between 'xosd_pos' and 'enum state'
[-Wenum-compare]
cfg.c:971:23: warning: comparison between 'xosd_align' and 'enum
state' [-Wenum-compare]
cfg.c: At top level:
cfg.c:45:29: warning: 'RCSid' defined but not used [-Wunused-variable]
tpb.c:55:29: warning: 'RCSid' defined but not used [-Wunused-variable]
tpb.c: In function 'fork_app':
tpb.c:1260:12: warning: ignoring return value of 'chdir', declared
with attribute warn_unused_result [-Wunused-result]
tpb.c: In function 'daemonize':
tpb.c:779:8: warning: ignoring return value of 'chdir', declared with
attribute warn_unused_result [-Wunused-result]
tpb.c:790:6: warning: ignoring return value of 'dup', declared with
attribute warn_unused_result [-Wunused-result]
tpb.c:793:6: warning: ignoring return value of 'dup', declared with
attribute warn_unused_result [-Wunused-result]

dpkg-shlibdeps:

warning: package could avoid a useless dependency if
debian/tpb/usr/bin/tpb was not linked against libXinerama.so.1 (it
uses none of the library's symbols)
warning: package could avoid a useless dependency if
debian/tpb/usr/bin/tpb was not linked against libSM.so.6 (it uses none
of the library's symbols)
warning: package could avoid a useless dependency if
debian/tpb/usr/bin/tpb was not linked against libICE.so.6 (it uses
none of the library's symbols)
warning: package could avoid a useless dependency if
debian/tpb/usr/bin/tpb was not linked against libXext.so.6 (it uses
none of the library's symbols)

cppcheck:

[intl/localcharset.c:153]: (error) Common realloc mistake: 'res_ptr'
nulled but not freed upon failure

msgfmt:

lots of fuzzy/empty things ignored.

POFileChecker:

<po-file-checker>
  <file name="./debian/po/id.po" allow="0">
    <error line="46" message="5">missing .</error>
  </file>
  <file name="./debian/po/de.po" allow="0">
    <error line="53" message="5">missing .</error>
  </file>
  <file name="./debian/po/nl.po" allow="0">
    <error line="46" message="5">missing .</error>
  </file>
  <file name="./debian/po/ja.po" allow="0">
    <error line="54" message="5">missing .</error>
  </file>
  <file name="./po/fr.po" allow="0">
    <error line="577" message="122">extra &amp; acelerator</error>
    <error line="582" message="123">extra &amp; acelerator</error>
    <error line="587" message="124">extra &amp; acelerator</error>
    <error line="592" message="125">extra &amp; acelerator</error>
  </file>
  <file name="./po/da.po" allow="0">
    <error line="18" message="2">missing :</error>
  </file>
  <file name="./po/de.po" allow="0">
    <error line="216" message="49">missing :</error>
  </file>
</po-file-checker>

isutf8:

./debian/po/fr.po: line 38, char 1, byte offset 18: invalid UTF-8 code
./debian/po/de.po: line 31, char 1, byte offset 46: invalid UTF-8 code
./debian/po/sv.po: line 30, char 1, byte offset 18: invalid UTF-8 code
./debian/po/ja.po: line 32, char 1, byte offset 13: invalid UTF-8 code
./debian/po/cs.po: line 31, char 1, byte offset 22: invalid UTF-8 code
./debian/patches/add-french-traslation.patch: line 35, char 1, byte
offset 6: invalid UTF-8 code
./debian/patches/fix-spelling-error-in-binary.patch: line 18, char 1,
byte offset 44: invalid UTF-8 code
./po/fr.po: line 3, char 1, byte offset 5: invalid UTF-8 code
./po/da.po: line 19, char 1, byte offset 48: invalid UTF-8 code
./po/de.po: line 19, char 1, byte offset 10: invalid UTF-8 code
./po/ru.po: line 18, char 1, byte offset 9: invalid UTF-8 code
./intl/locale.alias: line 33, char 1, byte offset 5: invalid UTF-8 code


-- 
bye,
pabs

http://wiki.debian.org/PaulWise


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to