[Pkg-openssl-devel] Bug#622820: Preliminary Patches

Scott Schaefer saschaefer at neurodiverse.org
Mon May 2 23:48:08 UTC 2011


As you have noted, there are numerous POD errors in the documentation.

Most, but not all of them, are due to the pod2man translator enforcing 
the rule that, if the first "item" value is numeric, then all subsequent 
item values in the same "over/back" set be the next integer.  There are 
others, such as just plain missing 'back' elements.

Obviously, pod2man was modified at some point to enforce this rule.  I 
briefly reviewed, and there is no way to selectively 'quiet' it to 
ignore these errors.  They implement this via both a scream() method and 
a whine() method.  The latter can be disabled.  And, while the errors we 
are discussing are reported with whine(), there are a (large?) number of 
other tests which should be retained.  And, I am certainly in no 
position to suggest to QA that it modify its build processes ...

In addition, the lintian report lists a number of misspellings in both 
the POD documentation, and in both the built binaries and libraries; 
i.e. in a number of source files.

Unfortunately, this begs the question of the proper way to fix these.  I 
can proceed in most any way you want.  My $0.02:

I can fix the item-value-not-next-integer value by either:
   1) rearranging the values, or
   2) inserting extra back/over pairs between items (will have to verify 
this results in same output, but I believe it does).
   Note that upstream is probably more amenable to #2.

Obviously, I can fix misspellings from lintian report.

Which then gets us to question(s) about patch files.  Would you prefer:
   a) 1-2 patch files; i.e. misc-pod-errors.patch and 
misc-misspellings.patch
   b) separate patch files, one per individual file,
   c) merge the patches into any existing patch that references same 
source file prior to either a), b)






More information about the Pkg-openssl-devel mailing list