[0/4,v3] Revise crypto documentation, deprecate DES, add --disable-crypt

Message ID 20180506175153.22629-1-zackw@panix.com
Headers show
Series
  • Revise crypto documentation, deprecate DES, add --disable-crypt
Related show

Message

Zack Weinberg May 6, 2018, 5:51 p.m.
As requested by Rical Jasan, this revision of the patchset
<https://sourceware.org/ml/libc-alpha/2018-04/msg00628.html> carefully
splits the documentation changes into one patch that just moves text
around, and another patch that changes text but doesn't move it.  I
have also made some more improvements to the documentation.  There are
no new changes to code.

Rical said:

> I have some concern over the creation of a whole chapter for "Obsolete

> Encryption".  I don't think that sets a desirable precedent for chapter

> topics.  I think combining the disparate sections on randomness is a

> good call, though.

>

> What do you think about continuing to use crypt.texi for crytographic

> topics and creating sections for "Obsolete Encryption" and "Random

> Number Generation" there (while otherwise keeping the rest of the

> changes, such as dropping "Legal Problems" and FIPS compliance, updating

> the description of crypt, etc.)?  "Password Handling" could likewise be

> made its own section.


I agree that "Obsolete Encryption" does not need to be its own
chapter; in this revision it goes back to being a subsection of
crypt.texi.  "Random Number Generation" is still a new independent
chapter.  This is because I want to put "Unpredictable Bytes" right
after "Pseudo-Random Number Generation", and it doesn't seem to make
sense to have that consolidated subtree be either with math.texi or
crypt.texi.

I am not sure whether I remembered to address all of your other
comments on the documentation changes, but I got the impression you
wanted to re-review it from scratch anyway after the split-up?

Zack Weinberg (4):
  Reorganize manual sections on cryptography and random numbers.
  Revise manual sections on cryptography and random numbers.
  Deprecate DES encryption functions.
  New configure option --disable-crypt.

 INSTALL                    |  11 +
 Makeconfig                 |   9 +-
 NEWS                       |  20 +
 config.make.in             |   1 +
 configure                  |  18 +
 configure.ac               |  11 +
 conform/Makefile           |  11 +-
 crypt/Makefile             |   5 +-
 crypt/crypt.h              |  22 +-
 elf/Makefile               |  27 +-
 elf/tst-linkall-static.c   |   4 +-
 manual/Makefile            |   2 +-
 manual/arith.texi          |   2 +-
 manual/crypt.texi          | 662 +++++++++++++-------------------
 manual/examples/genpass.c  |  38 +-
 manual/examples/testpass.c |  34 +-
 manual/install.texi        |  12 +
 manual/math.texi           | 586 +---------------------------
 manual/random.texi         | 754 +++++++++++++++++++++++++++++++++++++
 manual/string.texi         |   6 +-
 manual/terminal.texi       |  45 +++
 manual/time.texi           |   2 +-
 posix/unistd.h             |  15 +-
 stdlib/stdlib.h            |   9 +-
 sunrpc/Makefile            |   9 +
 sunrpc/rpc/des_crypt.h     |  11 +-
 26 files changed, 1282 insertions(+), 1044 deletions(-)
 create mode 100644 manual/random.texi

-- 
2.17.0

Comments

Rical Jasan May 8, 2018, 10:56 p.m. | #1
On 05/06/2018 10:51 AM, Zack Weinberg wrote:
> I am not sure whether I remembered to address all of your other

> comments on the documentation changes, but I got the impression you

> wanted to re-review it from scratch anyway after the split-up?


Correct, thank you.  I'll be reviewing shortly.