All of lore.kernel.org
 help / color / mirror / Atom feed
From: akuster808 <akuster808@gmail.com>
To: Martin Jansa <martin.jansa@gmail.com>, Khem Raj <raj.khem@gmail.com>
Cc: openembedded-devel <openembedded-devel@lists.openembedded.org>
Subject: Re: [meta-networking][PATCH] geoip: add downloadfilename for geoip database files
Date: Tue, 27 Nov 2018 12:53:49 -0700	[thread overview]
Message-ID: <3c4b464d-b2f5-2f48-b48a-e84ad66f7940@gmail.com> (raw)
In-Reply-To: <CA+chaQeD+MHPAnK646z0kbT6GzQ7-fBoG_4YPfx6Hp=VJg9WwA@mail.gmail.com>

I am seeing failures on some of the stable branches.

The sources change every Tuesday of the month so we will be chasing this
monthly on top of these other dl issues.

- armin


On 11/27/18 12:14 PM, Martin Jansa wrote:
> I'm seeing different checksum than both your builder and the recipe.
>
> $ wget http://geolite.maxmind.com/download/geoip/database/GeoIPv6.dat.gz
> --2018-11-27 20:06:32--
> http://geolite.maxmind.com/download/geoip/database/GeoIPv6.dat.gz
> Resolving geolite.maxmind.com... 104.16.37.47, 104.16.38.47,
> 2606:4700::6810:252f, ...
> Connecting to geolite.maxmind.com|104.16.37.47|:80... connected.
> HTTP request sent, awaiting response... 200 OK
> Length: 1209046 (1.2M) [application/octet-stream]
> Saving to: ‘GeoIPv6.dat.gz’
>
> 2018-11-27 20:06:32 (10.3 MB/s) - ‘GeoIPv6.dat.gz’ saved [1209046/1209046]
>
> $ md5sum GeoIPv6.dat.gz
> d98a04eded0b08b945b5629106d58760  GeoIPv6.dat.gz
>
> Is this the legacy archive? Because
> https://dev.maxmind.com/geoip/legacy/geolite/ now shows following:
>
> Updated versions of the GeoLite Legacy databases are now only available to
> redistribution license customers, although anyone can continue to download
> the March 2018 GeoLite Legacy builds. Starting January 2, 2019, the last
> build will be removed from our website. GeoLite Legacy database users will
> need to switch to the GeoLite2 or commercial GeoIP databases and update
> their integrations by January 2, 2019.
>
> For more information, please visit our Support Center.
>
> Note: After careful consideration, taking into account customer feedback,
> we have decided against removing latitude and longitude coordinates from
> the GeoLite2 databases. We are in the process of reviewing coordinates used
> in all of our GeoLite2 and GeoIP databases to ensure there is no risk of
> misuse.
>
> * GeoLite2 databases are free IP geolocation databases comparable to, but
> less accurate than, MaxMind’s GeoIP2 databases. Please remember to use the
> accuracy radius if displaying coordinates on a map.
>
> New Database Format Available: This page is for our GeoLite Legacy
> databases. For our latest database format, please visit our GeoLite2
> Databases page. We offer redistribution licenses for our newer GeoLite2
> databases that will allow you to package the databases with your commercial
> products. For more information, please visit the GeoLite2 Commercial
> Redistribution page.
>
>
> On Tue, Nov 27, 2018 at 8:04 PM Khem Raj <raj.khem@gmail.com> wrote:
>
>> I am seeing errors on some builders
>>
>> http://errors.yoctoproject.org/Errors/Details/202341/
>> On Mon, Nov 19, 2018 at 11:29 PM Martin Jansa <martin.jansa@gmail.com>
>> wrote:
>>> Looks, good.
>>>
>>> Adding variable for 20181120 might help with maintenance (if we need to
>>> change just checksums in future). And with good variable name (or a
>>> comment) it would also document in the recipe why we're using this in
>>> downloadfilename.
>>>
>>> On Tue, Nov 20, 2018 at 7:48 AM <changqing.li@windriver.com> wrote:
>>>
>>>> From: Changqing Li <changqing.li@windriver.com>
>>>>
>>>> previous change of checksum don't trigger archive to re-downloaded,
>>>> , which will cause checksum mismatch. add downloadfilename to
>>>> trigger re-download.
>>>>
>>>> 1. for user with PREMIRROR, another benefit is it can still compile
>>>> success event upstream checksum change frequently.
>>>> 2. but for user don't use PREMIRROR, if upstream checksum changed,
>>>> still might have checksum mismatch problem.
>>>>
>>>> Signed-off-by: Changqing Li <changqing.li@windriver.com>
>>>> ---
>>>>  .../recipes-support/geoip/geoip_1.6.12.bb          | 24
>>>> +++++++++++-----------
>>>>  1 file changed, 12 insertions(+), 12 deletions(-)
>>>>
>>>> diff --git a/meta-networking/recipes-support/geoip/geoip_1.6.12.bb
>>>> b/meta-networking/recipes-support/geoip/geoip_1.6.12.bb
>>>> index 1e1a7ff..57f0b67 100644
>>>> --- a/meta-networking/recipes-support/geoip/geoip_1.6.12.bb
>>>> +++ b/meta-networking/recipes-support/geoip/geoip_1.6.12.bb
>>>> @@ -9,10 +9,10 @@ HOMEPAGE = "http://dev.maxmind.com/geoip/"
>>>>  SECTION = "libdevel"
>>>>
>>>>  SRC_URI = "git://github.com/maxmind/geoip-api-c.git \
>>>> -
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoLiteCountry/GeoIP.dat.gz;apply=no;name=GeoIP-dat
>>>> \
>>>> -
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoIPv6.dat.gz;apply=no;name=GeoIPv6-dat
>>>> \
>>>> -
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoLiteCity.dat.gz;apply=no;name=GeoLiteCity-dat
>>>> \
>>>> -
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoLiteCityv6-beta/GeoLiteCityv6.dat.gz;apply=no;name=GeoLiteCityv6-dat
>>>> \
>>>> +
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoLiteCountry/GeoIP.dat.gz;apply=no;name=GeoIP-dat;downloadfilename=GeoIP.dat.20181120.gz
>>>> \
>>>> +
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoIPv6.dat.gz;apply=no;name=GeoIPv6-dat;downloadfilename=GeoIPv6.dat.20181120.gz
>>>> \
>>>> +
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoLiteCity.dat.gz;apply=no;name=GeoLiteCity-dat;downloadfilename=GeoLiteCity.dat.20181120.gz
>>>> \
>>>> +
>>>>
>> http://geolite.maxmind.com/download/geoip/database/GeoLiteCityv6-beta/GeoLiteCityv6.dat.gz;apply=no;name=GeoLiteCityv6-dat;downloadfilename=GeoLiteCityv6.dat.20181120.gz
>>>> \
>>>>             file://run-ptest \
>>>>  "
>>>>  SRCREV = "4b526e7331ca1d692b74a0509ddcc725622ed31a"
>>>> @@ -20,14 +20,14 @@ SRCREV = "4b526e7331ca1d692b74a0509ddcc725622ed31a"
>>>>  SRC_URI[GeoIP-dat.md5sum] = "d538e57ad9268fdc7955c6cf9a37c4a9"
>>>>  SRC_URI[GeoIP-dat.sha256sum] =
>>>> "b9c05eb8bfcf90a6ddfdc6815caf40a8db2710f0ce3dd48fbd6c24d485ae0449"
>>>>
>>>> -SRC_URI[GeoIPv6-dat.md5sum] = "00dc3b3f53eb5f7bf6915f939998f12c"
>>>> -SRC_URI[GeoIPv6-dat.sha256sum] =
>>>> "e72221fc630094d7256521b307e9142fb77d2167a27de2c11da536a2ba1d1480"
>>>> +SRC_URI[GeoIPv6-dat.md5sum] = "a908d7a51c3883eb8f693ca197c7a142"
>>>> +SRC_URI[GeoIPv6-dat.sha256sum] =
>>>> "1b06e34eed2c539606a9ab4d97b2140ac80bf9f528d16ad2e3831c75014c2710"
>>>>
>>>>  SRC_URI[GeoLiteCity-dat.md5sum] = "d700c137232f8e077ac8db8577f699d9"
>>>>  SRC_URI[GeoLiteCity-dat.sha256sum] =
>>>> "90db2e52195e3d1bcdb2c2789209006d09de5c742812dbd9a1b36c12675ec4cd"
>>>>
>>>> -SRC_URI[GeoLiteCityv6-dat.md5sum] = "7f6333d433d8127181ea3c5828287c98"
>>>> -SRC_URI[GeoLiteCityv6-dat.sha256sum] =
>>>> "c339e20cdf326c18aa3571bcd27845e6daf41ca5bfe9401bc472908aaeb42ac7"
>>>> +SRC_URI[GeoLiteCityv6-dat.md5sum] = "8ac3506ccabaae7e9f7d6301c2654313"
>>>> +SRC_URI[GeoLiteCityv6-dat.sha256sum] =
>>>> "1c5fced1657d09448b3ecc14b65c9536228a627d93ed4159d3438b2c6faa8506"
>>>>
>>>>  LICENSE = "LGPL-2.1"
>>>>
>>>> @@ -44,10 +44,10 @@ EXTRA_OECONF = "--disable-static               \
>>>>  do_install() {
>>>>      make DESTDIR=${D} install
>>>>      install -d ${D}/${datadir}/GeoIP
>>>> -    install ${WORKDIR}/GeoIP.dat ${D}/${datadir}/GeoIP/
>>>> -    install ${WORKDIR}/GeoIPv6.dat ${D}/${datadir}/GeoIP/
>>>> -    install ${WORKDIR}/GeoLiteCity.dat ${D}/${datadir}/GeoIP/
>>>> -    install ${WORKDIR}/GeoLiteCityv6.dat ${D}/${datadir}/GeoIP/
>>>> +    install ${WORKDIR}/GeoIP.dat.20181120
>> ${D}/${datadir}/GeoIP/GeoIP.dat
>>>> +    install ${WORKDIR}/GeoIPv6.dat.20181120
>>>> ${D}/${datadir}/GeoIP/GeoIPv6.dat
>>>> +    install ${WORKDIR}/GeoLiteCity.dat.20181120
>>>> ${D}/${datadir}/GeoIP/GeoLiteCity.dat
>>>> +    install ${WORKDIR}/GeoLiteCityv6.dat.20181120
>>>> ${D}/${datadir}/GeoIP/GeoLiteCityv6.dat
>>>>      ln -s GeoLiteCity.dat ${D}${datadir}/GeoIP/GeoIPCity.dat
>>>>  }
>>>>
>>>> --
>>>> 2.7.4
>>>>
>>>> --
>>>> _______________________________________________
>>>> Openembedded-devel mailing list
>>>> Openembedded-devel@lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>>>>
>>> --
>>> _______________________________________________
>>> Openembedded-devel mailing list
>>> Openembedded-devel@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel




  reply	other threads:[~2018-11-27 19:53 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-20  6:48 [meta-networking][PATCH] geoip: add downloadfilename for geoip database files changqing.li
2018-11-20  7:28 ` Martin Jansa
2018-11-27 19:03   ` Khem Raj
2018-11-27 19:14     ` Martin Jansa
2018-11-27 19:53       ` akuster808 [this message]
2018-11-29  1:30       ` Changqing Li
2018-11-29  1:55         ` Khem Raj
2018-11-30  2:18           ` Changqing Li
2018-11-30  8:22             ` Martin Jansa
2018-12-03  2:06               ` Changqing Li
2018-12-03  2:38                 ` Khem Raj
2018-12-03  9:24                   ` Martin Jansa
2018-12-03 16:54                     ` Khem Raj
2018-11-29  6:22         ` Martin Jansa

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3c4b464d-b2f5-2f48-b48a-e84ad66f7940@gmail.com \
    --to=akuster808@gmail.com \
    --cc=martin.jansa@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=raj.khem@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.