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