Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 661498 - www-servers/nginx - missing dependency on geoip libraries
Summary: www-servers/nginx - missing dependency on geoip libraries
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Thomas Deutschmann (RETIRED)
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2018-07-18 12:39 UTC by Petr Cerny [:hrosik]
Modified: 2018-07-20 22:45 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
untested patch for ebuild blindly copying dependecies from http modules (nginx-1.15.1.ebuild__geoip_stream_mods.patch,1.14 KB, patch)
2018-07-18 12:39 UTC, Petr Cerny [:hrosik]
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Petr Cerny [:hrosik] 2018-07-18 12:39:38 UTC
Created attachment 539946 [details, diff]
untested patch for ebuild blindly copying dependecies from http modules

www-servers/nginx-1.15.1 currently misses dependency on geoip libraries for the nginx_modules_stream_geo* flags - that means the dependency doesn't get pulled in if stream_geoip is selected but http_geoip is not.

Without looking in deeper, I suppose copying dependencies from the complementary http modules (as in the attached patch) might do the trick.
Comment 1 Larry the Git Cow gentoo-dev 2018-07-20 19:44:15 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0faf2544464e203ea2fd80a14c45cff6f7d8cea5

commit 0faf2544464e203ea2fd80a14c45cff6f7d8cea5
Author:     Thomas Deutschmann <whissi@gentoo.org>
AuthorDate: 2018-07-20 19:41:45 +0000
Commit:     Thomas Deutschmann <whissi@gentoo.org>
CommitDate: 2018-07-20 19:44:06 +0000

    www-servers/nginx: rev bump to fix missing deps for geoip modules
    
    Closes: https://bugs.gentoo.org/661498
    Package-Manager: Portage-2.3.43, Repoman-2.3.10
    RepoMan-Options: --force

 www-servers/nginx/{nginx-1.14.0-r2.ebuild => nginx-1.14.0-r3.ebuild} | 4 +++-
 www-servers/nginx/{nginx-1.15.1.ebuild => nginx-1.15.1-r1.ebuild}    | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)
Comment 2 Thomas Deutschmann (RETIRED) gentoo-dev 2018-07-20 19:46:53 UTC
Thanks for the report!