Re: [PATCH] Package content search and listing functionality for cygcheck

[David Rothenberger]

Corinna Vinschen wrote:
> Thanks for the patch, it's really cool,

I agree, very cool, Igor.

Any chance the return in package_find() could be changed to
continue?  I went to try it out for /bin/ssh and found it didn't
work because diffutils is missing the package list.  I didn't think
to even try verbose until I read the code.

Here's the ridiculously small patch if you agree.

Dave
==============================
2003-08-15  David Rothenberger  <[EMAIL PROTECTED]>

        * dump_setup.cc (package_find): Don't stop searching on missing
        file list.


Index: dump_setup.cc
===================================================================
RCS file: /cvs/src/src/winsup/utils/dump_setup.cc,v
retrieving revision 1.10
diff -u -p -r1.10 dump_setup.cc
--- dump_setup.cc       15 Aug 2003 20:26:11 -0000      1.10
+++ dump_setup.cc       16 Aug 2003 02:03:17 -0000
@@ -454,7 +454,7 @@ package_find (int verbose, char **argv)
        if (verbose)
          printf ("Can't open file list /etc/setup/%s.lst.gz for package %s\n",
              packages[i].name, packages[i].name);
-       return;
+       continue;
       }
 
       char buf[MAX_PATH + 2];