I followed these OSX instructions on a new iMac (OSX 10.10.1) (https://github.com/e14n/pump.io/wiki/HowTo-for-building-Pumpa-on-OS-X))

and encountered the following build error:

In file included from src/main.cpp:25:
In file included from src/pumpapp.h:52:
In file included from src/messagewindow.h:35:
In file included from src/messageedit.h:29:
src/qaspell.h:32:10: fatal error: 'aspell.h' file not found
#include <aspell.h>
                   ^
1 error generated.
make: *** [obj/main.o] Error 1

homebrew installs aspell.h into /usr/local/include.

I edited the pumpa Makefile and added

-I/usr/local/include

to the "INCPATH" and executed a successful build.

Perhaps adding /usr/local/include to the pumpa Makefile INCPATH a good enough solution.

I'm unsure about this. If you are using homebrew and expect to compile programs against it you could also have the include path in a global setting, e.g. by adding this to your .bashrc:

export C_INCLUDE_PATH=/usr/local/include
Comment by sazius [id.mayfirst.org] Tue Dec 23 13:24:18 2014

Sazius, I just tried rebuilding with C_INCLUDE_PATH=/usr/local/include in the environment and had the same error:

src/qaspell.h:32:10: fatal error: 'aspell.h' file not found
#include <aspell.h>
              ^
1 error generated.
make: *** [obj/main.o] Error 1

Maybe include C_INCLUDE_PATH in INCPATH in Makefile?

Comment by Bill Tue Dec 23 14:56:01 2014

Hmm, strange what about using CPLUS_INCLUDE_PATH instead? :-)

The thing is I'd rather not add include paths for every possibility. If you compile something in an environment it should already be configured so that the C++ compiler will find the headers needed. It shouldn't be the responsibility of each application to do that.

Sorry for being difficult :-)

Comment by sazius [id.mayfirst.org] Tue Dec 23 16:32:00 2014

First, you are not being difficult. Less code == less maintenance.

Second, thanks for the suggestion; CPLUS_INCLUDE_PATH worked. I have been away from C/C++ programming for some time.

I will work on getting the pump.io/wiki page updated to include specifying this environment variable.

Thanks, and have a great new year.

Bill A

Comment by Bill Tue Dec 23 17:56:31 2014
Great! Thanks, and have a great holiday season too!
Comment by sazius [id.mayfirst.org] Tue Dec 23 20:46:33 2014