Re: Please review fluxter

[Michael-John Turner <mj%turner.org.za@localhost>]

On Mon, Mar 20, 2006 at 09:24:09AM -0800, Min Sik Kim wrote:
> - The correct variable name is "USE_LANGUAGES", not "USE_LANGUAGE".
Fixed

> - bsd.prefs.mk included in Makefile doesn't seem to be necessary.
Fixed

> - Shouldn't this package depend on wm/fluxbox?

AFAIK it doesn't require fluxbox (in much the same way as wm/bbpager works
with both blackbox and fluxbox and a number of window managers). I've left
out the dependency for this reason.

Thanks for the suggestions - I've committed the changes.

-mj
-- 
Michael-John Turner | http://weblogs.turner.org.za/mj/
mj%turner.org.za@localhost    | Open Source in WC ZA - http://www.clug.org.za/


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
pkgsrc-wip-review mailing list
pkgsrc-wip-review%lists.sourceforge.net@localhost
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-review