patchWeeChat - Patches: patch #6858, port weechat to enchant instead of...

 
 
Show feedback again

patch #6858: port weechat to enchant instead of aspell

Submitted by:  Caolan McNamara <caolan>
Submitted on:  Tue 30 Jun 2009 04:39:29 PM UTC  
 
Category: aspell pluginPriority: 3 - Low
Status: Need InfoPrivacy: Public
Assigned to: Sebastien Helleu <flashcode>Open/Closed: Open
IRC nick: 

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat 12 Sep 2009 04:45:50 PM UTC, comment #3:

As an aside, you could use "iso-codes" library to convert the lang/territory ids to the (optionally translated) language/territory names

Caolan McNamara <caolan>
Wed 01 Jul 2009 04:50:16 PM UTC, comment #2:

Hmm... I didn't notice something important: your patch is against WeeChat 0.2.6.3, you'll have to rewrite it against last devel version (devel version is very different from 0.2.6.x branch).

Sebastien Helleu <flashcode>
Project AdministratorIn charge of this item.
Wed 01 Jul 2009 12:14:55 PM UTC, comment #1:

Hi,

Thank you for patch.

Would it be possible to add another patch for cmake stuff? (cmake is now recommended way to compile weechat, even if there is still autotools).

Moreover, this patch will be used in version 0.3.1 (I hope, or later, I don't know), because new features are freezed for 0.3.0.

Sebastien Helleu <flashcode>
Project AdministratorIn charge of this item.
Tue 30 Jun 2009 04:39:29 PM UTC, original submission:

As described at http://fedoraproject.org/wiki/Releases/FeatureDictionary in fedora for some time we've been trying to unify dictionary support so that all apps use the same spelling dictionaries by default. (other distros have similar efforts)

In that vein attached is a patch that prefers enchant to aspell as the spell-checker for weechat, which gives the same spelling dictionaries by default as used by OpenOffice.org/Firefox/GNOME/KDE etc.

Little bit tricky is that the spelling plugin and command is called "aspell" in weechat, so rather than add another one, or rename the current one, the patch retains /aspell etc and just default to implementing it with enchant when available

Caolan McNamara <caolan>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #18702:  weechat-gitHEAD.enchant.patch added by caolan (27kB - text/x-patch - update for against git HEAD)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by seschwar
  • -unavailable- added by flashcode (Posted a comment)
  • -unavailable- added by caolan (Submitted the item)
  • -unavailable- added by caolan
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 02 Jun 2012 05:24:33 PM UTCseschwarCarbon-Copy-=>Added seschwar
    Sat 12 Sep 2009 04:43:59 PM UTCcaolanAttached File-=>Added weechat-gitHEAD.enchant.patch, #18702
    Wed 01 Jul 2009 04:50:15 PM UTCflashcodeStatusPostponed=>Need Info
    Wed 01 Jul 2009 12:14:55 PM UTCflashcodeCategoryNone=>aspell plugin
      Priority5 - Normal=>3 - Low
      StatusNone=>Postponed
      Assigned toNone=>flashcode
    Tue 30 Jun 2009 04:39:30 PM UTCcaolanAttached File-=>Added weechat-0.2.6.3.enchant.patch, #18346
      Carbon-Copy-=>Added -unavailable-
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup