Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 345009 - media-libs/alsa-lib: typo in pkg_postinst(): "alsa-drivers" instead of "alsa-driver"
Summary: media-libs/alsa-lib: typo in pkg_postinst(): "alsa-drivers" instead of "alsa-...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Gentoo ALSA team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-11 07:44 UTC by David
Modified: 2011-02-17 17:28 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David 2010-11-11 07:44:48 UTC
There is a slight misspelling in the alsa-lib ebuild:  the pkg_postinst() section shows "alsa-drivers"  when the package is actually "alsa-driver"   (no "s")


I almost filed a bug to say there is the "alsa-drivers" package does not exist and so the pkg_postinst() section of alsa-lib can be removed.  But then I realized it was just a misspelling    :-)

Reproducible: Always

Steps to Reproduce:
1. emerge alsa-lib

Actual Results:  
pkg_postinst() {
        elog "Please try in-kernel ALSA drivers instead of the alsa-drivers ebuild."
        elog "If alsa-drivers works for you where a *recent* kernel does not, we want "
        elog "to know about this. Our e-mail address is alsa-bugs@gentoo.org"
        elog "However, if you notice no sound output or instability, please try to "
        elog "upgrade your kernel to a newer version first."
}


Expected Results:  
pkg_postinst() {
        elog "Please try in-kernel ALSA drivers instead of the alsa-driver ebuild."
        elog "If alsa-driver works for you where a *recent* kernel does not, we want "
        elog "to know about this. Our e-mail address is alsa-bugs@gentoo.org"
        elog "However, if you notice no sound output or instability, please try to "
        elog "upgrade your kernel to a newer version first."
}


There are two occurrences of this misspelling in the ebuild.

I don't know if other ALSA-related ebuilds have this misspelling as well.

Perhaps this message could be displayed only if the user has alsa-driver installed?  I don't have it installed and *do* have the in-kernel ALSA drivers installed, but I still get the warning to use the in-kernel drivers "instead" of the alsa-driver package.

Perhaps rather than fixing the misspelling, the name of the package actually should be plural?  (I don't know)
Comment 1 Sebastian Pipping gentoo-dev 2011-02-17 17:28:20 UTC
Fixing the typo only.

+  17 Feb 2011; Sebastian Pipping <sping@gentoo.org> alsa-lib-1.0.20-r1.ebuild,
+  alsa-lib-1.0.21a.ebuild, alsa-lib-1.0.22.ebuild, alsa-lib-1.0.22-r1.ebuild,
+  alsa-lib-1.0.23.ebuild, alsa-lib-1.0.24.1.ebuild:
+  Fix typo alsa-drivers to alsa-driver (bug #345009)
+