findmnt: libsmartcols ASCII art problem [patch]

[Roman Odaisky]

The libsmartcols component used by findmnt to draw the filesystem tree is smart enough to wrap lines in case the output doesn’t fit into the screen. However, it fails to draw the lines correctly in this case. Attached patch fixes the problem. Someone more knowledgeable in libsmartcols internals might want to change the buffer size from an arbitrary 1024 to something taken from actual data, the best way to do that eludes me.

Sample of incorrect output:

--%<----
├─/boot /dev/sda1 ext3 rw,relatime,data=ordered
├─/usr /dev/mapper/vgshadow-ubuntu_usr
                                                ext4 rw,noatime,data=ordered
├─/var /dev/mapper/vgshadow-ubuntu_var
                                                ext4 rw,noatime,data=ordered
├─/home /dev/mapper/vgshadow-home
                                                ext4 rw,noatime,data=ordered
└─/srv /dev/mapper/vgshadow-srv
                                                reiserfs rw,noatime,notail
--%<----

Patch adds missing vertical lines at any level necessary.

[Roman Odaisky (rdaysky)]

Attaching full sample of incorrect output in a separate file.

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "libsmartcols-art-fix.diff" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

[Martin Pitt (pitti)]

Thanks! Forwarded to upstream: https://github.com/karelzak/util-linux/issues/137

[Martin Pitt (pitti)]

Fixed upstream in https://github.com/karelzak/util-linux/commit/8501d9b

Thanks!