[aspell-devel] address@hidden: Bug#310911: aspell: meaning of -l changed from 0.50 with no warning]

[Brian Nelson]

Do you think this patch is worth applying?  It seems sane enough to
me...

-- 
Society is never going to make any progress until we all learn to
pretend to like each other.
> --- Begin Message ---
>
>
>
> Subject: 
>
> Bug#310911: aspell: meaning of -l changed from 0.50 with no warning
>
>
>
>
> Date: 
>
> Thu, 26 May 2005 14:32:54 -0700
>
>
>
>
> Package: aspell
> Version: 0.60.2+20050121-2
> Severity: minor
>
> Between CVS r1.74 and r1.75 of prog/aspell.cpp the meaning of -l changed
> from list to --lang. This change broke a script I was using when I upgraded
> from aspell 0.50* to aspell6. Debugging the problem took a while because 
> I saw 'You must specify a parameter for "-l"' and added a --lang
> parameter to -l. 
>
> I think changing the meaning of a parameter to something totally
> different should have at least been mentioned in changelog.Debian.gz or
> changelog.gz but the only "documentation" I can find for the change is
> bug #301026.
>
> I've attached a patch which outputs a more helpful error if one uses -l
> without a parameter.
>
> In handsight I should have seen and been clued in by a 'You must specify
> an action' message from aspell but I did not see it.
>
>
> -- System Information:
> Debian Release: 3.1
>   APT prefers testing
>   APT policy: (650, 'testing'), (600, 'unstable'), (1, 'experimental')
> Architecture: i386 (i686)
> Kernel: Linux 2.6.11-1-686-smp
> Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
>
> Versions of packages aspell depends on:
> ii  aspell-bin             0.60.2+20050121-2 GNU Aspell standalone spell-check 
> ii  aspell-en [aspell6-dic 6.0-0-3           English dictionary for GNU Aspell
>
> -- no debconf information
>
> *** aspell-warn-about-l.patch
> --- aspell-0.60.2+20050121/prog/aspell.cpp      2004-12-13 13:20:09.000000000 
> -0800
> +++ aspell-0.60.2+20050121.mine/prog/aspell.cpp 2005-05-26 14:16:04.000000000 
> -0700
> @@ -322,6 +322,9 @@
>            i += 1;
>          } else {
>            print_error(_("You must specify a parameter for \"%s\"."), argv[i]);
> +          if (argv[i][0] == '-' && argv[i][1] == 'l' && argv[i][2] == '\0') {
> +            print_error(_("The meaning of -l changed in aspell 0.60. Use the 
> list command to get the behavior of -l in aspell 0.50.5."));
> +          }
>            return 1;
>          }
>        } else {
>
>
> --- End Message ---