[1003.1(2016/18)/Issue7+TC2 0001743]: mailx: revert faulty change

[Austin Group Bug Tracker via austin-group-l at The Open Group]

A NOTE has been added to this issue. 
====================================================================== 
https://austingroupbugs.net/view.php?id=1743 
====================================================================== 
Reported By:                steffen
Assigned To:                
====================================================================== 
Project:                    1003.1(2016/18)/Issue7+TC2
Issue ID:                   1743
Category:                   Shell and Utilities
Type:                       Error
Severity:                   Editorial
Priority:                   normal
Status:                     New
Name:                       steffen 
Organization:                
User Reference:              
Section:                    mailx 
Page Number:                3085, 3087 
Line Number:                103783, 103869 
Interp Status:              --- 
Final Accepted Text:         
====================================================================== 
Date Submitted:             2023-06-08 20:48 UTC
Last Modified:              2023-06-09 13:17 UTC
====================================================================== 
Summary:                    mailx: revert faulty change
====================================================================== 

---------------------------------------------------------------------- 
 (0006317) steffen (reporter) - 2023-06-09 13:17
 https://austingroupbugs.net/view.php?id=1743#c6317 
---------------------------------------------------------------------- 
One miss in the above (page 3093, line 104069); and better yet, instead of
the above.

On page 3085, line 103783 ff., change

  Suppress the deletion of the user’s login name and any alternative
addresses from the recipient list when replying to a message or sending to
a group. The default shall be nometoo.

to

  Suppress the deletion of the user’s login name from the recipient list
when replying to a message or sending to a group. The default shall be
nometoo.

On page 3087, line 103869 ff., change

  Declare a list of alternative addresses for the address consisting of
the
user’s login name. When responding to a message or sending to a group,
if
the metoo variable is unset these alternative addresses shall be removed
from the list of recipients. The comparison of addresses shall be
performed
in a case-insensitive manner. With no arguments, alternates shall write
the
current list of alternative addresses.

to

  Declare a list of alternative addresses for the address consisting of
the
user’s login name. When responding to a message these alternative
addresses shall be removed from the list of recipients. The comparison of
addresses shall be performed in a case-insensitive manner. With no
arguments, alternates shall write the current list of alternative
addresses.

On page 3093, line 104069 ff., change

  If the metoo variable is unset, addresses consisting of the login name
  of the user and any alternative addresses declared using the
  alternates command shall be removed from the list.

to

  If the metoo variable is unset, addresses consisting of the login name
  of the user shall be removed from the list.
However.  While you ;) are at it.  The never documented full truth was
as shown by the code below, therefore,
on page 3087, line 103862 ff., insert after

  [.] for example, when hlj is an alias, h...@posix.com does not trigger
  the alias substitution.

the sentences

  An alias that expands only to itself is substituted regardless of the
  setting of metoo.
  Recursive expansion of an alias group member can be prevented by
  prefixing it with an unquoted <backslash>.

==

V10, CSRG (4.4BSD):
        metoo = (value("metoo") != NOSTR);
        while (np != NIL) {
                if (np->n_name[0] == '\\') {
                        cp = np->n_flink;
                        new = put(new, np);
                        np = cp;
                        continue;
                }
                gh = findgroup(np->n_name);
                cp = np->n_flink;
                if (gh != NOGRP)
                        new = gexpand(new, gh, metoo, np->n_type);
                else
                        new = put(new, np);
                np = cp;

Apple (10.10, 13.2), OpenBSD, FreeBSD, etc are identical but replace
"NOSTR" and "NOGRP" with NULL.

findgroup() then iterates over g[roup]h[ead] entries gp

                cp = gp->ge_name;
                if (*cp == '\\')
                        goto quote;
                if (strcmp(cp, gh->g_name) == 0)
                        goto quote;
                if ((ngh = findgroup(cp)) != NOGRP) {
                        nlist = gexpand(nlist, ngh, metoo, ntype);
                        continue;
                }
  quote:
                np = nalloc(cp, ntype);
                /*
                 * At this point should allow to expand
                 * to self if only person in group
                 */
                if (gp == gh->g_list && gp->ge_link == NOGE)
                        goto skip;
                if (!metoo && strcmp(cp, myname) == 0)
                        np->n_type |= GDEL;
  skip:
                nlist = put(nlist, np); 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2023-06-08 20:48 steffen        New Issue                                    
2023-06-08 20:48 steffen        Name                      => steffen         
2023-06-08 20:48 steffen        Section                   => mailx           
2023-06-08 20:48 steffen        Page Number               => 3085, 3087      
2023-06-08 20:48 steffen        Line Number               => 103783, 103869  
2023-06-09 13:17 steffen        Note Added: 0006317                          
======================================================================