RM: irda-utils and ircp-tray (was: irda-utils ftbfs in eoan)

Bug #1843742 reported by Matthias Klose
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ircp-tray (Ubuntu)
Fix Released
Medium
Andy Whitcroft
irda-utils (Ubuntu)
Fix Released
Medium
Andy Whitcroft

Bug Description

https://launchpadlibrarian.net/441262268/buildlog_ubuntu-eoan-amd64.irda-utils_0.9.18-15ubuntu1_BUILDING.txt.gz

gcc -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fgnu89-inline -W -Wall -I. `pkg-config --cflags glib-2.0` -I/usr/include -I../include/ -c irdadump.c
irdadump.c: In function ‘print_time’:
irdadump.c:82:13: warning: g_string_append_printf
   82 | g_string_sprintfa(str, "%02d:%02d:%02d.%06u ",
      | ^~~~~~~~~~~~~~~~~~~~~~~~
irdadump.c: In function ‘print_diff_time’:
irdadump.c:108:13: warning: g_string_append_printf
  108 | g_string_sprintfa(str, "(%07.2f ms) ", diff);
      | ^~~~~~~~~~~~~~~~~~~~~~~~
irdadump.c: In function ‘parse_irda_frame’:
irdadump.c:178:13: warning: g_string_append_printf
  178 | g_string_sprintfa(str, "(%d) ", len);
      | ^~~~~~~~~~~~~~~~~~~~~~~~
irdadump.c:189:13: warning: g_string_append_printf
  189 | g_string_sprintfa(str, "%02x", frame_buf->head[i]);
      | ^~~~~~~~~~~~~~~~~~~~~~~~
irdadump.c:196:13: warning: g_string_append_printf
  196 | g_string_sprintfa(str, " %c", c);
      | ^~~~~~~~~~~~~~~~~~~~~~~~
irdadump.c:207:13: warning: g_string_append_printf
  207 | g_string_sprintfa(str, "%02x ", frame_buf->head[i]);
      | ^~~~~~~~~~~~~~~~~~~~~~~~
irdadump.c: In function ‘irdadump_loop’:
irdadump.c:286:16: error: ‘SIOCGSTAMP’ undeclared (first use in this function); did you mean ‘SIOCGARP’?
  286 | if (ioctl(fd, SIOCGSTAMP, curr_time) < 0) {
      | ^~~~~~~~~~
      | SIOCGARP
irdadump.c:286:16: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [Makefile:48: irdadump.o] Error 1
make[1]: Leaving directory '/<<PKGBUILDDIR>>/irdadump'

Matthias Klose (doko)
Changed in irda-utils (Ubuntu):
status: New → Confirmed
importance: Undecided → High
tags: added: ftbfs rls-ee-incoming
Matthias Klose (doko)
tags: added: rls-ff-incoming
Revision history for this message
Stefan Bader (smb) wrote :

The irda-utils package was removed in Debian (https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=907453) mentioning the lack of kernel support. Checking the upstream kernel repo, the following commit was part of v4.17:

commit d64c2a76123f0300b08d0557ad56e9d599872a36
Author: Greg Kroah-Hartman <email address hidden>
Date: Wed Mar 14 13:12:26 2018 +0100

    staging: irda: remove the irda network stack and drivers

    No one has publicly stepped up to maintain this broken codebase for
    devices that no one uses anymore, so let's just drop the whole thing.

    If someone really wants/needs it, we can revert this and they can fix
    the code up to work properly.

    Cc: David S. Miller <email address hidden>
    Signed-off-by: Greg Kroah-Hartman <email address hidden>

This would mean that no Ubuntu kernel after 18.04/Bionic has kernel support. Proposing to remove this package from Groovy onward.

Changed in irda-utils (Ubuntu):
assignee: nobody → Stefan Bader (smb)
Revision history for this message
Stefan Bader (smb) wrote :

There seems to be one package which depends on irda-utils (ircp-tray in universe/misc). It was removed from testing in Debian due to the removal of irda-utils there. Seems to make no sense to keep any tray icon around which shows irda activity without userspace tools and kernel support around.

Revision history for this message
Stefan Bader (smb) wrote :

Exact status in Debian:
irda-utils | 0.9.18-12 | oldoldstable | source, amd64, armel, armhf, i386
irda-utils | 0.9.18-15 | oldstable | source, amd64, arm64, armel, armhf, i386, mips, mips64el, mipsel, ppc64el, s390x

ircp-tray | 0.7.6-1.1 | oldoldstable | source, amd64, armel, armhf, i386
ircp-tray | 0.7.6-1.2 | oldstable | source
ircp-tray | 0.7.6-1.2 | unstable | source
ircp-tray | 0.7.6-1.2 | unstable-debug | source
ircp-tray | 0.7.6-1.2+b1 | oldstable | amd64, arm64, armel, armhf, i386, mips, mips64el, mipsel, ppc64el, s390x
ircp-tray | 0.7.6-1.2+b1 | unstable | amd64, arm64, armel, armhf, i386, mips64el, mipsel, ppc64el, s390x

For Ubuntu the proposal would be to remove both source and binaries of irda-utils and ircp-tray from Groovy.

Revision history for this message
Andy Whitcroft (apw) wrote :

Confirmed no external reverse-depends and removed in Debian too. Removing as requested.

summary: - irda-utils ftbfs in eoan
+ RM: irda-utils and ircp-tray (was: irda-utils ftbfs in eoan)
Changed in ircp-tray (Ubuntu):
status: New → In Progress
Changed in irda-utils (Ubuntu):
status: Confirmed → In Progress
Changed in ircp-tray (Ubuntu):
importance: Undecided → Medium
Changed in irda-utils (Ubuntu):
importance: High → Medium
assignee: Stefan Bader (smb) → Andy Whitcroft (apw)
Changed in ircp-tray (Ubuntu):
assignee: nobody → Andy Whitcroft (apw)
Revision history for this message
Andy Whitcroft (apw) wrote :

Removing packages from groovy:
        irda-utils 0.9.18-15ubuntu1 in groovy
                irda-utils 0.9.18-15ubuntu1 in groovy amd64
                irda-utils 0.9.18-15ubuntu1 in groovy arm64
                irda-utils 0.9.18-15ubuntu1 in groovy armhf
                irda-utils 0.9.18-15ubuntu1 in groovy ppc64el
                irda-utils 0.9.18-15ubuntu1 in groovy s390x
        ircp-tray 0.7.6-1.2ubuntu3 in groovy
                ircp-tray 0.7.6-1.2ubuntu3 in groovy amd64
                ircp-tray 0.7.6-1.2ubuntu3 in groovy arm64
                ircp-tray 0.7.6-1.2ubuntu3 in groovy armhf
                ircp-tray 0.7.6-1.2ubuntu3 in groovy ppc64el
                ircp-tray 0.7.6-1.2ubuntu3 in groovy s390x
Comment: No longer supported upstream, removed in Debian, ftbfs in grooby; LP: #1843742
2 packages successfully removed.

Steve Langasek (vorlon)
Changed in irda-utils (Ubuntu):
status: In Progress → Fix Released
Changed in ircp-tray (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.