pycousin / gnome-mplayer

Automatically exported from code.google.com/p/gnome-mplayer
GNU General Public License v2.0
0 stars 0 forks source link

Error in 'configure' script on 0.9.8 #351

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Run 'configure'

What is the expected output? What do you see instead?
While checking for pulseaudio and gnome-power-manager, 'configure' returns
the following (non-fatal) errors:

configure: Checking for flat audio support
./configure: line 10055: pulseaudio: command not found
./configure: line 10056: test: -gt: unary operator expected
configure: Checking for gnome-power-manager support
./configure: line 10090: test: Gn: integer expression expected
./configure: line 10097: test: Gn: integer expression expected

What version of the product are you using? On what operating system?
gnome-mplayer 0.9.8 on CentOS 5.2

Please provide any additional information below.
1) The pulseaudio error happens because of the execution of this line:

    if test -x `which pulseaudio`; then

However, the output of `which pulseaudio` is "pulseaudio: Command not
found" ... I'm not certain why, but the test procedure does not fail on
this, and therefore the subsequent version test (within the 'if' block)
generates the error.

2) The gnome-power-manager error happens because of this line:

            GPMVER=`gnome-power-manager --version | sed -e 's/Version 2.//'
| cut -b1-2`

This does not work as expected on my system... on my system,
'gnome-power-manager --version' returns:

Gnome gnome-power-manager 2.16.0

The 'sed' command therefore doesn't replace anything, and the 'cut' command
thus chooses the wrong bytes.  It would be preferable to run this through
'awk' instead, using a regexp to pick out the version number
([0-9]+[.][0-9]+[.][0-9]+), checking to ensure that the first number = 2
and, if so, the second number is within the desired range.  Using a regexp
with awk will prevent the above error regardless of the output format.

Original issue reported on code.google.com by aca...@gmail.com on 30 Jan 2010 at 2:25

GoogleCodeExporter commented 8 years ago
(I should note that I know pulseaudio isn't installed... I bring this up to 
point out
that if pulseaudio is missing, the configure script generates this error rather 
than
gracefully ignoring pulseaudio.)

Original comment by aca...@gmail.com on 30 Jan 2010 at 2:26

GoogleCodeExporter commented 8 years ago
The test for gnome-power-manager has been improved in SVN. The pulseaudio test 
is
still odd but you can disable it.

Original comment by kdeko...@gmail.com on 4 Feb 2010 at 4:04

GoogleCodeExporter commented 8 years ago

Original comment by kdeko...@gmail.com on 10 Feb 2010 at 3:42