← Back to team overview

lp-l10n-fr-community team mailing list archive

Re: Fwd: Re: Clamtk patch

 

Bonjour,

Un bon moyen de s'assurer qu'il soit résolu un jour est de déclarer qu'on est affecté par ce bug. A l'heure actuelle, nous sommes 6 personnes à l'avoir fait et la priorité du bug est à "low".
Si on le fait tous, ça pourrait évoluer un peu.

Cordialement,
   Laurent Lyaudet

On 16/08/2013 14:46, Nicolas Delvaux wrote:
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


_______________________________________________
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




References