MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001259Orfeo Toolbox (OTB)Generalpublic2016-07-29 00:222017-04-27 15:33
Reportersebastic 
Assigned Togpasero 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Summary0001259: Patch for various spelling errors in 5.6.0
DescriptionThe lintian QA tool reported a couple of spelling errors for the OTB 5.6.0 Debian package build:

 * allows to <verb> -> allows <verb>ing
 * formated -> formatted
 * Unkown -> Unknown
TagsNo tags attached.
Attached Filespatch file icon spelling-errors.patch [^] (5,939 bytes) 2016-07-29 00:22 [Show Content]

- Relationships

-  Notes
(0003613)
grizonnetm (administrator)
2016-07-29 09:47

We should automate the use of lintial during the release process to handle these spelling errors before the release
(0003614)
sebastic (reporter)
2016-07-29 09:54

lintian is not easy to use outside of the Debian package workflow.

For upstream projects I suggest to use codespell [0]. GDAL uses a modified codespell with the dictionary from lintian and QGIS [1], which is probably a good option for OTB too.

[0] https://github.com/lucasdemarchi/codespell/ [^]
[1] https://trac.osgeo.org/gdal/browser/trunk/gdal/scripts/fix_typos.sh [^]
(0003619)
gpasero (administrator)
2016-07-29 15:45

Thanks again! Patch pushed in branch release-5.6
https://git.orfeo-toolbox.org/otb.git/commit/30b55712719bbee182a0ff4f473f2941d1c98b98 [^]
(0003730)
grizonnetm (administrator)
2016-09-29 09:42

Sebastic,

following your suggestion I've added a script to check spelling errors in OTB:

https://git.orfeo-toolbox.org/otb.git/blob/refs/heads/develop:/Utilities/Maintenance/fix_typos.sh [^]

It will be applied before releasing the sources.

Thanks again.

- Issue History
Date Modified Username Field Change
2016-07-29 00:22 sebastic New Issue
2016-07-29 00:22 sebastic File Added: spelling-errors.patch
2016-07-29 09:47 grizonnetm Note Added: 0003613
2016-07-29 09:54 sebastic Note Added: 0003614
2016-07-29 15:45 gpasero Note Added: 0003619
2016-07-29 15:45 gpasero Status new => resolved
2016-07-29 15:45 gpasero Resolution open => fixed
2016-07-29 15:45 gpasero Assigned To => gpasero
2016-09-29 09:42 grizonnetm Note Added: 0003730
2017-04-27 15:33 julien Status resolved => closed


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker