[voikko-devel] Voikko-provider Enchantille

Anssi Hannula anssi.hannula at gmail.com
Sun Aug 20 00:17:43 EEST 2006


Harri Pitkänen wrote:
> On Saturday 19 August 2006 22:24, Anssi Hannula wrote:
> 
>>Hei!
>>
>>Liitteenä on patchi enchantin 1.2.6 -versiolle, joka lisää tuen
>>Voikolle. Se on muokattu versio tmispell-voikon sisältämästä
>>enchant-pluginista siten, että se käyttää suoraan Voikkoa.
>>
>>Tarkoitus olisi lähettää tämä patchi lisättäväksi enchantiin, mutta
>>laitan kuitenkin tänne ensin, jos joku haluaa sen tarkistaa virheiden
>>varalta.
> 
> 
> Näyttääpä hyvin samantapaiselta mitä itse lähetin Enchantiin lisättäväksi 
> tänään iltapäivällä :)
> http://bugzilla.abisource.com/show_bug.cgi?id=10450

Tosiaankin :)

> On tuossa sinun patchissasi selvästi joitakin kohtia, jotka on paremmin tehty 
> kuin minulla. Erityisesti minulta puuttuu nuo voikko_set_*_option -kutsut, 
> kun en ole ehtinyt tutkia miten Enchantia käyttävät sovellukset yleensä 
> olettavat oikolukukomponenttien toimivan esimerkiksi numeroita sisältävien 
> sanojen kohdalla. Luultavasti kuitenkin nuo asettamasi asetukset ovat aivan 
> oikein ja tarpeellisia.

Täytyy kyllä nyt myöntää, että asetin nuo kutsut vain koska myös
tmispell tekee niin.

> dict_check -funktiosta voi ilmeisesti palauttaa myös -1 jos oikolukumoduuli 
> havaitsee sisäisen virheen (2=internal error, 3=charset error).
> 
> Oma patchini on Enchantin CVS HEAD-versioon, ja sen pohjana on käytetty 
> uusinta Aspell-pluginia. Noissa uusissa versioissa on mukana ylimääräinen 
> lisäoikeus lisenssitekstissä, ja koska halusin sen yksinkertaisuuden vuoksi 
> pitää mukana, en käyttänyt tmispellin mukana tulevaa versiota tuon patchin 
> pohjana. Tuo lisenssipoikkeus on Voikko-moduulille merkityksetön, joten en 
> tiedä olisiko sitä ollut tarpeen säilyttää.
> 
> Ehkä meidän pitäisi yhdistää nämä patchit ja laittaa tuohon bugiin päivitetty 
> versio. Jos haluat tämän tehdä, olisin siitä kiitollinen koska minulla on 
> pariksi seuraavaksi viikoksi aika tiukasti töitä tämän COSS:in 
> kesäkoodiprojektin sopimuksen mukaisten osien (siis Joukahaisen) 
> viimeistelyssä. Parasta varmaan on tehdä niin, että siirrät omia 
> parannuksiasi tekemääni patchiin niin saadaan tiedoston lisenssi pidettyä 
> muodossa LGPL+poikkeus eikä tarvitse lähteä Pauli Virtaselta erikseen näin 
> pienen asian takia kyselemään lupia uudelleenlisensöintiin.
> 
> Muuttamani ja lisäämäni tiedostot löytyvät myös SVN:stä (trunk/enchant).

Voinhan minä sen tehdä, juu. Ei tuossa sinun patchissa oikeastaan juuri
ole muutettavaa, paitsi että:
- configure.inissä näyttäisi olevan virhe ("-lvoikko" ei asetettu kun
--with-voikko-prefix ei käytetty)
- voikko_handlen tallettaisin suoraan user_dataan

Lisäksi tarkistan vielä Abiwordilla, ovatko nuo set_bool_option-kutsut
tarpeellisia, ja lisään niitä tarvittaessa.

-- 
Anssi Hannula




More information about the devel mailing list