Merge lp:~skellat/ffscreencast/test into lp:ffscreencast

Proposed by Stephen Michael Kellat
Status: Merged
Merge reported by: Stephen Michael Kellat
Merged at revision: not available
Proposed branch: lp:~skellat/ffscreencast/test
Merge into: lp:ffscreencast
Diff against target: 25 lines (+3/-3)
1 file modified
ffscreencast.sh (+3/-3)
To merge this branch: bzr merge lp:~skellat/ffscreencast/test
Reviewer Review Type Date Requested Status
Alan Pope 🍺🐧🐱 πŸ¦„ Approve
Review via email: mp+84391@code.launchpad.net

Description of the change

Trying to clean up error messages in the prerequisites check to make them more informative.

To post a comment you must log in.
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ffscreencast.sh'
2--- ffscreencast.sh 2011-12-03 13:56:47 +0000
3+++ ffscreencast.sh 2011-12-04 05:59:23 +0000
4@@ -8,19 +8,19 @@
5 SOX=$(which sox)
6
7 if [ $? == "1" ]; then
8- echo 'sox missing'
9+ echo 'sox appears to be missing. To correct this please install the following packages: sox, libsox-fmt-all'
10 exit 1
11 fi
12
13 FFMPEG=$(which ffmpeg)
14 if [ $? == "1" ]; then
15- echo 'ffmpeg missing'
16+ echo 'ffmpeg appears to be missing. To correct this please install the following packages: ffmpeg, libxvidcore4 libfaac0 '
17 exit 1
18 fi
19
20 PAREC=$(which parec)
21 if [ $? == "1" ]; then
22- echo 'parec missing'
23+ echo 'parec appears to be missing. PulseAudio will be necessary for this script to function. To correct this please install the following packages: pulseaudio-utils '
24 exit 1
25 fi
26

Subscribers

People subscribed via source and target branches