[GRASS-dev] Re: [GRASS-CVS] michael: grass6/lib/init gis_set.tcl, 1.24, 1.25

[Markus Neteler]

Hi,

a few questions:


On Mon, Jul 17, 2006 at 07:28:42PM +0200, grass at intevation.de wrote:
> Author: michael
> 
> Update of /grassrepository/grass6/lib/init
> In directory doto:/tmp/cvs-serv30054
> 
> Modified Files:
> 	gis_set.tcl 
> Log Message:
> update exec and eval exec statements to properly parse arguments
> 
> Index: gis_set.tcl
> ===================================================================
> RCS file: /grassrepository/grass6/lib/init/gis_set.tcl,v
> retrieving revision 1.24
> retrieving revision 1.25
> diff -u -d -r1.24 -r1.25
> --- gis_set.tcl	24 Jun 2006 19:33:59 -0000	1.24
> +++ gis_set.tcl	17 Jul 2006 17:28:40 -0000	1.25
> @@ -312,10 +312,10 @@
>      pack $introtitle -side top
>  
>      .frame0.intro.msg tag configure all -justify center
> -    .frame0.intro.msg insert end [G_msg "Welcome to GRASS GIS Version $GRASSVERSION\n"]
> +    .frame0.intro.msg insert end [G_msg "Welcome to GRASS GIS version $GRASSVERSION\n"]
>      .frame0.intro.msg insert end [G_msg "The world's leading open source GIS\n\n"]
> -    .frame0.intro.msg insert end [G_msg "Select an existing project location and mapset\n"]
> -    .frame0.intro.msg insert end [G_msg "or define a new project location\n"]
> +    .frame0.intro.msg insert end [G_msg "Select an existing projection location and GIS mapset\n"]

... what does "existing projection location" mean?
I only know "existing project location". The change adds confusion in my opinion.
If projected is meant, it is not right since latlong (usually) and xy (always) are
unprojected.



> +    .frame0.intro.msg insert end [G_msg "or define a new projection location\n"]

...also here.

>      .frame0.intro.msg tag add all 1.0 end
>      .frame0.intro.msg configure -state disabled
>  
> @@ -338,7 +338,7 @@
>      label .frame0.frameDB.left.label \
>      	-anchor {n} \
>      	-justify right \
> -    	-text [G_msg "Path to location:    \n(GRASS Database)"]
> +    	-text [G_msg "Path to location : "]

"GRASS Database" is a widely used term in the GRASS literature,
but sure if it should be removed.


>      entry .frame0.frameDB.mid.entry \
>      	-relief {sunken} \
> @@ -408,7 +408,7 @@
>  
>      label .frame0.frameMS.label \
>      	-anchor {w} \
> -    	-text [G_msg "Accessible Mapsets"] 
> +    	-text [G_msg "(Accessible) GIS Mapsets"] 

... this re-reverted the last fix...
The () do not add information in my opionion.


>      listbox .frame0.frameMS.listbox \
>      	-relief {sunken} \
> @@ -460,7 +460,7 @@
>  
>      label .frame0.frameNMS.first.label \
>      	-anchor {n} \
> -    	-text [G_msg "Create new mapset\nin selected location"]
> +    	-text [G_msg "Create new GIS mapset\nin currrent location"]

I am also not sure of that. In the beginning nothing is selected,
so there is no currrent location.
  
>      entry .frame0.frameNMS.second.entry \
>      	-relief {sunken} \
> @@ -495,7 +495,7 @@
>  
>      label .frame0.frameNMS.fourth.label \
>      	-anchor {n} \
> -    	-text [G_msg "\nDefine new location by:"]
> +    	-text [G_msg "Define new projection location"]

... again confusing (for me).

>  
>      button .frame0.frameNMS.fifth.button \
>

It seems that this needs to be discussed again: at least I
would like to better understand most of the changes.

thanks

 Markus