Autotools

[gbonnema]

This patch includes the new versions of Makefile.am and configure.ac. The changes are non-trivial and represent a lot of grinding over de code. Also, I had to take some decisions, that we may need to revert or change later on. Below a summary of what the patch contains. I have the changes in detail summarized in a file called changelog-autotools.md. The git log and diff are of course the most complete and detailed changes.

• Replaced Makefile.in (handcrafted) by Makefile.am and renamed the old Makefile.in to Makefile.in.old
• Replace configure.in to configure.ac and renamed the old configure.in to configure.in.old
• Did a detailed scrutiny of all defines that were in the old configure and Makefile and changed the code appropriately. Also adapted the new configure.ac and Makefile.am to these insights.
• Created a subdirectory structure where src is the main subdirectory.
• Created a man page subdirectory (man).
• Started using libtool for portable library handling.
• Separated installation of xwpe and xwe from wpe and we (solves #18).

Related issues are #57, #56, #55, #52, #18.

• 57 Mistyped define for NCURSES, is a major headache that we can solve later on, but until then even in the face of ncurses being present, we still use curses at best.

• 56 Mistyped define for RANLIB is solved by using libtool and defining RANLIB separately in Makefile.am

• 55 LIBRARY_DIR hard coded, was solved by checking that the define is in the makefile during precompilation. No hard coding needed.

• 52 configure.in (using autoconf) seems to not work properly, is no longer an issue using the new versions of Makefile.am and configure.ac.

• 18 "make install" installs X versions, even if not compiled in, was solved using standard Makefile.am features.

EDIT: I also managed to eliminate the warnings that Travis shows. My own compiles do not show these warnings, even if I explicitly request these specific warnings. I tested -Wunused-result which is default for gcc-6 and should always show, but they don't.

I hope this summary gives enough information for you to know what was done in the commits of this patch.

Kind regards, Guus.

[gbonnema]

Hey Alessandro,

The weird thing is, that the travis tests show warnings that I do not get from my Fedora system, even though we both use gcc-6. The compile statement mentions no warnings. Are the defaults for warnings in ubuntu different from Fedora? Should we be explicit about what warnings to provide?

Kind regards, Guus

[gbonnema]

Hi Alessandro,

I finished everything I wanted to do on this branch. If you are swamped (many people are), then I will just merge them by the end of tomorrow.

In the meantime I will work on getting ncurses working properly (see #57 for the issue) and if I do before tomorrow night, I will commit it to this branch (autotools) before merging it.

Remark: I am merging straight into experimental, so I will not create a separate branch for these commits on your repo.

[gbonnema]

Hi Alessandro,

This morning I successfully reinstated the full NCURSES functionality (see #57). I also saw that CLANG gives warnings by default that gcc generates for -Wall. So in order to progress in eliminating warnings, I will merge this branch and start a new one for solving the warnings.

Nonetheless, if you have time to checkout the changes sofar and have a look at them, please do.

Kind regards, Guus.

[gbonnema]

This pull request fixes #51, fixes #52, fixes #53, fixes #55 and fixes #57.