usr/lib/byobu/release includes more than the release when lsb_release is used

Bug #1068581 reported by Daniel Hahler on 2012-10-19
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu
Medium
Dustin Kirkland 

Bug Description

__release in usr/lib/byobu/release will output more than only the release in case /etc/issue is not used.

The patch to fix this looks as follows:

=== modified file 'usr/lib/byobu/release'
--- usr/lib/byobu/release 2012-06-24 20:05:52 +0000
+++ usr/lib/byobu/release 2012-10-19 11:21:48 +0000
@@ -46,20 +46,7 @@
        fi
        if [ -z "$RELEASE" ] && $BYOBU_TEST lsb_release >/dev/null 2>&1; then
                # If lsb_release is available, use it
- local r=$(lsb_release -s -d)
- case "$r" in
- Ubuntu*)
- # Use the -d if an Ubuntu LTS
- RELEASE="$r"
- ;;
- *)
- # But for other distros the description
- # is too long, so build from -i and -r
- local i=$(lsb_release -s -i)
- r=$(lsb_release -s -r)
- RELEASE="$i $r"
- ;;
- esac
+ RELEASE=$(lsb_release -s -r)
        fi
        if [ -n "$RELEASE_ABBREVIATED" ] && [ $RELEASE_ABBREVIATED -gt 0 ]; then
                color bold2; printf "%.${RELEASE_ABBREVIATED}s" "$RELEASE"; color --

This affects e.g. CentOS 5.4.

Dustin Kirkland  (kirkland) wrote :

Committed revision 1985.

Changed in byobu:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Dustin Kirkland (kirkland)
status: In Progress → Fix Committed
Changed in byobu:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers