Remove id(1) and groups(1)

[alejandro-colomar]

Distributions use id(1) from GNU coreutils or BusyBox. Drop ours.

Closes: #1005
Suggested-by: @dkwo
Cc: @hallyn
Cc: @ikerexxe
Cc: @jubalh
Cc: @thesamesam

---

Revisions:

v1b

• Rebase

$ git range-diff master..gh/idgroups shadow/master..idgroups 
1:  8e856bbd = 1:  ba31ca7b Remove id(1)

v1c

• Rebase

$ git range-diff alx/master..gh/idgroups shadow/master..idgroups 
1:  ba31ca7b = 1:  b2cec0be Remove id(1)

v2

• Remove groups(1)

$ git range-diff shadow/master gh/idgroups idgroups 
1:  b2cec0be = 1:  b2cec0be Remove id(1)
-:  -------- > 2:  451debfe Remove groups(1)

v2b

• Rebase

$ git range-diff master..gh/idgroups shadow/master..idgroups 
1:  b2cec0be ! 1:  61d117c4 Remove id(1)
    @@ src/id.c (deleted)
     -#include <stdio.h>
     -#include <sys/types.h>
     -
    --#include "alloc.h"
    +-#include "alloc/malloc.h"
     -#include "defines.h"
     -
     -/* local function prototypes */
2:  451debfe ! 2:  a539c303 Remove groups(1)
    @@ src/groups.c (deleted)
     -#include <pwd.h>
     -#include <stdio.h>
     -
    --#include "alloc.h"
    +-#include "alloc/x/xmalloc.h"
     -#include "defines.h"
     -#include "prototypes.h"
     -#include "shadowlog.h"

[ikerexxe]

I think we should do an announcement in a release notes indicating that we are deprecating it. Once that is done and nobody has complained we can proceed to remove it.

[alejandro-colomar]

I think we should do an announcement in a release notes indicating that we are deprecating it. Once that is done and nobody has complained we can proceed to remove it.

Okay. I'll then prepare the patch for groups(1) already, and will make this a draft, ready for when the announcement is done.