Bug 4632

Summary: rpmsrate is missing a lot packages in CAT_SPELLCHECK
Product: Mageia Reporter: Manuel Hiebel <manuel.mageia>
Component: Release (media or process)Assignee: Kamil Rytarowski <n54>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: release_blocker CC: ennael1, lmenut, n54, pterjan, sysadmin-bugs, thierry.vignaud
Version: Cauldron   
Target Milestone: ---   
Hardware: i586   
OS: Linux   
URL: http://distrib-coffee.ipsl.jussieu.fr/pub/linux/Mageia/distrib/cauldron/x86_64/media/media_info/rpmsrate
Whiteboard:
Source RPM: hunspell CVE:
Status comment:
Attachments: I think this is because hunspell-XX doesn't require locale-XX
perl example how to bump mkrel & add needed requires

Description Manuel Hiebel 2012-02-22 11:43:11 CET
Description of problem:
After the cleaning on the rpmsrate-raw, it seems that the rpmsrate is become buggy, or it was wrong fixed

CAT_SPELLCHECK
  5 hunspell-bg hunspell-ca
  4 voikko-fi

versus

CAT_SPELLCHECK
  5 myspell-af_ZA myspell-am_AM myspell-ar_AR myspell-az_AZ myspell-bg_BG myspell-bn_BN myspell-br myspell-ca_ES myspell-cs_CZ myspell-csb_CSB 
[....]
myspell-ve_ZA myspell-vi_VI myspell-xh_ZA myspell-zu_ZA

http://svnweb.mageia.org/packages/cauldron/meta-task/current/SOURCES/rpmsrate-raw?view=log
Manuel Hiebel 2012-02-22 11:45:08 CET

Priority: Normal => release_blocker
CC: (none) => boklm, ennael1, lmenut, n54, thierry.vignaud

Comment 1 Anne Nicolas 2012-02-22 11:46:54 CET
Be carefull this is rpmsrate-raw. It is explanded then on final repository in media/media_info/rpmsrate. So we don't have to list all of them, only 2.
Comment 2 Manuel Hiebel 2012-02-22 11:49:44 CET
ok, but http://distrib-coffee.ipsl.jussieu.fr/pub/linux/Mageia/distrib/1/x86_64/media/media_info/rpmsrate is the one used no ? (cf url)
Comment 3 Anne Nicolas 2012-02-22 11:52:30 CET
Oups sorry read this too fast. Indeed,it was not expanded properly
Comment 4 Anne Nicolas 2012-03-14 10:18:04 CET
Thierry or Pascal can you have a look there? looks like mkcd stroke again

CC: (none) => pterjan

Comment 5 Thierry Vignaud 2012-03-14 11:20:20 CET
well
CAT_SPELLCHECK
 5 hunspell-bg hunspell-ca 129 4 voikko-fi

got expanded into:
CAT_SPELLCHECK
  5 myspell-af_ZA myspell-am_AM (...)
  4 aspell-af aspell-am aspell-az (...)
  4 voikko-fi

Guess it's considering the provides when expanding
For you Pascal...

Source RPM: meta-task => mgatools

Comment 6 Thierry Vignaud 2012-03-14 11:23:12 CET
BTW I can't reproduce locally with mgatools
What's running on the server that is expanding? mdv? mga1?
Comment 7 Pascal Terjan 2012-03-14 11:24:59 CET
mga1
Comment 8 Thierry Vignaud 2012-03-14 11:29:20 CET
Created attachment 1762 [details]
I think this is because hunspell-XX doesn't require locale-XX

Note that this should NOT be applied.
hunspell-XX should be fixed instead
Comment 9 Thierry Vignaud 2012-03-14 11:50:33 CET
Kamil, can you add the proper Requires:locales-XX to hunspell-XX ?
Should be pretty automatisable with perl
Comment 10 Thierry Vignaud 2012-03-14 11:50:54 CET
Created attachment 1764 [details]
perl example how to bump mkrel & add needed requires
Comment 11 Kamil Rytarowski 2012-03-14 21:01:36 CET
(In reply to comment #9)
> Kamil, can you add the proper Requires:locales-XX to hunspell-XX ?
> Should be pretty automatisable with perl

Is this really the only way? I was looking previously into distros, and dictionaries don't need locales.... how about users who just want use dictionaries in different languages? Are they forced to install extra unneeded locales?

Also there are many languages without locales...
Comment 12 Kamil Rytarowski 2012-03-14 22:33:10 CET
(In reply to comment #9)
> Kamil, can you add the proper Requires:locales-XX to hunspell-XX ?
> Should be pretty automatisable with perl

We are using univeral providing for Hunspell (and I have to check the Finnish voikko) dictionary-XX. Can we make a use of it?
Comment 13 Thierry Vignaud 2012-03-15 13:36:02 CET
Well, we don't care how other distro do it :-)
This is how we always worked (for more than a decade).
Eg: aspell-XX requires the proper locales-XX package
Same for firefox-XX

As for "dictionaries don't need locales", this is not true, things like word breaking can depend on locale definition (what compound letters should remains as a block, what is word break, ...).
see glibc-i18ndata
Comment 14 Thierry Vignaud 2012-03-15 14:23:25 CET
Last but not least, _NO_ other distro have locales-XX :-) (modulo mdv of course)
So distro comparison is offtopic

So please add those requires.
Thierry Vignaud 2012-03-15 14:23:58 CET

Attachment 1764 description: example => perl example how to bump mkrel & add needed requires

Thierry Vignaud 2012-03-15 14:24:28 CET

Assignee: bugsquad => n54
Source RPM: mgatools => hunspell

Comment 15 Kamil Rytarowski 2012-03-15 15:14:44 CET
(In reply to comment #14 and #15)
Ah. OK, I will do it tonight

Status: NEW => ASSIGNED

Comment 16 Thierry Vignaud 2012-03-15 17:43:54 CET
Thx
Comment 17 Kamil Rytarowski 2012-03-17 21:33:27 CET
(In reply to comment #16)
> Thx

Hi! Excuse me, I'm practicing my Perl programming and I'm writing a script in Perl to do it globally and under control. And now for a while I'm on a small trip, so it's slower.
Comment 18 Kamil Rytarowski 2012-03-20 03:15:15 CET
(In reply to comment #9)
> Kamil, can you add the proper Requires:locales-XX to hunspell-XX ?

hunspell-so-0.1.5-4.mga2
hspell-1.2-2.mga2
hunspell-zu-0.20100126.1-6.mga2
hunspell-yi-1.1-5.mga2
hunspell-xh-0.20091030.1-5.mga2
hunspell-wa-0.4.15-7.mga2
hunspell-vi-0.20080604.1-7.mga2
hunspell-ve-0.20091030.1-5.mga2
hunspell-uz-0.6-7.mga2
hunspell-ur-0.64-6.mga2
hunspell-uk-1.6.0-5.mga2
hunspell-ts-0.20091101.1-5.mga2
hunspell-tn-0.20091101.1-5.mga2
hunspell-tl-0.20050109.1-8.mga2
hunspell-tk-0.01-4.mga2
hunspell-th-0.20061212.1-9.mga2
hunspell-te-0.20050929.1-8.mga2
hunspell-ta-20100226-5.mga2
hunspell-sw-0.20050819.1-8.mga2
hunspell-sv-1.44-5.mga2
hunspell-st-0.20091030.1-5.mga2
hunspell-ss-0.20091030.1-5.mga2
hunspell-sr-0.20100920.1-6.mga2
hunspell-sq-1.6.2-4.mga2
hunspell-sl-0.20070127.1-9.mga2
hunspell-sk-0.20091213.1-6.mga2
hunspell-si-0.2.1-6.mga2
hunspell-shs-0.20090828.1-5.mga2
hunspell-se-1.0-0.3.beta7.3.mga2
hunspell-sc-0.20081101.1-7.mga2
hunspell-rw-0.20050109.1-7.mga2
hunspell-ru-0.99f7-8.mga2
hunspell-ro-3.3.6-5.mga2
hunspell-pt-0.20111025.1-5.mga2
hunspell-pl-0.20110708.1-5.mga2
hunspell-pa-20050726-8.mga2
hunspell-or-20050726-8.mga2
hunspell-oc-0.5-7.mga2
hunspell-nso-0.20091201.1-5.mga2
hunspell-nr-0.20091030.1-5.mga2
hunspell-nl-2.00-6.mga2
hunspell-ne-20080425-5.mga2
hunspell-nds-0.1-7.mga2
hunspell-mt-0.20020708.1-7.mga2
hunspell-ms-0.20050117.1-8.mga2
hunspell-mr-20060920-10.mga2
hunspell-mn-0.20080709.1-6.mga2
hunspell-ml-0.1-7.mga2
hunspell-mk-0.20051126.1-6.mga2
hunspell-mi-0.20080630.1-7.mga2
hunspell-mg-0.20050109.1-8.mga2
hunspell-mai-1.0.1-4.mga2
hunspell-lv-0.9.4-7.mga2
hunspell-lt-1.2.1-9.mga2
hunspell-lb-0.20110513.1-5.mga2
hunspell-ky-0.20090415.1-6.mga2
hunspell-ku-0.21-10.mga2
hunspell-ko-0.5.3-4.mga2
hunspell-kn-1.0.3-5.mga2
hunspell-km-1.1-6.mga2
hunspell-kk-1.1-5.mga2
hunspell-it-2.4-0.6.20070901.4.mga2
hunspell-is-0.20090823.1-5.mga2
hunspell-id-0.20040812.1-8.mga2
hunspell-hy-0.20.0-5.mga2
hunspell-hu-1.6.1-5.mga2
hunspell-ht-0.06-4.mga2
hunspell-hsb-0.20060327.3-3.mga2
hunspell-hr-0.20040608.1-11.mga2
hunspell-hi-20050726-10.mga2
hunspell-gv-0.20040505.1-7.mga2
hunspell-gu-20061015-6.mga2
hunspell-gl-0.20080515.1-7.mga2
hunspell-gd-2.1-3.mga2
hunspell-ga-4.6-4.mga2
hunspell-fy-2.0.1-5.mga2
hunspell-fur-0.20050912.1-8.mga2
hunspell-fr-4.2-5.mga2
hunspell-fo-0.4.0-4.mga2
hunspell-fa-0.20070116.1-8.mga2
hunspell-eu-0.20080507.1-7.mga2
hunspell-et-0.20030606.1-11.mga2
hunspell-es-0.20081215.1-7.mga2
hunspell-eo-1.0-0.3.dev.5.mga2
hunspell-en-0.20110318.1-5.mga2
hunspell-el-0.8-6.mga2
hunspell-de-0.20110609.1-5.mga2
hunspell-da-1.7.37-4.mga2
hunspell-cy-0.20040425.1-9.mga2
hunspell-cv-1.02-4.mga2
hunspell-csb-0.20050311.1-9.mga2
hunspell-cs-20060303-15.mga2
hunspell-ca-2.2.0-6.mga2
hunspell-br-0.8-4.mga2
hunspell-bn-0.04-5.mga2
hunspell-bg-4.3-5.mga2
hunspell-ber-0.20080210.1-6.mga2
hunspell-be-1.1-6.mga2
hunspell-az-0.20040827.1-9.mga2
hunspell-ast-0.02-2.mga2
hunspell-as-1.0.3-7.mga2
hunspell-ar-0.20080110.1-9.mga2
hunspell-am-0.20090704.1-8.mga2
hunspell-af-0.20080825.1-8.mga2

104 packages are updated.
The last one seems to be -nb, we shall consider to obsolete locales-no with locales-nb.

What about mythes-*, hyphen-* and autocorr-*?

> Should be pretty automatisable with perl
It took me 1 full day to do it with Perl.
Comment 19 Thierry Vignaud 2012-03-20 08:25:46 CET
No need to obsolete it, locales-no provides it (it was renamed)
Comment 20 Kamil Rytarowski 2012-03-20 10:00:35 CET
(In reply to comment #19)
> No need to obsolete it, locales-no provides it (it was renamed)

OK. I have added the requirement on locales-no for hunspell-nn and hunspell-nb.

So everything is done with hunspell.

Now it's time for mythes-*, hyphen-* and autocorr-*.

I will do the same for these packages.
Comment 21 Kamil Rytarowski 2012-03-20 10:03:29 CET
(In reply to comment #8)
> Created attachment 1762 [details]
> I think this is because hunspell-XX doesn't require locale-XX
> 
> Note that this should NOT be applied.
> hunspell-XX should be fixed instead

Tell me please if it's now working.
Comment 22 Thierry Vignaud 2012-03-20 10:05:15 CET
> (In reply to comment #19)
Note it's not urgent since those are not in rpmsrate and are only required by
packages (libreoffice-langpack-XX) that also requires locales-XX(In reply to comment #20)

> (In reply to comment #20)
Yes. Just look at media/media_info/rpmsrate on your favorite mirror
Comment 23 Kamil Rytarowski 2012-03-20 10:10:35 CET
> > (In reply to comment #19)
> Note it's not urgent since those are not in rpmsrate and are only required by
> packages (libreoffice-langpack-XX) that also requires locales-XX(In reply to
> comment #20)
OK! So I won't apply changes. I am planning to clean these packages for mga3 (importing possibly missing ones, etc).

> 
> > (In reply to comment #20)
> Yes. Just look at media/media_info/rpmsrate on your favorite mirror

Wow, nice.
Feel free to close this bug as resolved if everything is fine.
Comment 24 Thierry Vignaud 2012-03-20 10:40:02 CET
Closing

Status: ASSIGNED => RESOLVED
Resolution: (none) => FIXED

Comment 25 Manuel Hiebel 2012-03-20 10:45:37 CET
thanks all \o/
Nicolas Vigier 2014-05-08 18:05:26 CEST

CC: boklm => (none)