Odd problem with mpg123

joe.krzeszewski · 09-13-2021, 04:06 PM

Greetings all!

I don't know if this is the right place to ask the question, but I have an issue with mpg123 on Slackware Current.

The short version of the story is this:

mpg123 doesn't work if called with the full path but does work if just called.

e.g.

Code:

mpg123 foo.mp3

works while

Code:

/usr/bin/mpg123 foo.mp3

will fail with this:

Code:

/usr/bin/mpg123 *.mp3
High Performance MPEG 1.0/2.0/2.5 Audio Player for Layers 1, 2 and 3
        version 1.29.0; written and copyright by Michael Hipp and others
        free software (LGPL) without any warranty but with best wishes
[src/libout123/module.c:open_module_here():133] error: Failed to open module pulse.
[src/libout123/libout123.c:out123_open():484] error: Found no driver out of [pulse] working with device <default>.
main: [src/mpg123.c:check_fatal_output():331] error: out123 error 3: failure loading driver module

I had a clean 14.2 64 bit system then upgraded to Current. I noticed the problem because calling /usr/bin/mpg123 in cron failed while calling mpg123 on the command line worked. When I called /usr/bin/mpg123 on the command line it failed with the above error.

I can't figure out what is different between the two cases.

Is this something broken in Current, or just me?

Thanks!

gegechris99 · 09-14-2021, 02:43 PM

mpg123 works here with both mpg123 and /usr/bin/mpg123 commands in Slackware64 current.

Something might be missing from your upgrade from 14.2 to current.

As a first step, check that pulse module is available:

Code:

$/usr/bin/mpg123 --list-modules

Available modules
-----------------
alsa           output  Output audio using Advanced Linux Sound Architecture (ALSA).
dummy          output  Dummy audio output - does not output audio.
pulse          output  Output audio using PulseAudio Server
sdl            output  Output audio using SDL (Simple DirectMedia Layer).
oss            output  Output audio using OSS
raw            output  raw headerless stream (builtin)
cdr            output  compact disc digital audio stream (builtin)
wav            output  RIFF WAVE file (builtin)
au             output  Sun AU file (builtin)
test           output  output into the void (builtin)
hex            output  interleaved hex printout (builtin)
txt            output  plain text printout, a column per channel (builtin)

then provide the output of this cat command:

Code:

$cat /etc/asound.conf 
# ALSA system-wide config file
# By default, redirect to PulseAudio:
pcm.default pulse
ctl.default pulse

joe.krzeszewski · 09-15-2021, 12:20 AM

Thanks for the help. It sounds like it is just me.

As suggested...

Code:

> /usr/bin/mpg123 --list-modules

Available modules
-----------------
raw            output  raw headerless stream (builtin)
cdr            output  compact disc digital audio stream (builtin)
wav            output  RIFF WAVE file (builtin)
au             output  Sun AU file (builtin)
test           output  output into the void (builtin)
hex            output  interleaved hex printout (builtin)
txt            output  plain text printout, a column per channel (builtin)

and

Code:

> cat /etc/asound.conf 
# ALSA system-wide config file
# By default, redirect to PulseAudio:
pcm.default pulse
ctl.default pulse

So it looks like /usr/bin/mpg123 doesn't have the output modules it needs... however, the one that works does.

Code:

> mpg123 --list-modules

Available modules
-----------------
alsa           output  Output audio using Advanced Linux Sound Architecture (ALSA).
oss            output  Output audio using OSS
pulse          output  Output audio using PulseAudio Server
dummy          output  Dummy audio output - does not output audio.
sdl            output  Output audio using SDL (Simple DirectMedia Layer).
raw            output  raw headerless stream (builtin)
cdr            output  compact disc digital audio stream (builtin)
wav            output  RIFF WAVE file (builtin)
au             output  Sun AU file (builtin)
test           output  output into the void (builtin)
hex            output  interleaved hex printout (builtin)
txt            output  plain text printout, a column per channel (builtin)

This feels for all the world like I somehow have two copies of mpg123 hanging around.

I would have thought that /usr/bin/mpg123 would have been upgraded when I changed to Current. Then again, I am not sure it would have even worked in 14.2 without alsa or oss modules.

Petri Kaukasoina · 09-15-2021, 12:44 AM

Try

Code:

type mpg123

joe.krzeszewski · 09-15-2021, 12:45 AM

Aha! Progress...

I figured out that if I install the "mpg123-compat32-1.29.0-x86_64-1compat32" from "multilib" it breaks /usr/bin/mpg123 but not mpg123 without the full path.
If I uninstall "mpg123-compat32-1.29.0-x86_64-1compat32" both mpg123 and /usr/bin/mpg123 work fine.

So this is a multilib issue?

gsl · 09-15-2021, 12:57 AM

Quote:

Originally Posted by joe.krzeszewski

Thanks for the help. It sounds like it is just me.

It's not just you. I see the same thing with "mpg123 --list-modules" showing pulse, alsa, etc. and "/usr/bin/mpg123 --list-modules" not showing them.

Code:

> which mpg123
/usr/bin/mpg123

My system is multilib and "/usr/bin/32/mpg123 --list-modules" shows all modules and plays mp3s fine, but "/usr/bin/mpg123" doesn't work.

No idea why it happens or whether it is evidence of a problem but it does seem strange.

Geoff.

drgibbon · 09-15-2021, 02:35 AM

Quote:

Originally Posted by joe.krzeszewski

If I uninstall "mpg123-compat32-1.29.0-x86_64-1compat32" both mpg123 and /usr/bin/mpg123 work fine.

You can check which binary is being called with 'type' (or 'which'). From the sounds of it, without mpg123-compat32-1.29.0-x86_64-1compat32 installed `type mpg123` will return "/usr/bin/mpg123", whereas with that package installed `type mpg123` will return "/usr/bin/32/mpg123"?

*edit* Hmm no, just read the post above, it must break mpg123 in a different way..

Petri Kaukasoina · 09-15-2021, 06:35 AM

Maybe you have /etc/profile.d/32dev.sh executable? If you are not building 32-bit stuff just now, don't.

kingbeowulf · 09-16-2021, 07:34 PM

Interesting...I was able to reproduce the OP issue here. This seems to be a multilib quirk. All Current systems updated as of 7 days ago. /etc/profile.d/32dev.sh is not executable

System 1: Slackware64-Current w/ mulitlib, clean install when current hit kernel 5.4 last year.
mpg123 works
/usr/bin/mpg123 fails

System 2: Slackware64-Current pure no multilib
mpg123 works
/usr/bin/mpg123 works

System 3: Slackware64-Current w/ mulitlib, clean install a few months ago
mpg123 works
/usr/bin/mpg123 fails

On the multilib systems, '/usr/bin/32/mpg123' works as well, and sees the full module list. Module listing on System 2 (no multilib) provides identical output for mpg123 and /usr/bin/mpg123.

Weird stuff.

gsl · 09-16-2021, 08:02 PM

As another data point, I checked my work PC running 14.2 multilib with mpg123 1.23.4 and all cases played mp3s fine (mpg123, /usr/bin/mpg123 & /usr/bin/32/mpg123).

Geoff.

Pithium · 09-17-2021, 09:57 PM

I think I found the source of the problem. On runtime mpg123 scans for its module directory. If $MPG123_MODDIR is not specified then it tries to scan based on the path specified by the user. You can manually override this by adding the env variable like this:

Code:

   
$ mpg123 --list-modules

Available modules
-----------------
alsa           output  Output audio using Advanced Linux Sound Architecture (ALSA).
dummy          output  Dummy audio output - does not output audio.
oss            output  Output audio using OSS
pulse          output  Output audio using PulseAudio Server
sdl            output  Output audio using SDL (Simple DirectMedia Layer).
raw            output  raw headerless stream (builtin)
cdr            output  compact disc digital audio stream (builtin)
wav            output  RIFF WAVE file (builtin)
au             output  Sun AU file (builtin)
test           output  output into the void (builtin)
hex            output  interleaved hex printout (builtin)
txt            output  plain text printout, a column per channel (builtin)
$ /usr/bin/mpg123 --list-modules

Available modules
-----------------
raw            output  raw headerless stream (builtin)
cdr            output  compact disc digital audio stream (builtin)
wav            output  RIFF WAVE file (builtin)
au             output  Sun AU file (builtin)
test           output  output into the void (builtin)
hex            output  interleaved hex printout (builtin)
txt            output  plain text printout, a column per channel (builtin)
$ MPG123_MODDIR=/usr/lib64/mpg123/ /usr/bin/mpg123 --list-modules

Available modules
-----------------
alsa           output  Output audio using Advanced Linux Sound Architecture (ALSA).
dummy          output  Dummy audio output - does not output audio.
oss            output  Output audio using OSS
pulse          output  Output audio using PulseAudio Server
sdl            output  Output audio using SDL (Simple DirectMedia Layer).
raw            output  raw headerless stream (builtin)
cdr            output  compact disc digital audio stream (builtin)
wav            output  RIFF WAVE file (builtin)
au             output  Sun AU file (builtin)
test           output  output into the void (builtin)
hex            output  interleaved hex printout (builtin)
txt            output  plain text printout, a column per channel (builtin)
$ MPG123_MODDIR=/usr/lib/mpg123/ /usr/bin/mpg123 --list-modules

Available modules
-----------------
raw            output  raw headerless stream (builtin)
cdr            output  compact disc digital audio stream (builtin)
wav            output  RIFF WAVE file (builtin)
au             output  Sun AU file (builtin)
test           output  output into the void (builtin)
hex            output  interleaved hex printout (builtin)
txt            output  plain text printout, a column per channel (builtin)

Pithium · 09-17-2021, 10:31 PM

I think the problem might be upstream. If you don't specify the environment it tries to search relative to the binpath. modulesearch[] is not aware of 64bit paths.

lib64_moddir_fix.patch

Code:

diff -Naur mpg123-1.28.2-original/src/libout123/module.c mpg123-1.28.2/src/libout123/module.c
--- mpg123-1.28.2-original/src/libout123/module.c	2021-07-11 00:17:04.000000000 -0700
+++ mpg123-1.28.2/src/libout123/module.c	2021-09-17 20:04:45.764270212 -0700
@@ -29,7 +29,8 @@
    to wide and then replace / by \. No need to define another list. */
 static const char* modulesearch[] =
 {
-	 "../lib/mpg123"
+      "../lib64/mpg123"
+	, "../lib/mpg123"
 	,"plugins"
 	,"libout123/modules/.libs"
 	,"libout123/modules"

patch -p1 < lib64_moddir_fix.patch in source/ap/mpg123/mpg123.SlackBuild seems to solve the problem.