Bug 161163 - [patch] textproc/codespell: Update to latest version
Summary: [patch] textproc/codespell: Update to latest version
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Frederic Culot
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-30 16:30 UTC by Glen Barber
Modified: 2011-10-01 09:10 UTC (History)
0 users

See Also:


Attachments
file.diff (1.03 KB, patch)
2011-09-30 16:30 UTC, Glen Barber
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Glen Barber freebsd_committer 2011-09-30 16:30:11 UTC
Update textproc/codespell to version 1.2:

1. There's an option to use chardet for encoding detection. If your
code base has mixed encodings, other then utf-8 / ascii / iso8859-1,
this is a nice addition since it will be able to properly detect the
encoding. I kept the other one as default because utf-8 is the most
used encoding (at least in source code) and chardet is much slower
than my hackish encoding detection (from my benchmarks, ~10 times).

2. Skip files that match a pattern. One issue with codespell when
running in recursive mode is that it would try to fix whatever file is
not inside .svn/.git dirs. It can ignore binary files, but there are
some text files that we are better not patching: *.eps, cscope.out,
*.svg etc. Now there's an option to skip them accepting globs:
codespell.py --skip "*.eps,cscope.out,*.svg".

3. Makefile to ease the integration in distributions. I created a
package for Archlinux using it
(http://aur.archlinux.org/packages.php?ID=49980)

Fix: Patch is attached.

Tinderbox logs can be found here:

https://builder.glenbarber.us/tb/logs/9-64-FreeBSD/codespell-1.2.log
https://builder.glenbarber.us/tb/logs/8-64-FreeBSD/codespell-1.2.log
https://builder.glenbarber.us/tb/logs/7-64-FreeBSD/codespell-1.2.log
https://builder.glenbarber.us/tb/logs/9-32-FreeBSD/codespell-1.2.log
https://builder.glenbarber.us/tb/logs/8-32-FreeBSD/codespell-1.2.log
https://builder.glenbarber.us/tb/logs/7-32-FreeBSD/codespell-1.2.log



Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer 2011-09-30 16:30:22 UTC
Responsible Changed
From-To: freebsd-ports-bugs->gjb

Submitter is maintainer and has GNATS access (via the GNATS Auto Assign 
Tool)
Comment 2 Glen Barber freebsd_committer 2011-09-30 16:34:38 UTC
Responsible Changed
From-To: gjb->freebsd-ports-bugs

Back to the ports committers.
Comment 3 Frederic Culot freebsd_committer 2011-09-30 16:43:42 UTC
Responsible Changed
From-To: freebsd-ports-bugs->culot

I'll take it.
Comment 4 dfilter service freebsd_committer 2011-10-01 09:05:13 UTC
culot       2011-10-01 08:05:00 UTC

  FreeBSD ports repository

  Modified files:
    textproc/codespell   Makefile distinfo 
  Log:
  - Update to 1.2
  
  PR:             ports/161163
  Submitted by:   gjb@ (maintainer)
  
  Revision  Changes    Path
  1.4       +1 -1      ports/textproc/codespell/Makefile
  1.4       +2 -2      ports/textproc/codespell/distinfo
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 5 Frederic Culot freebsd_committer 2011-10-01 09:05:57 UTC
State Changed
From-To: open->closed

Committed. Thanks gjb!