On Thu, Aug 12, 2010 at 4:33 PM, Bram Moolenaar <[email protected]> wrote:
>
> Jakson Aquino wrote:
>
>> >> On Tue, Aug 10, 2010 at 6:15 PM, Bram Moolenaar <[email protected]>
>> >> wrote:
>> >> > Unfortunately, when I run Aap, applying the diff fails. Â It looks like
>> >> > the diff was made for another pt_PT.aff file. Â The one that was
>> >> > downloaded looks like this:
>> >> >
>> >> >
>> >> > SET UTF-8
>> >> > LANG pt_PT
>> >> > TRY
>> >> > aerisontcdmlupvgbfzáhçqjÃxãóéêâúõACMPSBTELGRIFVDkHJONôywUKXZWQÃ
>> >> > Yà Éà ÓèÂÚ
>> >> > KEY qwertyuiop|asdfghjkl|zxcvbnm
>> >> > WORDCHARS -
>> >> >
>> >> >
>> >> > The diff expects:
>> >> >
>> >> > Â SET ISO8859-1
>> >> > - TRY
>> >> > aerisontcdmlupvgbfzáhçqjÃxãóéêâúõACMPSBTELGRIFVDkHJONôywUKXZWQÃ
>> >> > Yà Éà ÓèÂÚ
>> >> >
>> >> >
>> >> > Also, the .aff file has lots of trailing text. Â Are we using the same
>> >> > download?
>> >> > I have made an attempt in fixing this. Â Please check the files on the
>> >> > ftp server: ftp://ftp.vim.org/pub/vim/runtime/spell/pt/
>> >> >
>> >> > Also check spell suggestions with the generated .spl file.
>> >> >
>> >>
>> >> It seems that everything is ok now. Thank you very much for fixing it!
>> >>
>> >> I deleted all files from my local spell/pt directory and restored them
>> >> with "hg updated -C". I compared the files generated by "hg update -C"
>> >> with the ones in the ftp server and they have the same md5sums. Then,
>> >> I generated and copied the pt.utf-8.spl and pt.latin1.spl files to
>> >> ~/.vim/spell. European and Brazilian spell check seems to be correct
>> >> and up to date and the spell suggestions are good.
>> >>
>> >> However, the pt.utf-8.spl from the ftp server seems to be outdated...
>> >> Did you do "aap clean" before generating it?
>> >
>> > No, I didn't update the pt_BR files. Â I'll do that now. Â Hmm, I see a
>> > few warnings...
>>
>> I see some duplicate warnings...
>>
>> fixdup.vim can delete 122 duplicates, but about 1600 would remain.
>>
>> The lines weren't duplicates in the original pt_PT.dic because each
>> root entry---even though carrying the same suffixes---had a different
>> grammatical category. They became duplicates after the deletion of the
>> grammatical information. Most of the remaining duplicates can be
>> deleted with:
>>
>> :2,$ sort u
>>
>> But about 200 duplicated roots would still remain because they have
>> different suffixes. Example:
>>
>> advindo/L
>> advindo/fp
>>
>> I guess that, if there is no problem in the .dic being sorted, we can
>> use ":2,$ sort u" to delete the real duplicates, but I guess that it's
>> better to let the remaining 200 duplicates in the file.
>>
>> So, what to do?
>
> The duplicates are not a real problem. It's useful if you are writing a
> dictionary.
>
> I think using that sort command is good. You can verify that with and
> without the sort command the spell file is the same.
I've made a copy of pt_BR.aff, pt_BR.dic, pt_PT.aff and pt_PT.dic into
four directories. In one directory I ran ":so fixdup.vim" on
pt_PT.dic; in other I ran "2,$ sort u"; and in the third one I did
both. Then I did ":mkspell". The result:
md5sum pt*/pt.utf*
9aee104f71db890d10f6cb49525abc0b ptFixdup/pt.utf-8.spl
9aee104f71db890d10f6cb49525abc0b ptFixdupSorted/pt.utf-8.spl
99bdb0f4a6eaa09503206f604022d2a1 ptOrig/pt.utf-8.spl
99bdb0f4a6eaa09503206f604022d2a1 ptSorted/pt.utf-8.spl
It's confusing to work with so many files with the same name... If I
didn't do any mistake the conclusion is: the sort command doesn't
change the result, but the generated spell file is not the same if
fixdup.vim is sourced. I'm sending a patch to add the sort command to
the preparation of pt_PT.dic.
Thanks,
Jakson
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
diff -r 20e83abf88b1 runtime/spell/pt/main.aap
--- a/runtime/spell/pt/main.aap Thu Aug 12 22:19:09 2010 +0200
+++ b/runtime/spell/pt/main.aap Thu Aug 12 21:34:22 2010 -0300
@@ -59,7 +59,7 @@
:delete {f} description.xml
:delete {f} dictionaries.xcu
:delete {f} LICENSES.txt
- :sys $VIM pt_PT.dic -u NONE -e -c "%s/\t.*//" -c update -c q
+ :sys $VIM pt_PT.dic -u NONE -e -c "%s/\t.*//" -c "2,$$ sort u" -c update -c q
:sys $VIM pt_PT.aff -u NONE -e -c "%s/\S\+=\S\+$$//" -c update -c q
@if not os.path.exists('pt_PT.orig.aff'):
:copy pt_PT.aff pt_PT.orig.aff
