lp-l10n-fr-community team mailing list archive
-
lp-l10n-fr-community team
-
Mailing list archive
-
Message #00362
Re: Fwd: Re: Clamtk patch
Bonjour et merci pour cette contribution,
Si ce n'est pas déjà fait, il faut que tu upload ce fichier PO sur
Launchpad via ce lien:
https://translations.launchpad.net/clamtk/trunk/+pots/clamtk/fr/+upload
(il vaut mieux que ça soit toi qui le fasse car Launchpad attribue les
modifications à l'uploader)
Il sera ensuite beaucoup plus facile pour nous de relire tes traductions
car tous tes changements apparaîtront en tant que nouvelles suggestions
dans Launchpad.
Prévient nous dès que ce sera fait :-)
Cordialement,
Nicolas
Le 11/08/2013 17:04, Laurent Lyaudet a écrit :
> Bonjour,
>
> Voici un fichier de traduction pour clamtk modifié afin que vous le
> validiez.
> J'ai principalement corrigé afin de faire en sorte que tous les menus et
> actions correspondent à des combinaisons Alt+lettre différentes.
>
> Cordialement,
> Laurent Lyaudet
>
>
> -------- Original Message --------
> Subject: Re: Clamtk patch
> Date: Sun, 11 Aug 2013 07:35:02 -0500
> From: Dave M <dave.nerd@xxxxxxxxx>
> To: laurent lyaudet <laurent.lyaudet@xxxxxxxxxxxxxxx>
>
>
>
> Hello Laurent,
>
> Thank you for the help - I will take a look at it today. Note that I
> am not doing much with the 4.xx series because I am trying to work on
> the 5.xx series, which will be very different. Maybe for the better,
> and maybe not. :)
> Nevertheless, I will certainly have a look.
>
> Regarding the translation - would you mind working it through Launchpad?
> https://translations.launchpad.net/clamtk/trunk/+pots/clamtk/fr/+translate
>
> It is much easier on both of us - that way I do not mess something
> up... If you have any questions about that, please let me know. I
> think this link - http://doc.ubuntu-fr.org/lp_l10n_fr - may be of help
> as well.
>
> Again, thank you for contacting me. I'll be looking at the patch today.
>
> Sorry, one more thing - the jabber one is for chat, which I don't use.
> I need to remove it from the website. Please use this gmail account
> for contact.
>
> respectfully
> dave
>
> On Sun, Aug 11, 2013 at 7:27 AM, laurent lyaudet
> <laurent.lyaudet@xxxxxxxxxxxxxxx> wrote:
>> Hi,
>>
>> I corrected a minor bug in clamtk.
>> In fact at first, I wanted to correct the bug that scanning recursively a
>> directory would not scan recursively if the preference was not set for
>> recursive.
>> But it appears that this bug was already corrected between 4.38 (ubuntu
>> latest version) and 4.45 (it doesn't appear in the changelog).
>> Anyway the correction is still buggy:
>>
>> if ( !$prefs{Recursive} ) {
>> if ( $option ne 'recur' ) {
>> $directive .= ' --max-dir-recursion=1';
>> }
>> } elsif ( $prefs{Recursive} ) {
>> $directive .= ' --recursive';
>> }
>>
>> if ( $option eq 'recur' ) {
>> $directive .= ' --recursive';
>> }
>>
>> Code above is buggy:
>> if $option == 'recur' and $prefs{Recursive} is true, you end up with
>> $directive .= ' --recursive'; twice.
>> At least it is less buggy than the code in 4.38 and that I also found here
>> http://clamtk.cvs.sourceforge.net/clamtk/
>>
>> 921 if ( !$prefs{Recursive} ) {
>> 922 $directive .= ' --max-dir-recursion=1'
>> 923 if ( $option ne 'recur' );
>> 924 } elsif ( $prefs{Recursive} || $option eq 'recur' ) {
>> 925 $directive .= ' --recursive';
>> 926 }
>>
>> Curiously that's the code that appears when you annotate head version on
>> http://clamtk.cvs.sourceforge.net/clamtk/
>> It should be the same code as in GUI.pm in 4.45 (does the cvs on the website
>> stop before version 4.45? It seems it stopped to version 4.39.)
>>
>> Anyway I suggest the following code instead:
>>
>> if ( $option eq 'recur' || $prefs{Recursive}) {
>> $directive .= ' --recursive';
>> }
>> else{
>> $directive .= ' --max-dir-recursion=1';
>> }
>> I attach a corrected version of GUI.pm.
>>
>> I also corrected the translation file fr.po (joined to this mail), mainly to
>> ensure that all Menus and actions are linked to different Alt+letter
>> combinations. I don't know if there is some kind of translation team that
>> needs to validate that file.
>>
>> I noted that on clamtk website you give two mail address (one as text, one
>> when you click email contact):
>> dave.nerd@xxxxxxxxx
>> dave-m at jabber.org
>> Are you still using both? (if not I hope I choosed the good one ;) )
>>
>> Best regards,
>> Laurent Lyaudet
>>
>
>
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~lp-l10n-fr-community
> Post to : lp-l10n-fr-community@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~lp-l10n-fr-community
> More help : https://help.launchpad.net/ListHelp
>
Follow ups
References