Opened 16 years ago

Closed 15 years ago

#184 closed enhancement (wontfix) - detect gpstrans or gardump instead of user flag

Reported by: kyngchaos Owned by: hamish
Priority: minor Milestone: 6.4.0
Component: Default Version: 6.3.0
Keywords: Cc: grass-dev@…
CPU: Unspecified Platform: Unspecified

Description uses a flag for the user to specify whther to use gardump as an alternate to the default gpstrans to get data from the GPS. This requires the user to know which is installed in their system, or bundled with a binary distribution of GRASS. This can also confuse the user if gardump is installed (whether they realize it or not) and they don't use the flag, then tells them that gpstrans is not installed. Something like:

if [ "`type -t gpstrans`" ] ; then
    if [ "`type -t gardump`" ] ; then
	    g.message -e "either gpstrans or gardump is required, install one first\\"
	exit 1

Then test for USE_GPSTRANS or USE_GARDUMP later in the script for which to use.

Change History (3)

comment:1 by hamish, 16 years ago

Type: defectenhancement

comment:2 by hamish, 16 years ago

Cc: grass-dev@… added
Owner: changed from grass-dev@… to hamish
Status: newassigned

On reflection I think the flag is better. Of course we can improve the error message to suggest using the flag if the download program isn't found.

The two programs (and gpsbabel too for that matter) are not direct equivalents. Each one has its own strengths WRT attribute handling on different garmin models (usually WRT waypoint metadata fields).

I have both gardump and gpstrans installed, so hardcoding the script to run the first one it finds removes my ability to choose. Adding some sort of environment variable to override that hardcoding is a regression from the current situation IMO.

I don't consider it such a bad thing that the user know what 3rd party programs they have installed and are running.


comment:3 by hamish, 15 years ago

CPU: Unspecified
Platform: Unspecified
Resolution: wontfix
Status: assignedclosed

closing bug as "won't fix" for reasons stated earlier in the bug report.


Note: See TracTickets for help on using tickets.