Bug 123267 - [UPDATE] www/mod_geoip2 to 1.2.3
Summary: [UPDATE] www/mod_geoip2 to 1.2.3
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Rong-En Fan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-30 16:30 UTC by Yarodin
Modified: 2008-05-04 04:20 UTC (History)
0 users

See Also:

Attachments
file.diff (1006 bytes, patch)
2008-04-30 16:30 UTC, Yarodin 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yarodin 2008-04-30 16:30:01 UTC 
Update to 1.2.3

- Skip files that we can not open for some reason ( permission/typo ). Check and skip empty slots in geoip_header_parser
- Fix race condition for IO based caches like Standard. MMapCache and MemoryCache are not affected.
- Slight performance increase - don't make per_dir ap_get_module_config call where we return DECLINED because cfg->GeoIPEnabled is true
- NetWare/Win32 compilation fixes, since can't declare vars in middle of code

Fix: How to apply:
1) Put patch-mod_geoip2.txt to /usr/ports/www/
2) cd /usr/ports/www/ && patch -E -p <patch-mod_geoip2.txt && find /usr/ports/www/mod_geoip2/ -type f -name "*.orig" -exec rm -f {} \; 

Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-04-30 16:30:16 UTC 
Maintainer of www/mod_geoip2,

Please note that PR ports/123267 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/123267

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2008-04-30 16:30:20 UTC 
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 3 jau 2008-05-03 06:12:15 UTC 
	Accepted.


	Cheers,
		// jau
.---  ..-  -.-  -.-  .-    .-  .-.-.-    ..-  -.-  -.-  ---  -.  .  -.
  /    Jukka A. Ukkonen,                             Oxit Ltd, Finland
 /__   M.Sc. (sw-eng & cs)                    (Phone) +358-500-606-671
   /   Internet: Jukka.Ukkonen(a)Oxit.Fi        (Home) +358-9-6215-280
  /    Internet: jau(a)iki.fi
 v
        .---  .-  ..-  ...-.-  ..  -.-  ..  .-.-.-  ..-.  ..
+ + + + My opinions are mine and mine alone, not my employers. + + + +
Comment 4 Mark Linimon freebsd_committer freebsd_triage 2008-05-03 06:47:59 UTC 
State Changed
From-To: feedback->open

Maintainer approved.
Comment 5 Rong-En Fan freebsd_committer freebsd_triage 2008-05-03 10:05:26 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->rafan

I'll take it.
Comment 6 Rong-En Fan freebsd_committer freebsd_triage 2008-05-04 04:18:16 UTC 
State Changed
From-To: open->closed

Committed. Thanks!
Comment 7 dfilter service freebsd_committer freebsd_triage 2008-05-04 04:18:24 UTC 
rafan       2008-05-04 03:18:16 UTC

  FreeBSD ports repository

  Modified files:
    www/mod_geoip2       Makefile distinfo 
  Log:
  - Update to 1.2.3
  
  PR:             ports/123267
  Submitted by:   Yarodin <yarodin at gmail.com>
  Approved by:    Jukka A. Ukkonen <jau at iki.fi> (maintainer)
  
  Revision  Changes    Path
  1.9       +1 -1      ports/www/mod_geoip2/Makefile
  1.7       +3 -3      ports/www/mod_geoip2/distinfo
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"