On Mon, Feb 10, 2014 at 11:05 AM, Alan Mizrahi <alan+crux@mizrahi.com.ve>wrote:
Normally the configure script would detect and enable features if available, this should be reported as a bug upstream.
I completely agree. Whoever "cares" enough about this should report a bug upstream! This is kind of not our (CRUX) problem :)
You could also detect if aspell is installed or not, and provide --disable- aspell if its not, for example:
... build() { # Detect if aspell is available if [ -f /usr/include/aspell.h ]; then aspell=y else aspell=n fi
cd $name-$version ./configure \ --prefix=/usr \ "$([ $aspell = y ] || echo --disable-aspell)" ...
I don't personally like this as a solution because it overcomplicates the resulting Pkgfile and creates more work and effort for Maintainers... However kudos on your Bash skills :)
I noticed you forcefully disable other things that are correctly detected and enabled by the configure script, such as meanwhile. This is in fact the reason why I created my own port for pidgin.
Agreed. IHMO Maintainers of Ports __should not__ deliberately disable optional features that are automatically picked up by the respective build system (e.g: autoconf in this case) unless they __have a very good reason__. If I were you I'd ask for this "forced" --disable-meanwhile to be removed from the pidgin port :) But you'd have to file a issue on http://crux.nu/bugs/ or Email/Chat toe the Maintainer :) cheers James James Mills / prologic E: prologic@shortcircuit.net.au W: prologic.shortcircuit.net.au