Bug#701639: hamexam: Lower dependency on eog

[Edi Meier <liste_franken@gmx.de>]

Package: hamexam
Version: 1.2.0-1
Severity: minor

Dear Maintainer,

some people install instead of gnome from the kde/lxde or xfce-CDs so that
eog is not in their installation.
Therefore I propose something like this:

--- control     2012-01-11 23:04:28.000000000 +0100
+++ control.new 2013-02-25 16:22:46.000000000 +0100
@@ -8,7 +8,7 @@

 Package: hamexam
 Architecture: all
-Depends: ${misc:Depends}, eog
+Depends: ${misc:Depends}, eog | gwenview | gpicview | ristretto
 Description: Study tool for USA FCC amateur radio (ham) exams.
  hamexam is an interactive study tool for the 3 USA FCC amateur radio
  (ham radio) question pools.


--- hamexam     2013-02-25 16:32:19.000000000 +0100
+++ hamexam.new 2013-02-25 16:32:19.000000000 +0100
@@ -5,6 +5,17 @@
 # dir where question pools reside?
 INSTALLDIR="$(dirname "$(readlink -f $0)")"

+# determine viewer
+if type "eog" > /dev/null 2>&1; then
+    VIEWER="eog"
+elif type "gwenview" > /dev/null 2>&1; then
+    VIEWER="gwenview"
+elif type "gpicview" > /dev/null 2>&1; then
+    VIEWER="gpicview"
+else
+    VIEWER="ristretto"
+fi
+
 # dir to keep user's unanswered questions
 USERDIR=~/.hamexam

@@ -82,9 +93,9 @@
 fi

 # bring up diagrams for this element
-eog ${INSTALLDIR}/${POOL}pngs/${POOL}1.png 2>/dev/null &
+${VIEWER} ${INSTALLDIR}/${POOL}pngs/${POOL}1.png 2>/dev/null &
 if [ $? -ne 0 ]; then
-    echo "eog failed to display diagrams"
+    echo "${VIEWER} failed to display diagrams"
 fi

 # report how many questions remain


Thank you very much.

Regards,
Edi



-- System Information:
Debian Release: 7.0
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)