Summary: | Patch for Bind 9.4.x to allow use of Maxmind's free GeoIPCityLite | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Michael Moody <michael> |
Component: | New packages | Assignee: | BIND Maintainers (DISABLED) <bind+disabled> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | michael, mike, walter |
Priority: | Highest | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://www.caraytech.com/geodns/ | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Patch against Bind 9.4.2 for GeoDNS
INSTALL for BIND patch README for patch Patch against 9.4.2-P1 Updated patch for 9.4.3-P1 INSTALL for 9.4.3-P1 README for 9.4.3-P1 Changelog for 9.4.3-P1 Patch Updated patch for 9.4.3-P1 patch to bind 9.4.3-P1 for region_* and country_* GeoDNS support Patch to bind-9.4.3_p1.ebuild to add geodns support Updated README corrected region_* support corrected region_* support |
Description
Michael Moody
2008-07-08 23:53:52 UTC
Created attachment 159917 [details, diff]
Patch against Bind 9.4.2 for GeoDNS
Created attachment 159919 [details]
INSTALL for BIND patch
Created attachment 159921 [details]
README for patch
yes, please add ASAP! Created attachment 160153 [details]
Patch against 9.4.2-P1
This is a patch against 9.4.2-P1, which is now the latest. I have cleaned up the readme/install instructions, and have added Gentoo-specific instructions.
Someday, it would be great to see this as a USE flag against the bind ebuild.
Michael
(In reply to comment #5) > Created an attachment (id=160153) [edit] > Patch against 9.4.2-P1 > > This is a patch against 9.4.2-P1, which is now the latest. I have cleaned up > the readme/install instructions, and have added Gentoo-specific instructions. Please don't add archives, it's much easier to have a quick look at plaintext files :) > Please don't add archives, it's much easier to have a quick look at plaintext
> files :)
But not as easy as a working ebuild! :)
Created attachment 178544 [details, diff]
Updated patch for 9.4.3-P1
This is an updated patch for 9.4.3-P1. Nothing changed except clean patching against 9.4.3-P1.
Michael
Created attachment 178547 [details]
INSTALL for 9.4.3-P1
Updated INSTALL with cleaner instructions for 9.4.3-P1
Created attachment 178549 [details]
README for 9.4.3-P1
Created attachment 178551 [details]
Changelog for 9.4.3-P1 Patch
(In reply to comment #8) > Created an attachment (id=178544) [edit] > Updated patch for 9.4.3-P1 > > This is an updated patch for 9.4.3-P1. Nothing changed except clean patching > against 9.4.3-P1. > > Michael > I forgot to mention, that the cflags in the instructions have been updated to contain "-g -O2" as well. Michael Created attachment 178552 [details, diff]
Updated patch for 9.4.3-P1
Original uploaded 9.4.3-P1 patch had problems with windows/linux newlines.
Michael
Patch, ebuild diff, and readme, for GeoLite Country, and GeoLite City, in bind coming next. I've been running GeoLite Country now for 2 days, at hundreds of requests per second, with no noticeable performance or load issues. Created attachment 179167 [details, diff]
patch to bind 9.4.3-P1 for region_* and country_* GeoDNS support
Created attachment 179168 [details, diff]
Patch to bind-9.4.3_p1.ebuild to add geodns support
Created attachment 179170 [details]
Updated README
(In reply to comment #14) > Patch, ebuild diff, and readme, for GeoLite Country, and GeoLite City, in bind > coming next. > I've been running GeoLite Country now for 2 days, at hundreds of requests per > second, with no noticeable performance or load issues. > Well go ahead and show me up then :) One question however: Have you tested region? I see you changed the following - + } else if ((0 == (strncmp("region_", name, 8))) && (10 == strlen(name))) { + /* It is a region code */ + de->type = dns_aclelementtype_ipregion; + de->u.region[0] = name[8]; + de->u.region[1] = name[9]; + de->u.region[2] = '\0'; used to be - + } else if ((0 == (strncmp("region_", name, 7))) && (9 == strlen(name))) { + /* It is a region code */ + de->type = dns_aclelementtype_ipregion; + de->u.region[0] = name[7]; + de->u.region[1] = name[8]; + de->u.region[2] = '\0'; Other than that, really, thanks for the ebuild! Michael (In reply to comment #18) > Well go ahead and show me up then :) Heh, sorry, we've got a large project that hinges around country base Geo distribution :) > One question however: > > Have you tested region? I see you changed the following - You know what. I haven't! Pretty obvious what I did to mess up your changes too. > + } else if ((0 == (strncmp("region_", name, 8))) && (10 > used to be - > + } else if ((0 == (strncmp("region_", name, 7))) && (9 > == strlen(name))) { > Other than that, really, thanks for the ebuild! > > Michael No probs, I've got 3 more machines to roll this out on to, so the ebuild helps me significantly. Mike Created attachment 179186 [details, diff]
corrected region_* support
Mike, I suspect that you marked as obsolete the wrong patch: bind-9.4.3_p1-geodns.patch is not corrected, and geodns-bind-9.4.3_p1.ebuild.patch was the most recent upload. Michael Comment on attachment 179186 [details, diff]
corrected region_* support
duplicate ebuild patch
Created attachment 179323 [details, diff]
corrected region_* support
Oops, I should have uploaded the updated bind patch to fix region_* acl support, not the ebuild patch again!
bind-9.6.1_p3-r1 got a geoip patch http://code.google.com/p/bind-geoip/ I'll close this bug in case you're ok with it otherwise let me know if and esp. why not. |