incubator-flex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Om <bigosma...@gmail.com>
Subject Re: InstallApacheFlex - single-mirror-url.cgi
Date Thu, 09 Aug 2012 20:09:04 GMT
On Thu, Aug 9, 2012 at 12:59 PM, Erik de Bruin <erik@ixsoftware.nl> wrote:

> > Yes, I see that the html file I added with just " [preferred] " in it has
> > been padded with the standard template html around it.
>
> I've added code to parse the HTML to my most recent patch. The 'single
> word' HTML output is probably easier to parse than the 'raw' output of
> "mirror.cgi", as the URL we're looking for is the only text inside of
> a DIV with the id "content"...
>

The last patch I see is:
InstallApacheFlex_Patch_FLEX-33149_FLEX-33150_2012-08-03.txt

Is this the one you are talking about?  I dint see any code for parsing the
HTML in there.



> Maybe we should ask infra politely if there is a solution that
> prevents the 'frame' HTML from being automatically added, and if not,
> we should think about if we want the GeoIP solution or the HTML
> parsing solution.
>

Nick mentioned that he will work on it today.  I will wait for a day or so
before starting to bug infra.

In my mind: Single-mirror-url solution > HTML parsing solution > GeoIP
solution

In my testing the GeoIP provider sometimes cannot resolve the country code
and just fails.  The HTML parsing solution, even though kludgy, will always
work.


>
> EdB
>
>
>
> --
> Ix Multimedia Software
>
> Jan Luykenstraat 27
> 3521 VB Utrecht
>
> T. 06-51952295
> I. www.ixsoftware.nl
>
>
>
> P.S. I would like everybody to know that as of this morning I broke my
> personal record for days lived.
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message