← Back to team overview

lp-l10n-fr-community team mailing list archive

Re: Fwd: Re: Clamtk patch

 

La bonne nouvelle c'est qu'il s'agit bien d'un bug :
https://bugs.launchpad.net/launchpad/+bug/293680

La mauvaise c'est que ça fait bientôt 5 ans que le problème est connu...


Le 16/08/2013 14:06, Nicolas Delvaux a écrit :
> Bonjour,
> 
> Le 15/08/2013 11:14, Laurent Lyaudet a écrit :
>> Bonjour,
>>
>> De rien :)
>> Je viens d'essayer d'uploader le fichier mais launchpad me dit que
>> l'accès à la page m'est interdit.
>> Je joins une capture d'écran de la page à mon mail.
> 
> En effet , je viens de créer un utilisateur test et il ne peut pas non
> plus uploader. Apparemment il faut être membre de l'équipe de traduction
> pour pouvoir le faire (ce que je trouve stupide, à moins que ce ne soit
> qu'un bug).
> Il est vrai qu'à la base la traduction sur Launchpad est sensé se faire
> via l'interface Web, mais bon...
> 
> Pour faire simple, je vais te rajouter à l'équipe le temps que tu upload
> le fichier.
> 
> Bien que ton travail sur Clamtk soit bon, on demande plus d'ancienneté
> avant d'accepter de nouveaux membres, du coup je te « virerai » dès que
> ce sera fait ;-)
> 
> Je te fais confiance pour ne pas faire de bêtise pendant que tu auras
> tes super pouvoirs ;-)
> 
> Nicolas
> 
>> Est-ce que mes changements sur les combinaisons Alt+lettre apparaîtront
>> aussi comme suggestions ?
>> J'ai aussi ajouté un gros commentaire sur une traduction non modifiée.
>>
>> Cordialement,
>>    Laurent
>>
>>
>> On 15/08/2013 02:40, Nicolas Delvaux wrote:
>>> 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
>>>>
>>>
>>
> 
> 
> _______________________________________________
> 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