Bug#702112: RFS: tpb/0.6.4-9 -- program to use the IBM ThinkPad(tm) special keys
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 & acelerator</error>
<error line="582" message="123">extra & acelerator</error>
<error line="587" message="124">extra & acelerator</error>
<error line="592" message="125">extra & 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
Reply to: