This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] add -p option to cygcheck to query website package search


On Mon, Jun 20, 2005 at 01:04:45PM -0700, Brian Dessent wrote:
>Here is a patch that implements the -p option to cygcheck that was
>mentioned on the list previously.  It uses the WinInet API to hit the
>package-grep.cgi URL on cygwin.com with the search regexp supplied by
>the user.

Five+ months later, I've checked in (most of) this patch to cygcheck.cc
and the documentation.

It seems like the bug fix that was in the patch was no longer needed
since it didn't apply and the code around the fix had changed.

One reason for the delay was that I wanted to do things a little
differently in package-grep.cgi.  I think I've done that, with the help
of HTML::TreeBuilder.  So, there is now a "text" option to this script,
similar to your patch, which will just dump raw text.

Another reason for the delay was that I didn't want to add another way
to overload the old sourceware.org.  However, the new system is
perfectly capable of handling anything we can throw at it for the
foreseeable future, so this seemed like a good time to implement this
functionality.

Other than that, your patch just went in as-is.  I'm going to make a
snapshot with this change in it soon, Brian, so if you have some new
insights after five months, please let me know so that I can incorporate
them.

I apologize for the very long lag time in getting this patch in. I do
appreciate your addition of this functionality to cygcheck.exe.  I hope
that it will be a useful addition to our bag of tech support tricks.

Thanks again.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]