Opened 14 years ago

Closed 14 years ago

#2804 closed patch (fixed)

Show used GDAL version in About dialog

Reported by: alexbruy Owned by: brushtyler
Priority: minor: annoyance Milestone: Version 1.6.0
Component: Python plugins and bindings Version: Trunk
Keywords: GdalTools Cc:
Must Fix for Release: No Platform: All
Platform Version: Awaiting user input: no

Description

Some users may have several GDAL installed, so it is usefull to show version of GDAL used by GDAL Tools plugin. Proposed patch add this info in About dialog. Also it fixes conversion GDAL version in string.

Attachments (3)

add_gdal_version.diff (2.1 KB ) - added by alexbruy 14 years ago.
Proposed patch
about.diff (671 bytes ) - added by jef 14 years ago.
QGIS' about box also doesn't have that information.
gdaltools_about.diff (2.2 KB ) - added by brushtyler 14 years ago.
alexbrui patch with cleanup

Download all attachments as: .zip

Change History (7)

by alexbruy, 14 years ago

Attachment: add_gdal_version.diff added

Proposed patch

comment:1 by borysiasty, 14 years ago

Owner: changed from borysiasty to brushtyler

by jef, 14 years ago

Attachment: about.diff added

QGIS' about box also doesn't have that information.

comment:2 by jef, 14 years ago

Milestone: Version 1.5.0Version 1.6.0

by brushtyler, 14 years ago

Attachment: gdaltools_about.diff added

alexbrui patch with cleanup

comment:3 by brushtyler, 14 years ago

Status: newassigned

Replying to alexbruy:

Some users may have several GDAL installed, so it is usefull to show version of GDAL used by GDAL Tools plugin. Proposed patch add this info in About dialog. Also it fixes conversion GDAL version in string.

Sure, it's a great idea!

However I think we shouldn't pass the Gdal version as parameter to the doAbout dialog. About dialog can retrieve it using the GdalTools_utils module, so we maintain the same interface for all tools.

In the attached patch there is also a cleanup converting GDAL version in string. If we want to use strings, we must use these everywhere in the Version class, otherwise the comparison may fail. e.g. ('2', 0) is not equal to ('2', '0')

comment:4 by brushtyler, 14 years ago

Resolution: fixed
Status: assignedclosed

Fixed (r13725)

Note: See TracTickets for help on using tickets.