This version eliminates the unless statement, but is basically the same as your example (you should use strict). I'm running this on a Solaris 8.0 system but I don't get any duplicates.
use strict; my @sec_gp; my $name = "xxxxx"; while (my ($gname,$gpasswd,$gid,$members)=getgrent) { my @list_members=split (/\s+/,$members); my @list=grep (/$name/,@list_members); if ($list[0] eq $name) { push (@sec_gp,"$gname"); } } print "\@sec_gp: "; print "$_\n" foreach @sec_gp;
Output:
@sec_gp: acad libsoftwrite helpdesk codes

--Jim Update: The above was hastily done. Better is the following (still room for improvement I'm sure):

push(@sec_gp, "$gname") if $list[0] eq $name;
for the original if statement.

In reply to Re: To get the secondary groups for linux by jlongino
in thread To get the secondary groups for linux by tbo

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":



  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.