Control: tags -1 moreinfo
On Fri, 21 Oct 2016 14:46:32 +0200 Alex Mestiashvili
<mailatgoogl@gmail.com> wrote:
> Package: sponsorship-requests
> Severity: wishlist
> X-Debbugs-Cc: mailatgoogl@gmail.com
>
> Dear mentors,
>
> [...]
>
> Best regards,
> Alex
Hi Alex,
Thanks for your contribution.
Have you informed upstream about the following compiler warning?
"""
hdparm.c: In function 'process_dev':
hdparm.c:2256:2: warning: this 'if' clause does not guard...
[-Wmisleading-indentation]
if (!quiet)
^~
hdparm.c:2258:3: note: ...this statement, but the latter is misleadingly
indented as if it is guarded by the 'if'
if (do_drive_cmd(fd, args, 0)) {
^~
"""
I checked it seems benign, so I am not blocking the upload on this. The
code in question being:
"""
if (security_freeze) {
__u8 args[4] = {ATA_OP_SECURITY_FREEZE_LOCK,0,0,0};
if (!quiet)
^^^^^^^^^^^ (should be 1 tab further in)
printf(" issuing security freeze command\n");
^^^^^^^^^^^^^^ (should be 1 tab further in as well)
if (do_drive_cmd(fd, args, 0)) {
err = errno;
perror(" HDIO_DRIVE_CMD[...] failed");
}
}
"""
Other minor nits:
* Spelling error in hdparm binary "Removeable" -> "Removable"
- NB: Upstream is inconsistent with the spelling, but seems to prefer
the "Removable" variant.
* Upstream's build system hardcodes -j2 on "make all" and the packaging
uses that without any respect to DEB_BUILD_OPTIONS
- Given the small size and low parallel limit, I don't think it is a
huge issue (but it would have been for other packages).
* The debian/hdparm.preinst file appears to be redundant (its replaced
by the debian/hdparm.maintscript)