Details Ticket 4110


Comment | Reply | Take | Open


Serial Number 4110
Subject v.digit: Could not set Tcl system encoding to
Area grass6
Queue grass
Requestors david.p.finlayson@gmail.com,glynn@gclements.plus.com,neteler@itc.it,roux@lmtg.obs-mip.fr,tutey@o2.pl
Owner none
Status resolved
Last User Contact Sat Feb 17 16:01:31 2007 (2 yr ago)
Current Priority 30
Final Priority 70
Due No date assigned
Last Action Sat Feb 17 16:01:31 2007 (2 yr ago)
Created Thu Mar 31 21:57:11 2005 (3 yr ago)

Transaction History Ticket 4110


Thu, Mar 31 2005 21:57:11    Request created by guest (as #3131)  
Subject: iso-8859-2 needed for attribute databases

Platform: GNU/Linux/i386
grass obtained from: Mirror of Trento site
grass binary for platform: Compiled from Sources
GRASS Version: 6.1cvs

iso-8859-2 encoding would be very appreciated by Central/Eastern Europe Grass
users. That would allow us to use national letters in attributes
databases. City/geographical/species/people etc. names without national letters
are often not recognizable.

The problem touches commands like:

- d.what.vect -e

- v.digit:

GRASS6.1.cvs(xy):~> g.gisenv set=GRASS_DB_ENCODING=iso-8859-2
GRASS6.1.cvs(xy):~> v.digit -n pokus
Map does not exist.
New empty map created.
cat integer , majitel varchar ( 50 )
UPOZORNĚNÍ: Výchozí ovladač / databáze nastavena na:
            ovladač: dbf
            databáze: $GISDBASE/$LOCATION_NAME/$MAPSET/dbf/
Could not set Tcl system encoding to iso-8859-2
Building topology ...

Maciek Sieczka
Fri, Apr 1 2005 14:32:42    Mail sent by blazek@itc.it (as #3131)  
Return-Path <blazek@itc.it>
Delivered-To grass-bugs@lists.intevation.de
Message-ID <424D3F66.5060400@itc.it>
Date Fri, 01 Apr 2005 14:32:38 +0200
From Radim Blazek <blazek@itc.it>
User-Agent Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6) Gecko/20040113
X-Accept-Language en-us, en
MIME-Version 1.0
To Request Tracker <grass-bugs@intevation.de>
Cc grass5@grass.itc.it
Subject Re: [GRASS5] [bug #3131] (grass) iso-8859-2 needed for attribute databases
References <20050331195711.D258D1006DD@lists.intevation.de>
In-Reply-To <20050331195711.D258D1006DD@lists.intevation.de>
Content-Type text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding 8bit
X-Spam-Status No, hits=-4.9 tagged_above=-999.0 required=3.0 tests=BAYES_00
X-Spam-Level
I have fixed that in QGIS.

Radim


Request Tracker wrote:
> this bug's URL: http://intevation.de/rt/webrt?serial_num=3131
> -------------------------------------------------------------------------
> 
> Subject: iso-8859-2 needed for attribute databases
> 
> Platform: GNU/Linux/i386
> grass obtained from: Mirror of Trento site
> grass binary for platform: Compiled from Sources
> GRASS Version: 6.1cvs
> 
> iso-8859-2 encoding would be very appreciated by Central/Eastern Europe Grass
users. That would allow us to use national letters in attributes
> databases. City/geographical/species/people etc. names without national letters
are often not recognizable.
> 
> The problem touches commands like:
> 
> - d.what.vect -e
> 
> - v.digit:
> 
> GRASS6.1.cvs(xy):~> g.gisenv set=GRASS_DB_ENCODING=iso-8859-2
> GRASS6.1.cvs(xy):~> v.digit -n pokus
> Map does not exist.
> New empty map created.
> cat integer , majitel varchar ( 50 )
> UPOZORNĚNÍ: Výchozí ovladač / databáze nastavena na:
>             ovladač: dbf
>             databáze: $GISDBASE/$LOCATION_NAME/$MAPSET/dbf/
> Could not set Tcl system encoding to iso-8859-2
> Building topology ...
> 
> Maciek Sieczka
> 
> -------------------------------------------- Managed by Request Tracker
> 
> _______________________________________________
> grass5 mailing list
> grass5@grass.itc.it
> http://grass.itc.it/mailman/listinfo/grass5


Sun, Apr 3 2005 10:26:55    Mail sent by werchowyna@epf.pl (as #3131)  
Return-Path <werchowyna@epf.pl>
Delivered-To grass-bugs@lists.intevation.de
Message-ID <000401c53827$017bfcb0$43d21d3e@eustahiush>
From "Maciek Sieczka" <werchowyna@epf.pl>
To "Radim Blazek" <blazek@itc.it>, "Request Tracker" <grass-bugs@intevation.de>
Cc "grass devel" <grass5@grass.itc.it>
References <20050331195711.D258D1006DD@lists.intevation.de> <424D3F66.5060400@itc.it>
Subject Re: [GRASS5] [bug #3131] (grass) iso-8859-2 needed for attribute databases
Date Sun, 3 Apr 2005 10:07:07 +0200
MIME-Version 1.0
Content-Type text/plain; format=flowed; charset="ISO-8859-1"; reply-type=original
Content-Transfer-Encoding 8bit
X-Priority 3
X-MSMail-Priority Normal
X-Mailer Microsoft Outlook Express 6.00.2900.2180
X-MimeOLE Produced By Microsoft MimeOLE V6.00.2900.2180
X-Antivirus avast! (VPS 0513-1, 2005-03-30), Outbound message
X-Antivirus-Status Clean
X-Spam-Status No, hits=-4.9 tagged_above=-999.0 required=3.0 tests=BAYES_00
X-Spam-Level
>I have fixed that in QGIS.
>
> Radim

that's cool but I'm asking about Grass
d.what.vect and querying from the GIS Manager in particular

Maciek

>
>
> Request Tracker wrote:
>> this bug's URL: http://intevation.de/rt/webrt?serial_num=3131
>> -------------------------------------------------------------------------
>>
>> Subject: iso-8859-2 needed for attribute databases
>>
>> Platform: GNU/Linux/i386
>> grass obtained from: Mirror of Trento site
>> grass binary for platform: Compiled from Sources
>> GRASS Version: 6.1cvs
>>
>> iso-8859-2 encoding would be very appreciated by Central/Eastern Europe
>> Grass users. That would allow us to use national letters in attributes
>> databases. City/geographical/species/people etc. names without national
>> letters are often not recognizable.
>>
>> The problem touches commands like:
>>
>> - d.what.vect -e
>>
>> - v.digit:
>>
>> GRASS6.1.cvs(xy):~> g.gisenv set=GRASS_DB_ENCODING=iso-8859-2
>> GRASS6.1.cvs(xy):~> v.digit -n pokus
>> Map does not exist.
>> New empty map created.
>> cat integer , majitel varchar ( 50 )
>> UPOZORNĚNÍ: Výchozí ovladač / databáze nastavena na:
>>             ovladač: dbf
>>             databáze: $GISDBASE/$LOCATION_NAME/$MAPSET/dbf/
>> Could not set Tcl system encoding to iso-8859-2
>> Building topology ...
>>
>> Maciek Sieczka
>>
>> -------------------------------------------- Managed by Request Tracker
>>
>> _______________________________________________
>> grass5 mailing list
>> grass5@grass.itc.it
>> http://grass.itc.it/mailman/listinfo/grass5
>
> _______________________________________________
> grass5 mailing list
> grass5@grass.itc.it
> http://grass.itc.it/mailman/listinfo/grass5
>


Wed, Feb 22 2006 11:25:59    Request created by guest  
Subject: v.digit: Could not set Tcl system encoding to

Platform: GNU/Linux/x86
grass obtained from: CVS
grass binary for platform: Compiled from Sources
GRASS Version: 2006-02-20

When editing attributes in v.digit plenty of "Could not set Tcl system encoding
to" are printed to stdout (3 per 1 edit action). An error? Warning? Why doesn't
it say what encoding it couldn't set to, actually?

v.digit map=huba -n 
D0/0: Name = cat
D0/0: Name = buba
D0/0: Name = _grass_internal_database_encoding
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to 
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to 
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to 

I have my GRASS_DB_ENCODING set:

GRASS 6.1.cvs (caves_utm33):~ > echo $GRASS_DB_ENCODING
utf-8

Maciek

P.S.

The new "Edit line/boundary" tool is excellent! Thanks to Ralf Gerlich and Markus
Neteler!

Maciek
Wed, Feb 22 2006 14:08:26    Mail sent by glynn@gclements.plus.com  
Return-Path <glynn@gclements.plus.com>
Delivered-To grass-bugs@lists.intevation.de
From Glynn Clements <glynn@gclements.plus.com>
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Transfer-Encoding 7bit
Message-ID <17404.25156.163419.956351@cerise.gclements.plus.com>
Date Wed, 22 Feb 2006 13:08:20 +0000
To Request Tracker <grass-bugs@intevation.de>
Cc grass5@grass.itc.it
Subject Re: [GRASS5] [bug #4110] (grass) v.digit: Could not set Tcl system encoding to
In-Reply-To <20060222102559.268201006A0@lists.intevation.de>
References <20060222102559.268201006A0@lists.intevation.de>
X-Mailer VM 7.07 under 21.4 (patch 15) "Security Through Obscurity" XEmacs Lucid
X-Spam-Status No, hits=-4.9 tagged_above=-999.0 required=3.0 tests=BAYES_00
X-Spam-Level
Request Tracker wrote:

> this bug's URL: http://intevation.de/rt/webrt?serial_num=4110
> -------------------------------------------------------------------------
> 
> Subject: v.digit: Could not set Tcl system encoding to
> 
> Platform: GNU/Linux/x86
> grass obtained from: CVS
> grass binary for platform: Compiled from Sources
> GRASS Version: 2006-02-20
> 
> When editing attributes in v.digit plenty of "Could not set Tcl
> system encoding to" are printed to stdout (3 per 1 edit action). An
> error? Warning? Why doesn't it say what encoding it couldn't set to,
> actually?
> 
> v.digit map=huba -n 
> D0/0: Name = cat
> D0/0: Name = buba
> D0/0: Name = _grass_internal_database_encoding
> D0/0: Name = _grass_internal_database_encoding
> Could not set Tcl system encoding to 
> D0/0: Name = _grass_internal_database_encoding
> Could not set Tcl system encoding to 
> D0/0: Name = _grass_internal_database_encoding
> Could not set Tcl system encoding to 

The two points in the code (lib/form/form.c) which generate this error
read:

		if ( Tcl_SetSystemEncoding(interp, Cols[i].value) == TCL_ERROR ) {
			fprintf(stderr, 
				"Could not set Tcl system encoding to %s\n", Cols[i].value);
		}
and:
	        if ( Tcl_SetSystemEncoding(interp, encoding_val) == TCL_ERROR ) {
	 		fprintf(stderr, 
			"Could not set Tcl system encoding to %s\n", encoding_val);
    		}

IOW, the encoding name is an empty string.

-- 
Glynn Clements <glynn@gclements.plus.com>


Tue, Apr 25 2006 19:58:43    Request created by guest (as #4339)  
Subject: v.digit cannot update sqlite table

Platform: GNU/Linux/x86
grass obtained from: Trento Italy site
grass binary for platform: Compiled from Sources
GRASS Version: grass-6.1.cvs_src_snapshot_2006_04_15

While digitizing into a new vector coverage using the sqlite driver, I get the
following error when the Form dialog attempts to update the attribute table:
WARNING: Cannot update table
D0/0: Name = cat
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to koi8-r
DBMI-SQLite driver error:
Error in sqlite3_prepare():
near "where": syntax error

Steps to reproduce:

Step 1. Link Grass to the sqlite driver:

> sqlite3 -version
3.2.8

> db.connect -p

driver:sqlite
database:/home/data/grass/UTM10_NAD83/pugetsound/sqlite3.db
schema:(null)
group:(null)

Step 2. Start digitizing a new (empty) vector:

> v.digit -n map=newvect
Map does not exist. Add flag -n to create a new map.
New empty map created.

Step 3. Create an attribute table for this vector from the Open Settings > Table
> Create Table dialog

(Should see dialog that Table was successfuly created)

Step 4. Digitize a new line

A dialog called "Form" (bad name) opens with an attribute tab:

New record was created

cat: 1

Assume data encoding as:

utf-8
ascii
iso8859-1
koi8-r

I selected utf-8 (I'm on Ubuntu Linux) and clicked submit. That's when I recieved
the error:

WARNING: Cannot update table
D0/0: Name = cat
D0/0: Name = _grass_internal_database_encoding
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to
D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to
DBMI-SQLite driver error:
Error in sqlite3_prepare():
near "where": syntax error



Tue, May 9 2006 14:40:04    Mail sent by msieczka  
Bug still applies.

Maciek
Tue, Jun 20 2006 19:01:20    Request created by guest (as #4645)  
Subject: v.digit under cygwin

Platform: WindowsNT/2000/XP
grass obtained from: Trento Italy site
grass binary for platform: Downloaded precompiled Binaries
GRASS Version: GRASS 6.1.cvs (2006)

Context :
I have a vector maps (formed of points) and whant to select points in two disjoined
areas.

Actions :
1. I use v.digit then:
1.a. with the option "no category":
- I define the first boundary;
- I attach its centroid;
- I define the second boundary.
1.b. with the option "next not used", I attach the centroid to the second boundary
and fill the form.
2. I use v.select for selecting the points into the digitized areas

Problems :
1. under windows and cygwin, only the points into the boundary which centroid
is linked to the database table are selected.
2. to make it work, I have to choose the option "next not used" for the two centroids,
but the "Form" window freezes during several minutes with the message "Could
not set Tcl system encoding to" (utf-8 encoding).

(remark : it works very well under linux (fedora core 4))

Please do you have any idea ?

Thanks in advance

Emmanuel

Tue, Jun 20 2006 21:32:28    Mail sent by werchowyna@epf.pl (as #4645)  
Return-Path <werchowyna@epf.pl>
Delivered-To grass-bugs@lists.intevation.de
Date Tue, 20 Jun 2006 21:32:16 +0200
From Maciek Sieczka <werchowyna@epf.pl>
To Request Tracker <grass-bugs@intevation.de>
Cc grass-dev@grass.itc.it
Subject Re: [GRASS-dev] [bug #4645] (grass) v.digit under cygwin
Message-Id <20060620213216.6352b453.werchowyna@epf.pl>
In-Reply-To <20060620170120.17C401006A8@lists.intevation.de>
References <20060620170120.17C401006A8@lists.intevation.de>
X-Mailer Sylpheed version 2.1.1 (GTK+ 2.8.6; i486-pc-linux-gnu)
Mime-Version 1.0
Content-Type text/plain; charset=US-ASCII
Content-Transfer-Encoding 7bit
X-Spam-Status No, hits=-4.9 tagged_above=-999.0 required=3.0 tests=BAYES_00
X-Spam-Level
On Tue, 20 Jun 2006 19:01:20 +0200 (CEST)
Request Tracker <grass-bugs@intevation.de> wrote:

> this bug's URL: http://intevation.de/rt/webrt?serial_num=4645

> 1. under windows and cygwin, only the points into the boundary which
> centroid is linked to the database table are selected. 2. to make it
> work, I have to choose the option "next not used" for the two
> centroids, but the "Form" window freezes during several minutes with
> the message "Could not set Tcl system encoding to" (utf-8 encoding).

This looks like this bug report:
http://intevation.de/rt/webrt?serial_num=4110

Merging it.

> (remark : it works very well under linux (fedora core 4))

Actually the error applies to Linux to, only apparently on Cygwin it
has more severe consequences.

Maciek

------------------------------------------------------------------------
CIEP?E KRAJE - CIEP?E MORZA. Szukasz atrakcyjnego wypoczynku w przyst?pnej cenie,
zapoznaj si? z nasz? ofert?.
ZAPRASZAMY

www.skarpatravel.pl


Tue, Jun 20 2006 21:34:28    Request 4645 merged into 4110 by msieczka (as #4645)  
Tue, Jun 20 2006 21:59:19    Mail sent by guest  
thank you very much.
concerning my first problem ("only the points into the boundary which centroid
is linked to the database table are selected"), is there any solution to make
grass under cygwin behave as under linux (i.e. defining only one database link)
?
thank you very much in advance
Emmanuel
Wed, Jun 21 2006 04:51:56    Mail sent by hamish_nospam@yahoo.com (as #4645)  
Return-Path <hamish_nospam@yahoo.com>
Delivered-To grass-bugs@lists.intevation.de
Date Wed, 21 Jun 2006 14:51:38 +1200
From Hamish <hamish_nospam@yahoo.com>
To Maciek Sieczka <werchowyna@epf.pl>
Cc grass-bugs@intevation.de, grass-dev@grass.itc.it
Subject Re: [GRASS-dev] [bug #4645] (grass) v.digit under cygwin
Message-Id <20060621145138.442617d1.hamish_nospam@yahoo.com>
In-Reply-To <20060620213216.6352b453.werchowyna@epf.pl>
References <20060620170120.17C401006A8@lists.intevation.de> <20060620213216.6352b453.werchowyna@epf.pl>
X-Mailer Sylpheed version 1.0.4 (GTK+ 1.2.10; i386-pc-linux-gnu)
X-Face M<EoB)"*Z~u!,vFhXmw}R_KbdBta*P_=T|rbBL'e1/CQ9;/1g\BU3&!=y8ria$2Uk!HT&BB 8i?|X_+7~1jsy}F~g$2va%3fV`*=L(*cem[@3\yg,G,@rg6/QMJ
Mime-Version 1.0
Content-Type text/plain; charset=US-ASCII
Content-Transfer-Encoding 7bit
X-Spam-Status No, hits=-4.0 tagged_above=-999.0 required=3.0 tests=BAYES_00, FORGED_YAHOO_RCVD
X-Spam-Level
On Tue, 20 Jun 2006 21:32:16 +0200
Maciek Sieczka <werchowyna@epf.pl> wrote:

> On Tue, 20 Jun 2006 19:01:20 +0200 (CEST)
> Request Tracker <grass-bugs@intevation.de> wrote:
> 
> > this bug's URL: http://intevation.de/rt/webrt?serial_num=4645
> 
> > 1. under windows and cygwin, only the points into the boundary which
> > centroid is linked to the database table are selected. 2. to make it
> > work, I have to choose the option "next not used" for the two
> > centroids, but the "Form" window freezes during several minutes with
> > the message "Could not set Tcl system encoding to" (utf-8 encoding).
> 
> This looks like this bug report:
> http://intevation.de/rt/webrt?serial_num=4110
> 
> Merging it.
> 
> > (remark : it works very well under linux (fedora core 4))
> 
> Actually the error applies to Linux to, only apparently on Cygwin it
> has more severe consequences.


symptom of the 100% cpu usage problems which still linger in v.digit?
(but less of a problem than before)


Hamish


Wed, Jun 21 2006 10:00:06    Mail sent by guest  
I found a solution to my problem ("only the points into the boundary which
centroid is linked to the database table are selected").

As v.select (under cygwin) only consider entities that are associated with a
"category" (see help page of v.select), I use the fonction "v.category".

So, in brief:
1. with v.digit and the option "no category", I define as many boundaries as
I
want and I attach one centroid to each of them. I only select the option "next
not used" for the last centroid I define. This works well without any frezzing
... I save and exit;
2. I use v.category to add categories to all the centroids (v.category in=...
out=... option=add type=centroid)
3. I use v.select ...

Thank you for your help
Cheers

Emmanuel

Wed, Jun 21 2006 10:21:47    Mail sent by guest  
I found a solution to my problem ("only the points into the boundary which
centroid is linked to the database table are selected").

As v.select (under cygwin) only consider entities that are associated with a
"category" (see help page of v.select), I use the fonction "v.category".

So, in brief:
1. with v.digit and the option "no category", I define as many boundaries as
I
want and I attach one centroid to each of them. I only select the option "next
not used" for the last centroid I define. This works well without any frezzing
... I save and exit;
2. I use v.category to add categories to all the centroids (v.category in=...
out=... option=add type=centroid)
3. I use v.select ...

Thank you for your help
Cheers

Emmanuel

Wed, Jun 21 2006 10:21:51    Mail sent by guest  
I found a solution to my problem ("only the points into the boundary which
centroid is linked to the database table are selected").

As v.select (under cygwin) only consider entities that are associated with a
"category" (see help page of v.select), I use the fonction "v.category".

So, in brief:
1. with v.digit and the option "no category", I define as many boundaries as
I
want and I attach one centroid to each of them. I only select the option "next
not used" for the last centroid I define. This works well without any frezzing
... I save and exit;
2. I use v.category to add categories to all the centroids (v.category in=...
out=... option=add type=centroid)
3. I use v.select ...

Thank you for your help
Cheers

Emmanuel

Fri, Jun 23 2006 20:43:58    Mail sent by msieczka  
Nice you found a workaround and can continue your work! Anyway, the freeze
still happens if you don't use the workaround, right?

Maciek
Sat, Jul 22 2006 15:00:13    Mail sent by mneteler (as #4339)  
Hi,

is this still an issue with current GRASS 6.1? The
tcl part was improved, but I don't know if encoding
was touched as well.

Markus
Sun, Jul 23 2006 22:27:30    User changed to roux@lmtg.obs-mip.fr,tutey@o2.pl by msieczka  
Wed, Jul 26 2006 14:36:19    User changed to tutey@o2.pl by msieczka (as #3131)  
Fri, Dec 1 2006 10:35:17    Taken by mneteler (as #3131)  
Fri, Dec 1 2006 10:35:17    Mail sent by mneteler (as #3131)  
Maciek,

I guess I found this one:

lib/form/generate.c
    static char *encoding_list[] = {
	"utf-8",
	"ascii",
	"iso8859-1",
	"koi8-r",
	NULL
    };

It's missing there, isn't it.
The message itself is generated in
lib/form/form.c
by Tcl_SetSystemEncoding().

Can you add it to lib/form/generate.c, compile and try again?

Markus
Sat, Dec 2 2006 18:09:15    Request 4339 merged into 4110 by mneteler (as #4339)  
Sat, Dec 2 2006 18:09:53    Request 3131 merged into 4110 by mneteler (as #3131)  
Sat, Dec 2 2006 18:17:08    Mail sent by mneteler  
Hi,

I just merged some apparently related reports. Please revisit the
bottom of
https://intevation.de/rt/webrt?serial_num=4110

I found available encoding in:
/usr/lib/tcl8.4/encoding/

Maybe we need to list them all? Maciek, can you try my suggestion?

Markus
Sun, Dec 3 2006 22:49:07    Mail sent by msieczka  
mneteler wrote (Sat, Dec 2 2006 18:17:08):

> I just merged some apparently related reports.

These are 2 different issues.

One thing is that ISO-8859-2 is not supported, which was the bug 4110. Adding
iso-8859-2 to lib/form/generate.c as you suggested made v.digit listing it as
one of available encodings when editing data attributes (cool).

However, still Polish accented letters in v.digit are displayed as garbage
when entered, and d.what.vect in form mode prints them as garbage as well.
This is propably to the other issue #3131, that v.digit keeps on printing the
error to the terminal:

D0/0: Name = _grass_internal_database_encoding
Could not set Tcl system encoding to

no matter what encoding is used. According to Glynn's analysis of
lib/form/form.c "the encoding name is an empty string" - see his note from 
Wed, Feb 22 2006 14:08:26.

Maciek
Mon, Dec 4 2006 15:10:03    Mail sent by mneteler  
I have added some encodings which are listed now (but cannot
be applied apart from UTF-8).

I found also that
../../include/form.h:#define F_ENCODING "_grass_internal_database_encoding"
which seems to be the default.

More reasonable G_debug messages are in CVS now (debug level 3).

Question: theurrent GRASS_DB_ENCODING is (null) - should predefine
GRASS_DB_ENCODING in lib/init/init.sh (if so, to which encoding?)

I don't quite understand how this encoding thing works.

Markus
Tue, Feb 6 2007 13:03:26    Status changed to resolved by mneteler  
Tue, Feb 6 2007 13:03:26    Mail sent by mneteler  
Hi,

https://intevation.de/rt/webrt?serial_num=4110

with much help from Roberto Flor we have fixed the encoding
problem. Submitted to CVS.

Cheers
Markus
Tue, Feb 6 2007 13:09:18    User changed to david.p.finlayson@gmail.com,neteler@itc.it,roux@lmtg.obs-mip.fr,tutey@o2.pl by mneteler  
Wed, Feb 7 2007 16:48:15    Status changed to open by msieczka  
Wed, Feb 7 2007 16:48:15    Mail sent by msieczka  
mneteler wrote (Tue, Feb 6 2007 13:03:26):

> Hi,
> 
> https://intevation.de/rt/webrt?serial_num=4110
> 
> with much help from Roberto Flor we have fixed the encoding
> problem. Submitted to CVS.

Unfortunately it's still not fixed. v.digit still complains:

WARNING: Could not set Tcl system encoding to 'iso8859-2' (unknown encoding
"iso8859-2")

and during entering or editing the ISO-8859-2 characters they are rendered
wrong as some "hieroglyphs". Same for d.what.vect and d.what.vect -e. However,
d.what.vect -x prints the ISO-8859-2 characters OK on the terminal.

Maciek
Fri, Feb 9 2007 09:10:29    Request created by neteler@itc.it (as #5488)  
Return-Path <neteler@itc.it>
Delivered-To grass-bugs@lists.intevation.de
X-AuditID d94d5003-aed5bbb0000004c3-db-45cc2c73af88
Message-ID <45CC2C73.5030206@itc.it>
Date Fri, 09 Feb 2007 09:10:27 +0100
From Markus Neteler <neteler@itc.it>
User-Agent Thunderbird 1.5.0.9 (X11/20061206)
MIME-Version 1.0
To Glynn Clements <glynn@gclements.plus.com>
Cc grass developers list <grass-dev@grass.itc.it>, Request Tracker <grass-bugs@intevation.de>
Subject Re: [GRASS-dev] v.digit: Could not set Tcl system encoding fixed (bug #4110)
References <45C9995E.7010700@itc.it> <45C9F585.30406@o2.pl> <45CAD4F8.5050400@itc.it> <17867.3200.384675.355130@cerise.gclements.plus.com> <45CB36F6.1070906@itc.it> <17867.42607.267308.193576@cerise.gclements.plus.com>
In-Reply-To <17867.42607.267308.193576@cerise.gclements.plus.com>
Content-Type text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding 7bit
X-OriginalArrivalTime 09 Feb 2007 08:10:27.0211 (UTC) FILETIME=[C1C84DB0:01C74C21]
X-Brightmail-Tracker AAAAAA==
X-Virus-Scanned by amavisd-new at intevation.de
X-Spam-Status No, hits=-5 tagged_above=-999 required=3 tests=[BAYES_00=-5]
X-Spam-Level
Glynn Clements wrote on 02/08/2007 11:38 PM:
> Markus Neteler wrote:
>
>   
>>>>>> Roberto Flor and me have fixed the bug
>>>>>> "v.digit: Could not set Tcl system encoding"
>>>>>> http://intevation.de/rt/webrt?serial_num=4110
>>>>>>
>>>>>> Please test. I'll backport it to 6.2.x then.
>>>>>>
>>>>>> Note: For mysterious Tcl (?) reasons I cannot set iso8859-1 and iso8859-2
>>>>>> but utf-8, iso8859-15, iso2210-jp, koi8-r, euc-jp etc work.
>>>>>> Maybe my installation...
>>>>>>     
>>>>>>         
>>>>>>             
>>>>> I confirm. Same happens on my instalation. tcl/tk 8.4.12, Ubuntu Dapper
>>>>> 32bit.
>>>>>       
>>>>>           
>>>> So we are happy since most of the encodings now work.
>>>>     
>>>>         
>>> Not having ISO-8859-1 working is a pretty major issue.
>>>   
>>>       
>> I am still not sure if it is a local problem or a general one or depends 
>> on the tcltk version.
>>     
>>> Can you provide more details, preferably including a recipe to
>>> reproduce the issue using either spearfish or a new location?
>>>   
>>>       
>> Sure (eg Spearfish):
>> d.mon x0
>> v.digit -n newmap
>> # now go to "Settings" icon, "Table" tab, "Add column", add a column of 
>> your choice, "Create table"
>> # close the "Settings" window
>> # digitize a line
>> # enter a value into the attribute popup form
>> # "submit button" (default is utf-8 which seems to work)
>> # change the encoding, click "submit button" again and voila' the error 
>> appear in the terminal
>>     
>
> Okay.
>
> I'm pretty sure that the problem is that lib/form/form.c doesn't call
> Tcl_Main(), so the library path doesn't get set, so Tcl can't find its
> .enc files.
>
> Realistically, if you are trying to use Tcl/Tk and your program
> *doesn't* look very much like tkAppInit.c, all bets are off.
>
> Rather than trying to process the data from the form library in C,
> form.c should just register the commands then do the rest in Tcl.
>   
Since I know nothing about tcl, I'll leave that to the experts. AFAIK 
form.c was
originally written by Radim.

Markus


Fri, Feb 9 2007 09:16:35    Request 5488 merged into 4110 by mneteler (as #5488)  
Sat, Feb 10 2007 15:59:12    Request created by glynn@gclements.plus.com (as #5490)  
Return-Path <glynn@gclements.plus.com>
Delivered-To grass-bugs@lists.intevation.de
From Glynn Clements <glynn@gclements.plus.com>
MIME-Version 1.0
Content-Type multipart/mixed; boundary="8XkdMeO807"
Content-Transfer-Encoding 7bit
Message-ID <17869.56760.385929.175604@cerise.gclements.plus.com>
Date Sat, 10 Feb 2007 14:59:04 +0000
To Markus Neteler <neteler@itc.it>
Cc Request Tracker <grass-bugs@intevation.de>, grass developers list <grass-dev@grass.itc.it>
Subject Re: [GRASS-dev] v.digit: Could not set Tcl system encoding fixed (bug #4110)
In-Reply-To <45CC2C73.5030206@itc.it>
References <45C9995E.7010700@itc.it> <45C9F585.30406@o2.pl> <45CAD4F8.5050400@itc.it> <17867.3200.384675.355130@cerise.gclements.plus.com> <45CB36F6.1070906@itc.it> <17867.42607.267308.193576@cerise.gclements.plus.com> <45CC2C73.5030206@itc.it>
X-Mailer VM 7.07 under 21.4 (patch 20) "Double Solitaire" XEmacs Lucid
X-Virus-Scanned by amavisd-new at intevation.de
X-Spam-Status No, hits=-5 tagged_above=-999 required=3 tests=[BAYES_00=-5]
X-Spam-Level
--8XkdMeO807
Content-Type: text/plain; charset=us-ascii
Content-Description: message body and .signature
Content-Transfer-Encoding: 7bit


Markus Neteler wrote:

> > I'm pretty sure that the problem is that lib/form/form.c doesn't call
> > Tcl_Main(), so the library path doesn't get set, so Tcl can't find its
> > .enc files.
> >
> > Realistically, if you are trying to use Tcl/Tk and your program
> > *doesn't* look very much like tkAppInit.c, all bets are off.
> >
> > Rather than trying to process the data from the form library in C,
> > form.c should just register the commands then do the rest in Tcl.
> 
> Since I know nothing about tcl, I'll leave that to the experts. AFAIK 
> form.c was
> originally written by Radim.

The attached patch appears to work, insofar as v.digit appears to work
as before, minus the encoding errors. OTOH, I know next to nothing
about v.digit (or the vector stuff in general), so it would probably
be a good idea for it to be tested by someone who is familiar with
v.digit.

Essentially, the form utility (lib/form/form.c) has been rewritten as
a typical "extended wish", with the communication with the form
library being performed in Tcl.

-- 
Glynn Clements <glynn@gclements.plus.com>


--8XkdMeO807
Content-Type: text/plain
Content-Description: lib/form patch
Content-Disposition: inline;
	filename="form.diff"
Content-Transfer-Encoding: base64

SW5kZXg6IGxpYi9mb3JtL2Zvcm0uYw0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0KUkNTIGZpbGU6IC9ncmFz
c3JlcG9zaXRvcnkvZ3Jhc3M2L2xpYi9mb3JtL2Zvcm0uYyx2DQpyZXRyaWV2aW5nIHJldmlz
aW9uIDEuMTcNCmRpZmYgLXUgLXIxLjE3IGZvcm0uYw0KLS0tIGxpYi9mb3JtL2Zvcm0uYwk2
IEZlYiAyMDA3IDExOjE1OjAzIC0wMDAwCTEuMTcNCisrKyBsaWIvZm9ybS9mb3JtLmMJMTAg
RmViIDIwMDcgMTQ6NTE6NTIgLTAwMDANCkBAIC0xMiwxMiArMTIsNiBAQA0KICNpbmNsdWRl
IDxncmFzcy9kYm1pLmg+DQogI2luY2x1ZGUgPGdyYXNzL2Zvcm0uaD4NCiANCi0jaWZkZWYg
X19NSU5HVzMyX18NCi0jaW5jbHVkZSA8d2luc29jay5oPg0KLSNkZWZpbmUgICAgICAgIEZf
U0VURkwgICAgICAgICA0CS8qIFNldCBmaWxlIHN0YXR1cyBmbGFncy4gICovDQotI2RlZmlu
ZSAgICAgICAgT19OT05CTE9DSyAgICAgIDB4MDAwNAkvKiBOb24tYmxvY2tpbmcgSS9PLiAg
Ki8NCi0jZW5kaWYNCi0NCiAvKiBTdHJ1Y3R1cmUgdG8gc3RvcmUgY29sdW1uIG5hbWVzIGFu
ZCB2YWx1ZXMgKi8NCiB0eXBlZGVmIHN0cnVjdA0KIHsNCkBAIC0yNiwyMSArMjAsMTEgQEAN
CiAgICAgY2hhciAqdmFsdWU7DQogfSBDT0xVTU47DQogDQotY2hhciAqRHJ2bmFtZSwgKkRi
bmFtZSwgKlRibG5hbWUsICpLZXk7DQotDQotQ09MVU1OICpDb2x1bW5zID0gTlVMTDsNCi1p
bnQgYWxsb2NhdGVkUm93cyA9IDA7CQkJLyogYWxsb2NhdGVkIHNwYWNlICovDQotaW50IG5S
b3dzID0gMDsNCitzdGF0aWMgY2hhciAqRHJ2bmFtZSwgKkRibmFtZSwgKlRibG5hbWUsICpL
ZXk7DQogDQotaW50IGZvcm1fb3BlbiA9IDA7DQotDQotLyogQ2xvc2UgZm9ybSAqLw0KLWlu
dCBjbG9zZV9mb3JtKENsaWVudERhdGEgY2RhdGEsIFRjbF9JbnRlcnAgKiBpbnRlcnAsIGlu
dCBhcmdjLCBjaGFyICphcmd2W10pDQotew0KLSAgICBHX2RlYnVnKDMsICJjbG9zZV9mb3Jt
KCkiKTsNCi0gICAgZm9ybV9vcGVuID0gMDsNCi0gICAgcmV0dXJuIFRDTF9PSzsNCi19DQor
c3RhdGljIENPTFVNTiAqQ29sdW1ucyA9IE5VTEw7DQorc3RhdGljIGludCBhbGxvY2F0ZWRS
b3dzID0gMDsJCQkvKiBhbGxvY2F0ZWQgc3BhY2UgKi8NCitzdGF0aWMgaW50IG5Sb3dzID0g
MDsNCiANCiAvKiBTdGFydCBuZXcgc3FsIHVwZGF0ZSAqLw0KIGludCByZXNldF92YWx1ZXMo
Q2xpZW50RGF0YSBjZGF0YSwgVGNsX0ludGVycCAqIGludGVycCwgaW50IGFyZ2MsIGNoYXIg
KmFyZ3ZbXSkNCkBAIC0yNjcsMTUxICsyNTEsNDkgQEANCiAvKiANCiAgKiAgRm9ybSANCiAg
Ki8NCi1pbnQgbWFpbihpbnQgYXJnYywgY2hhciAqYXJndltdKQ0KK2ludCBUY2xfQXBwSW5p
dChUY2xfSW50ZXJwICppbnRlcnApDQogew0KLSAgICBpbnQgbGVuZ3RoOw0KLSAgICBpbnQg
cmV0Ow0KLSAgICBjaGFyIGJ1Zls1MDAwXTsNCi0gICAgY2hhciAqY2hpbGRfaHRtbCwgKmNo
aWxkX3RpdGxlOw0KLSAgICBzdGF0aWMgRklMRSAqY2hpbGRfc2VuZCwgKmNoaWxkX3JlY3Y7
DQotICAgIHN0YXRpYyBUY2xfSW50ZXJwICppbnRlcnA7DQotICAgIHN0YXRpYyBpbnQgZnJt
aWQgPSAwOw0KLSAgICBjaGFyICplbmNvZGluZ192YWw7DQotDQotICAgIEdfZ2lzaW5pdCgi
Zm9ybSIpOw0KLQ0KLSAgICBHX2RlYnVnKDIsICJGb3JtOiBtYWluKCkiKTsNCi0NCi0gICAg
c2V0bG9jYWxlKExDX0NUWVBFLCAiIik7DQotDQotICAgIGNoaWxkX3JlY3YgPSBzdGRpbjsN
Ci0gICAgY2hpbGRfc2VuZCA9IHN0ZG91dDsNCi0NCi0gICAgd2hpbGUgKDEpIHsNCi0JZmRf
c2V0IHdhaXRzZXQ7DQotCXN0cnVjdCB0aW1ldmFsIHR2Ow0KLQ0KLQl0di50dl9zZWMgPSAw
Ow0KLQl0di50dl91c2VjID0gMjAwOw0KLQ0KLQlGRF9aRVJPKCZ3YWl0c2V0KTsNCi0JRkRf
U0VUKGZpbGVubyhzdGRpbiksICZ3YWl0c2V0KTsNCi0NCi0JaWYgKHNlbGVjdChGRF9TRVRT
SVpFLCAmd2FpdHNldCwgTlVMTCwgTlVMTCwgJnR2KSA8IDApIHsNCi0JICAgIHBlcnJvcigi
Zm9ybTogc2VsZWN0Iik7DQotCX0NCi0NCi0JcmV0ID0gcmVhZChmaWxlbm8oc3RkaW4pLCAm
KGJ1ZlswXSksIDEpOw0KLSNpZm5kZWYgX19NSU5HVzMyX18NCi0JZmNudGwoZmlsZW5vKGNo
aWxkX3JlY3YpLCBGX1NFVEZMLCBPX05PTkJMT0NLKTsJLyogRG9uJ3Qgd2FpdCBpZiBwaXBl
IGlzIGVtcHR5ICovDQotI2VuZGlmDQotCWlmIChyZXQgPT0gMCkNCi0JICAgIGJyZWFrOwkJ
LyogUGlwZSB3YXMgY2xvc2VkIGJ5IHBhcmVudCAtPiBxdWl0ICovDQotCWlmIChyZXQgPT0g
MSkgew0KLQkgICAgR19kZWJ1ZygzLCAiRm9ybTogcmVjZWl2ZWQgPSAnJWMnIiwgYnVmWzBd
KTsNCi0JICAgIGlmIChidWZbMF0gPT0gJ08nKSB7DQotCQlpZiAoIWZvcm1fb3Blbikgew0K
LQkJICAgIEdfZGVidWcoMywgIkZvcm0gaXMgbm90IG9wZW5lZCIpOw0KLQkJICAgIC8qIE9w
ZW4gdGhlIHdpbmRvdyBhbmQgZGlzcGxheSB0aGUgZm9ybSAqLw0KLQkJICAgIGludGVycCA9
IFRjbF9DcmVhdGVJbnRlcnAoKTsNCi0JCSAgICBpZiAoVGNsX0luaXQoaW50ZXJwKSA9PSBU
Q0xfRVJST1IpDQotCQkJR19mYXRhbF9lcnJvcigiVGNsX0luaXQgZmFpbGVkOiAlc1xuIiwg
aW50ZXJwLT5yZXN1bHQpOw0KLQkJICAgIGlmIChUa19Jbml0KGludGVycCkgPT0gVENMX0VS
Uk9SKQ0KLQkJCUdfZmF0YWxfZXJyb3IoIlRrX0luaXQgZmFpbGVkOiAlc1xuIiwgaW50ZXJw
LT5yZXN1bHQpOw0KLQ0KLQkJICAgIFRjbF9DcmVhdGVDb21tYW5kKGludGVycCwgInN1Ym1p
dCIsIChUY2xfQ21kUHJvYyAqKSBzdWJtaXQsDQotCQkJCSAgICAgIChDbGllbnREYXRhKSBO
VUxMLA0KLQkJCQkgICAgICAoVGNsX0NtZERlbGV0ZVByb2MgKikgTlVMTCk7DQotCQkgICAg
VGNsX0NyZWF0ZUNvbW1hbmQoaW50ZXJwLCAic2V0X3ZhbHVlIiwNCi0JCQkJICAgICAgKFRj
bF9DbWRQcm9jICopIHNldF92YWx1ZSwNCi0JCQkJICAgICAgKENsaWVudERhdGEpIE5VTEws
DQotCQkJCSAgICAgIChUY2xfQ21kRGVsZXRlUHJvYyAqKSBOVUxMKTsNCi0JCSAgICBUY2xf
Q3JlYXRlQ29tbWFuZChpbnRlcnAsICJyZXNldF92YWx1ZXMiLA0KLQkJCQkgICAgICAoVGNs
X0NtZFByb2MgKikgcmVzZXRfdmFsdWVzLA0KLQkJCQkgICAgICAoQ2xpZW50RGF0YSkgTlVM
TCwNCi0JCQkJICAgICAgKFRjbF9DbWREZWxldGVQcm9jICopIE5VTEwpOw0KLQkJICAgIFRj
bF9DcmVhdGVDb21tYW5kKGludGVycCwgImNsb3NlX2Zvcm0iLA0KLQkJCQkgICAgICAoVGNs
X0NtZFByb2MgKikgY2xvc2VfZm9ybSwNCi0JCQkJICAgICAgKENsaWVudERhdGEpIE5VTEws
DQotCQkJCSAgICAgIChUY2xfQ21kRGVsZXRlUHJvYyAqKSBOVUxMKTsNCi0NCi0JCSAgICBz
cHJpbnRmKGJ1ZiwgIiVzL2V0Yy9mb3JtL2Zvcm0udGNsIiwgR19naXNiYXNlKCkpOw0KLQkJ
ICAgIHJldCA9IFRjbF9FdmFsRmlsZShpbnRlcnAsIGJ1Zik7DQotCQkgICAgaWYgKHJldCA9
PSBUQ0xfRVJST1IpIHsNCi0JCQlpZiAoaW50ZXJwLT5yZXN1bHQgIT0gTlVMTCkNCi0JCQkg
ICAgR19mYXRhbF9lcnJvcigiQ2Fubm90IG9wZW4gZm9ybTogJXNcbiIsDQotCQkJCQkgIGlu
dGVycC0+cmVzdWx0KTsNCi0JCQllbHNlDQotCQkJICAgIEdfZmF0YWxfZXJyb3IoIkNhbm5v
dCBvcGVuIGZvcm1cbiIpOw0KLQkJICAgIH0NCisJaWYgKFRjbF9Jbml0KGludGVycCkgPT0g
VENMX0VSUk9SKQ0KKwkJcmV0dXJuIFRDTF9FUlJPUjsNCiANCisJaWYgKFRrX0luaXQoaW50
ZXJwKSA9PSBUQ0xfRVJST1IpDQorCQlyZXR1cm4gVENMX0VSUk9SOw0KIA0KLQkJICAgIGZv
cm1fb3BlbiA9IDE7DQotCQl9DQotCQlHX2RlYnVnKDIsICJPcGVuIGZvcm0gJWQiLCBmcm1p
ZCk7DQotCQkvKiBSZWFkIHRpdGxlICovDQotCQlmZ2V0cyhidWYsIDEwMDAsIGNoaWxkX3Jl
Y3YpOw0KLQkJbGVuZ3RoID0gYXRvaShidWYpOwkvKiBsZW5ndGggb2YgdGhlIHN0cmluZyAq
Lw0KLQkJR19kZWJ1ZygyLCAibGVuZ3RoID0gJWQiLCBsZW5ndGgpOw0KLQkJY2hpbGRfdGl0
bGUgPSAoY2hhciAqKUdfbWFsbG9jKGxlbmd0aCArIDEpOw0KLQkJZnJlYWQoY2hpbGRfdGl0
bGUsIGxlbmd0aCwgMSwgY2hpbGRfcmVjdik7DQotCQljaGlsZF90aXRsZVtsZW5ndGhdID0g
J1wwJzsNCi0NCi0JCS8qIFJlYWQgaHRtbCAqLw0KLQkJZmdldHMoYnVmLCAxMDAwLCBjaGls
ZF9yZWN2KTsNCi0JCWxlbmd0aCA9IGF0b2koYnVmKTsJLyogbGVuZ3RoIG9mIHRoZSBzdHJp
bmcgKi8NCi0JCUdfZGVidWcoMiwgImxlbmd0aCA9ICVkIiwgbGVuZ3RoKTsNCi0JCWNoaWxk
X2h0bWwgPSAoY2hhciAqKUdfbWFsbG9jKGxlbmd0aCArIDEpOw0KLQkJZnJlYWQoY2hpbGRf
aHRtbCwgbGVuZ3RoLCAxLCBjaGlsZF9yZWN2KTsNCi0JCWNoaWxkX2h0bWxbbGVuZ3RoXSA9
ICdcMCc7DQotDQotCQltZW1zZXQoYnVmLCAnXDAnLCBzdHJsZW4oYnVmKSk7DQotDQotCQll
bmNvZGluZ192YWwgPSBHX19nZXRlbnYoIkdSQVNTX0RCX0VOQ09ESU5HIik7DQotCQlUY2xf
RXh0ZXJuYWxUb1V0ZihpbnRlcnAsDQotCQkJCSAgVGNsX0dldEVuY29kaW5nKGludGVycCwg
ZW5jb2RpbmdfdmFsKSwNCi0JCQkJICBjaGlsZF9odG1sLCBzdHJsZW4oY2hpbGRfaHRtbCks
IDAsIE5VTEwsDQotCQkJCSAgYnVmLCBzdHJsZW4oY2hpbGRfaHRtbCkgKiAyLCBOVUxMLCBO
VUxMLA0KLQkJCQkgIE5VTEwpOw0KLQ0KLQkJR19kZWJ1ZygzLCAiQ3VycmVudCBHUkFTU19E
Ql9FTkNPRElORzogJXMiLCBlbmNvZGluZ192YWwpOw0KLQkJaWYgKFRjbF9TZXRTeXN0ZW1F
bmNvZGluZyhpbnRlcnAsIGVuY29kaW5nX3ZhbCkgPT0gVENMX0VSUk9SKSB7DQotCQkgICAg
ZnByaW50ZihzdGRlcnIsDQotCQkJICAgICJDb3VsZCBub3Qgc2V0IFRjbCBzeXN0ZW0gZW5j
b2RpbmcgdG8gJXNcbiIsDQotCQkJICAgIGVuY29kaW5nX3ZhbCk7DQotCQl9DQorCVRjbF9T
dGF0aWNQYWNrYWdlKGludGVycCwgIlRrIiwgVGtfSW5pdCwgVGtfU2FmZUluaXQpOw0KIA0K
LQkJR19kZWJ1ZygyLCAiRm9ybTogaHRtbCA9ICVzIiwgYnVmKTsNCisJLyoNCisJICogQ2Fs
bCBUY2xfQ3JlYXRlQ29tbWFuZCBmb3IgYXBwbGljYXRpb24tc3BlY2lmaWMgY29tbWFuZHMs
IGlmDQorCSAqIHRoZXkgd2VyZW4ndCBhbHJlYWR5IGNyZWF0ZWQgYnkgdGhlIGluaXQgcHJv
Y2VkdXJlcyBjYWxsZWQgYWJvdmUuDQorCSAqLw0KKw0KKwlUY2xfQ3JlYXRlQ29tbWFuZChp
bnRlcnAsICJzdWJtaXQiLCAoVGNsX0NtZFByb2MgKikgc3VibWl0LA0KKwkJCSAgKENsaWVu
dERhdGEpIE5VTEwsDQorCQkJICAoVGNsX0NtZERlbGV0ZVByb2MgKikgTlVMTCk7DQorCVRj
bF9DcmVhdGVDb21tYW5kKGludGVycCwgInNldF92YWx1ZSIsDQorCQkJICAoVGNsX0NtZFBy
b2MgKikgc2V0X3ZhbHVlLA0KKwkJCSAgKENsaWVudERhdGEpIE5VTEwsDQorCQkJICAoVGNs
X0NtZERlbGV0ZVByb2MgKikgTlVMTCk7DQorCVRjbF9DcmVhdGVDb21tYW5kKGludGVycCwg
InJlc2V0X3ZhbHVlcyIsDQorCQkJICAoVGNsX0NtZFByb2MgKikgcmVzZXRfdmFsdWVzLA0K
KwkJCSAgKENsaWVudERhdGEpIE5VTEwsDQorCQkJICAoVGNsX0NtZERlbGV0ZVByb2MgKikg
TlVMTCk7DQorCS8qDQorCSAqIFNwZWNpZnkgYSB1c2VyLXNwZWNpZmljIHN0YXJ0dXAgZmls
ZSB0byBpbnZva2UgaWYgdGhlIGFwcGxpY2F0aW9uDQorCSAqIGlzIHJ1biBpbnRlcmFjdGl2
ZWx5LiAgVHlwaWNhbGx5IHRoZSBzdGFydHVwIGZpbGUgaXMgIn4vLmFwcHJjIg0KKwkgKiB3
aGVyZSAiYXBwIiBpcyB0aGUgbmFtZSBvZiB0aGUgYXBwbGljYXRpb24uICBJZiB0aGlzIGxp
bmUgaXMgZGVsZXRlZA0KKwkgKiB0aGVuIG5vIHVzZXItc3BlY2lmaWMgc3RhcnR1cCBmaWxl
IHdpbGwgYmUgcnVuIHVuZGVyIGFueSBjb25kaXRpb25zLg0KKwkgKi8NCiANCi0JCS8qIElu
c2VydCBuZXcgcGFnZSAqLw0KLQkJVGNsX1NldFZhcihpbnRlcnAsICJodG1sIiwgYnVmLCAw
KTsNCi0JCXNwcmludGYoYnVmLCAiYWRkX2Zvcm0gJWQgXCIlc1wiIiwgZnJtaWQsIGNoaWxk
X3RpdGxlKTsNCi0JCVRjbF9FdmFsKGludGVycCwgYnVmKTsNCi0NCi0JCWZwcmludGYoY2hp
bGRfc2VuZCwgIk8iKTsJLyogT0sgKi8NCi0JCWZmbHVzaChjaGlsZF9zZW5kKTsNCi0JCWZy
bWlkKys7DQotCQlHX2RlYnVnKDIsICJGb3JtIGRpc3BsYXllZFxuIik7DQotCSAgICB9DQot
CSAgICBlbHNlIGlmIChidWZbMF0gPT0gJ0MnKSB7CS8qIGNsZWFyIG9sZCBmb3JtcyAqLw0K
LQkJVGNsX0V2YWwoaW50ZXJwLCAiY2xlYXJfbmIiKTsNCi0JCWZwcmludGYoY2hpbGRfc2Vu
ZCwgIk8iKTsJLyogT0sgKi8NCi0JCWZmbHVzaChjaGlsZF9zZW5kKTsNCi0JICAgIH0NCi0J
ICAgIGVsc2UgaWYgKGJ1ZlswXSA9PSAnRCcpIHsJLyogZG9uZSEgKi8NCi0JCVRjbF9FdmFs
KGludGVycCwgImNsZWFyX25iIik7DQotCQlmcHJpbnRmKGNoaWxkX3NlbmQsICJPIik7CS8q
IE9LICovDQotCQlmZmx1c2goY2hpbGRfc2VuZCk7DQotCQlicmVhazsNCi0JICAgIH0NCi0J
fQ0KLQ0KLQlUY2xfRXZhbChpbnRlcnAsICJ1cGRhdGUiKTsNCi0gICAgfQ0KKwlUY2xfU2V0
VmFyKGludGVycCwgInRjbF9yY0ZpbGVOYW1lIiwgIn4vLmdyYXNzZm9ybXJjIiwgVENMX0dM
T0JBTF9PTkxZKTsNCisJcmV0dXJuIFRDTF9PSzsNCit9DQogDQotICAgIFRjbF9FdmFsKGlu
dGVycCwgImRlc3Ryb3kgLiIpOw0KLSAgICBHX2RlYnVnKDMsICJGb3JtOiBlbmRcbiIpOw0K
LSAgICBleGl0KDApOw0KK2ludCBtYWluKGludCBhcmdjLCBjaGFyICphcmd2W10pDQorew0K
KwlHX2dpc2luaXQoImZvcm0iKTsNCisJR19kZWJ1ZygyLCAiRm9ybTogbWFpbigpIik7DQog
DQotICAgIHJldHVybiAwOw0KKwlUa19NYWluKGFyZ2MsIGFyZ3YsIFRjbF9BcHBJbml0KTsN
CisJcmV0dXJuIDA7DQogfQ0KKw0KSW5kZXg6IGxpYi9mb3JtL2Zvcm0udGNsDQo9PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09DQpSQ1MgZmlsZTogL2dyYXNzcmVwb3NpdG9yeS9ncmFzczYvbGliL2Zvcm0vZm9y
bS50Y2wsdg0KcmV0cmlldmluZyByZXZpc2lvbiAxLjMNCmRpZmYgLXUgLXIxLjMgZm9ybS50
Y2wNCi0tLSBsaWIvZm9ybS9mb3JtLnRjbAkxNyBGZWIgMjAwMyAwNjo0MTowMyAtMDAwMAkx
LjMNCisrKyBsaWIvZm9ybS9mb3JtLnRjbAkxMCBGZWIgMjAwNyAxNDo1MTo1MiAtMDAwMA0K
QEAgLTIsMTcgKzIsOSBAQA0KIHBhY2thZ2UgcmVxdWlyZSAtZXhhY3QgQldpZGdldCAxLjIu
MSANDQogI3BhY2thZ2UgcmVxdWlyZSBodHRwDQ0KIA0NCi1zZXQgZm9ybXBhdGggJGVudihH
SVNCQVNFKS9ldGMvZm9ybS8gDQ0KK3NldCBmb3JtcGF0aCAkZW52KEdJU0JBU0UpL2V0Yy9m
b3JtDQ0KIHNvdXJjZSAkZm9ybXBhdGgvaHRtbF9saWJyYXJ5LnRjbA0NCiANDQotc2V0IHN1
Ym1pdF9yZXN1bHQgIiINDQotc2V0IHN1Ym1pdF9tc2cgIiINDQotc2V0IGh0bWwgIiINDQot
DQ0KLXNldCBuYiBbTm90ZUJvb2sgLm5iXQ0NCi0kbmIgY29uZmlndXJlIC13aWR0aCAzMDAg
LWhlaWdodCA1MDANDQotcGFjayAubmIgLWZpbGwgYm90aCAtZXhwYW5kIHllcw0NCi0NDQog
cHJvYyBjcmVhdGVfc3VibWl0X21zZyB7IGZvcm1pZCAgfSAgew0NCiAgICAgZ2xvYmFsIHN1
Ym1pdF9yZXN1bHQgc3VibWl0X21zZyBmb3JtZg0NCiANDQpAQCAtNzAsNCArNjIsOTcgQEAN
CiAgICAgY3JlYXRlX3N1Ym1pdF9tc2cgJGZvcm1pZCAgIA0NCiB9DQ0KIA0NCitwcm9jIG1h
a2VfZm9ybSB7fSB7DQ0KKwlnbG9iYWwgbmINDQorDQ0KKwlzZXQgbmIgW05vdGVCb29rIC5u
Yl0NDQorCSRuYiBjb25maWd1cmUgLXdpZHRoIDMwMCAtaGVpZ2h0IDUwMA0NCisJcGFjayAu
bmIgLWZpbGwgYm90aCAtZXhwYW5kIHllcw0NCit9DQ0KKw0NCitwcm9jIGNsb3NlX2Zvcm0g
e30gew0NCisJZ2xvYmFsIGZvcm1fb3Blbg0NCisJd20gd2l0aGRyYXcgLg0NCisJc2V0IGZv
cm1fb3BlbiBmYWxzZQ0NCit9DQ0KKw0NCitwcm9jIHByb2Nlc3NfY29tbWFuZCB7fSB7DQ0K
KwlnbG9iYWwgZW52DQ0KKwlnbG9iYWwgY2hpbGRfcmVjdiBjaGlsZF9zZW5kIA0NCisJZ2xv
YmFsIGZvcm1fb3BlbiBlbmNvZGluZ192YWwgZnJtaWQNDQorCWdsb2JhbCBodG1sDQ0KKw0N
CisJaWYge1tlb2YgJGNoaWxkX3JlY3ZdfSB7DQ0KKwkJZXhpdCAwDQ0KKwl9DQ0KKw0NCisJ
c2V0IGNtZCBbcmVhZCAkY2hpbGRfcmVjdiAxXQ0NCisNDQorCXN3aXRjaCAkY21kIHsNDQor
CQlPIHsNDQorCQkJaWYgeyEgJGZvcm1fb3Blbn0gew0NCisJCQkJd20gc3RhdGUgLiBub3Jt
YWwNDQorCQkJCXNldCBmb3JtX29wZW4gdHJ1ZQ0NCisJCQl9DQ0KKwkJCSMgIFJlYWQgdGl0
bGUgDQ0KKwkJCXNldCBsZW5ndGggW2dldHMgJGNoaWxkX3JlY3ZdDQ0KKwkJCXNldCBjaGls
ZF90aXRsZSBbcmVhZCAkY2hpbGRfcmVjdiAkbGVuZ3RoXQ0NCisNDQorCQkJIyAgUmVhZCBo
dG1sIA0NCisJCQlzZXQgbGVuZ3RoIFtnZXRzICRjaGlsZF9yZWN2XQ0NCisJCQlzZXQgY2hp
bGRfaHRtbCBbcmVhZCAkY2hpbGRfcmVjdiAkbGVuZ3RoXQ0NCisNDQorCQkJc2V0IGNoaWxk
X2h0bWwgW2VuY29kaW5nIGNvbnZlcnRmcm9tICRlbmNvZGluZ192YWwgJGNoaWxkX2h0bWxd
DQ0KKw0NCisJCQkjICBJbnNlcnQgbmV3IHBhZ2UgDQ0KKwkJCXNldCBodG1sICRjaGlsZF9o
dG1sDQ0KKwkJCWFkZF9mb3JtICRmcm1pZCAkY2hpbGRfdGl0bGUNDQorDQ0KKwkJCXB1dHMg
LW5vbmV3bGluZSAkY2hpbGRfc2VuZCBPDQ0KKwkJCWZsdXNoICRjaGlsZF9zZW5kDQ0KKwkJ
CWluY3IgZnJtaWQNDQorCQl9DQ0KKwkJQyB7CSMgIGNsZWFyIG9sZCBmb3JtcyANDQorCQkJ
Y2xlYXJfbmINDQorCQkJcHV0cyAtbm9uZXdsaW5lICRjaGlsZF9zZW5kIE8NDQorCQkJZmx1
c2ggJGNoaWxkX3NlbmQNDQorCQl9DQ0KKwkJRCB7CSMgIGRvbmUhIA0NCisJCQljbGVhcl9u
Yg0NCisJCQlwdXRzIC1ub25ld2xpbmUgJGNoaWxkX3NlbmQgTw0NCisJCQlmbHVzaCAkY2hp
bGRfc2VuZA0NCisNDQorCQkJZGVzdHJveSAuDQ0KKwkJCWV4aXQgMA0NCisJCX0NDQorCX0N
DQorfQ0NCisNDQorbWFrZV9mb3JtDQ0KKw0NCit3bSBwcm90b2NvbCAuIFdNX0RFTEVURV9X
SU5ET1cgY2xvc2VfZm9ybQ0NCisNDQogYmluZCAuIDxEZXN0cm95PiB7IGlmIHsgIiVXIiA9
PSAiLiJ9IHsgY2xvc2VfZm9ybSB9IH0NDQorDQ0KK3NldCBzdWJtaXRfcmVzdWx0ICIiDQ0K
K3NldCBzdWJtaXRfbXNnICIiDQ0KK3NldCBodG1sICIiDQ0KKw0NCitzZXQgZnJtaWQgMA0N
CitzZXQgZm9ybV9vcGVuIHRydWUNDQorDQ0KK3NldCBjaGlsZF9yZWN2IHN0ZGluDQ0KK3Nl
dCBjaGlsZF9zZW5kIHN0ZG91dA0NCisNDQorc2V0IGVuY29kaW5nX3ZhbCBbZXhlYyBnLmdp
c2VudiBHUkFTU19EQl9FTkNPRElOR10NDQoraWYgeyRlbmNvZGluZ192YWwgPT0gIiJ9IHsN
DQorCXNldCBlbmNvZGluZ192YWwgW2VuY29kaW5nIHN5c3RlbV0NDQorfQ0NCisNDQoraWYg
e1tjYXRjaCB7ZW5jb2Rpbmcgc3lzdGVtICRlbmNvZGluZ192YWx9XX0gew0NCisJcHV0cyBz
dGRlcnIgIkNvdWxkIG5vdCBzZXQgVGNsIHN5c3RlbSBlbmNvZGluZyB0byAkZW5jb2Rpbmdf
dmFsIg0NCit9DQ0KKw0NCitmY29uZmlndXJlICRjaGlsZF9yZWN2IC1idWZmZXJpbmcgbm9u
ZSAtZW5jb2RpbmcgYmluYXJ5IC10cmFuc2xhdGlvbiBiaW5hcnkNDQorDQ0KK2ZpbGVldmVu
dCAkY2hpbGRfcmVjdiByZWFkYWJsZSBwcm9jZXNzX2NvbW1hbmQNDQpJbmRleDogbGliL2Zv
cm0vb3Blbi5jDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09DQpSQ1MgZmlsZTogL2dyYXNzcmVwb3NpdG9yeS9n
cmFzczYvbGliL2Zvcm0vb3Blbi5jLHYNCnJldHJpZXZpbmcgcmV2aXNpb24gMS4xMw0KZGlm
ZiAtdSAtcjEuMTMgb3Blbi5jDQotLS0gbGliL2Zvcm0vb3Blbi5jCTcgTm92IDIwMDYgMDY6
MDU6MDcgLTAwMDAJMS4xMw0KKysrIGxpYi9mb3JtL29wZW4uYwkxMCBGZWIgMjAwNyAxNDo1
MTo1MiAtMDAwMA0KQEAgLTUxLDkgKzUxLDYgQEANCiAjZW5kaWYgLypVU0VfR19TT0NLUyov
DQogICAgIGludCAgICAgICAgbGVuZ3RoOw0KICAgICAvKiBjaGlsZCAqLw0KLSAgICBjaGFy
ICAgICAgICBidWZbMjAwMF07DQotICAgIA0KLQ0KICAgICANCiAgICAgR19kZWJ1ZyAoIDIs
ICJGX29wZW4oKTogdGl0bGUgPSAlcyIsIHRpdGxlKTsNCiAgICAgDQpAQCAtNzAsNiArNjcs
OCBAQA0KICAgICB9DQogDQogICAgIGlmICggcGlkID09IDAgKSB7IC8qIENoaWxkICovDQor
CWNoYXIgY29tbWFuZFsyMDAwXSwgc2NyaXB0WzIwMDBdOw0KKw0KICAgICAgICAgR19kZWJ1
ZyAoIDIsICJDSElMRCIgKTsNCiANCiAgICAgICAgIC8qIE5vdGU6IElmIHlvdSBhcmUgZm9y
a2luZyBpbiBhIFRrIGJhc2VkIGFwbGxpY2F0aW9uICB5b3UNCkBAIC05NCw5ICs5MywxMCBA
QA0KIA0KIA0KIAkNCi0Jc3ByaW50ZihidWYsIiVzL2V0Yy9mb3JtL2Zvcm0iLCBHX2dpc2Jh
c2UoKSk7DQorCXNwcmludGYoY29tbWFuZCwgIiVzL2V0Yy9mb3JtL2Zvcm0iLCBHX2dpc2Jh
c2UoKSk7DQorCXNwcmludGYoc2NyaXB0LCAgIiVzL2V0Yy9mb3JtL2Zvcm0udGNsIiwgR19n
aXNiYXNlKCkpOw0KIA0KLQlleGVjbCAoIi9iaW4vc2giLCAic2giLCAiLWMiLCBidWYsIE5V
TEwpOw0KKwlleGVjbChjb21tYW5kLCAiZm9ybSIsICItZiIsIHNjcmlwdCwgTlVMTCk7DQog
CQ0KIAlHX2RlYnVnKDIsICJDSElMRCBFTkRcbiIpOw0KIAlleGl0ICgwKTs=

--8XkdMeO807--


Sat, Feb 10 2007 18:02:44    Request 5490 merged into 4110 by mneteler (as #5490)  
Sat, Feb 10 2007 18:03:19    Comments added by mneteler  
RT tends to behave strange. Time to move this bug
to the new bugtracker.

Markus
Thu, Feb 15 2007 22:00:21    Request created by glynn@gclements.plus.com (as #5496)  
Return-Path <glynn@gclements.plus.com>
Delivered-To grass-bugs@lists.intevation.de
From Glynn Clements <glynn@gclements.plus.com>
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Transfer-Encoding 7bit
Message-ID <17876.51681.722004.212265@cerise.gclements.plus.com>
Date Thu, 15 Feb 2007 21:00:17 +0000
To Markus Neteler <neteler@itc.it>, Request Tracker <grass-bugs@intevation.de>, grass developers list <grass-dev@grass.itc.it>
Subject Re: [GRASS-dev] v.digit: Could not set Tcl system encoding fixed (bug #4110)
In-Reply-To <17869.56760.385929.175604@cerise.gclements.plus.com>
References <45C9995E.7010700@itc.it> <45C9F585.30406@o2.pl> <45CAD4F8.5050400@itc.it> <17867.3200.384675.355130@cerise.gclements.plus.com> <45CB36F6.1070906@itc.it> <17867.42607.267308.193576@cerise.gclements.plus.com> <45CC2C73.5030206@itc.it> <17869.56760.385929.175604@cerise.gclements.plus.com>
X-Mailer VM 7.07 under 21.4 (patch 20) "Double Solitaire" XEmacs Lucid
X-Virus-Scanned by amavisd-new at intevation.de
X-Spam-Status No, score=-4.665 tagged_above=-999 required=3 tests=[BAYES_00=-5, DNS_FROM_RFC_ABUSE=0.2, FORGED_RCVD_HELO=0.135]
X-Spam-Score -4.665
X-Spam-Level
Glynn Clements wrote:

> > > I'm pretty sure that the problem is that lib/form/form.c doesn't call
> > > Tcl_Main(), so the library path doesn't get set, so Tcl can't find its
> > > .enc files.
> > >
> > > Realistically, if you are trying to use Tcl/Tk and your program
> > > *doesn't* look very much like tkAppInit.c, all bets are off.
> > >
> > > Rather than trying to process the data from the form library in C,
> > > form.c should just register the commands then do the rest in Tcl.
> > 
> > Since I know nothing about tcl, I'll leave that to the experts. AFAIK 
> > form.c was
> > originally written by Radim.
> 
> The attached patch appears to work, insofar as v.digit appears to work
> as before, minus the encoding errors. OTOH, I know next to nothing
> about v.digit (or the vector stuff in general), so it would probably
> be a good idea for it to be tested by someone who is familiar with
> v.digit.
> 
> Essentially, the form utility (lib/form/form.c) has been rewritten as
> a typical "extended wish", with the communication with the form
> library being performed in Tcl.

As I haven't had any response either way on this, I've committed the
changes.

-- 
Glynn Clements <glynn@gclements.plus.com>


Sat, Feb 17 2007 13:35:55    Area changed to grass6 by msieczka (as #5496)  
Sat, Feb 17 2007 13:36:22    Request 5496 merged into 4110 by msieczka (as #5496)  
Sat, Feb 17 2007 16:01:31    Status changed to resolved by msieczka  
Sat, Feb 17 2007 16:01:31    Mail sent by msieczka  
> Glynn Clements wrote:

> As I haven't had any response either way on this, I've committed the
> changes.

Sorry, I couldn't check this before.

I confirm it is all fixed now:

1. I can set encoding to any of the available

2. I can enter Polish diacritical characters in v.digit, when I set "Assume
data encoding as:" to UTF-8 (my locale is pl_PL.UTF-8) and they look fine, as
I enter them.

3. When querying, either in v.digit or d.what.vect -e all now works OK.

Excellent. Thanks much! Closing ticket.

Maciek
Comment | Reply | Take | Open

You are currently authenticated as guest.
[Show Configuration] [Login as another user]

Users Guide - Mail Commands - Homepage of RequestTracker 1.0.7 - list any request