From: Sebastian Pipping <sping@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Cc: Stanislaw Pusep <creaktive@gmail.com>
Subject: [gentoo-dev] The feature patch mess in the webalizer ebuild (and how to deal with it)
Date: Wed, 10 Mar 2010 03:43:39 +0100 [thread overview]
Message-ID: <4B97075B.8090605@gentoo.org> (raw)
Hello!
Intro
=====
The latest version of webalizer is 2.01 while upstream is at 2.21 by
now. The reason it's not been bumped in Gentoo is the two use flags
"geoip" and "xtended".
The former alone pulls in a feature patch called "geolizer", while the
latter pulls a feature patch called "webalizer xtended", either with or
without geoip support.
Problem
=======
Both of these patches have not been updated for quite some time.
To bump webalizer we would either need fresh version of both patches
or drop what's out of date which breaks the use flags.
geoip support is included in webalizer itself by now. Still, geoalizer
added a bit more than that: human-readable byte units might still be
missing. With weblizer xtended if may be similar, at least that's my guess.
The real problem is that we have an ebuild depending on feature patches.
Even if we had updated patches by now it were a question of time only
until the symptoms return.
Solution
========
1) Add two new packages to the tree:
- app-admin/geolizer (/usr/bin/geolizer)
- app-admin/webalizer-xtended (/usr/bin/webalizer-xtended)
2) Bump webalizer to 2.21 while
- no longer applying either feature patch
- removing use flag "xtended"
- keeping now hollow use flag "geoip" to reduce breakage
3) Close related bug 231859
https://bugs.gentoo.org/show_bug.cgi?id=231859
I volunteer to do that.
Any objections or suggestions?
Side note
=========
If anyone feels like playing around with merging changes from webalizer
into geolizer (and maybe geolizer-based webalizer xtended) he is welcome
to do that on top a dedicated Git repository for it:
http://git.goodpoint.de/?p=webalizer.git;a=summary
Contact me if that sounds like a job for you. Geolizer upstream is more
likely to migrate anytime soon, so checking back with Stanislaw Pusep
again before would be good.
Sebastian
next reply other threads:[~2010-03-10 2:44 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-10 2:43 Sebastian Pipping [this message]
2010-03-09 22:33 ` [gentoo-dev] The feature patch mess in the webalizer ebuild (and how to deal with it) Alistair Bush
2010-03-10 6:57 ` Benedikt Böhm
2010-03-10 17:21 ` Sebastian Pipping
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=4B97075B.8090605@gentoo.org \
--to=sping@gentoo.org \
--cc=creaktive@gmail.com \
--cc=gentoo-dev@lists.gentoo.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox