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