← Back to team overview

lp-l10n-fr-community team mailing list archive

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