public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
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



             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