Reviewing merge request #2: autotools build system

Hey!

I added a simple autotools build system for the lib.
Bulding the lib is now as easy as this:
./buildconf
./configure && make install

*updated merge request to include small change: main.cpp is now compiled as a separate binary called 'iz_test'.

Commits that would be merged:

Version 3
  • Version 1
  • Version 2
  • Version 3
  • c973d60
  • 13a5f3e

    • add autotools-based build system

  • cadcb1c

    • compile main.cpp under the name of iz_test

  • a03eca6

    • add missing optimization flags

Showing c973d60-13a5f3e

Comments

Pushed new version 1

avatar Antony Dovgal | 2012-01-31 20:02:48 UTC

Pushed new version 2

avatar Antony Dovgal | 2012-01-31 20:32:03 UTC

Thanks, I am not really fluent with autotools, but from quickly looking at it, it does not request the required optimization options (see the .pro file). Can those be added somewhere?

I am still struggling with gitorious, but once I find out, I can start merging stuff :)

avatar Christoph Feck | 2012-02-01 01:32:50 UTC

Pushed new version 3

avatar Antony Dovgal | 2012-02-01 08:56:18 UTC

Indeed, I missed that file completely.
Added the flags to non-debug build CXX flags and updated the merge request.

avatar Antony Dovgal | 2012-02-01 08:57:01 UTC

→ State changed from Open to Closed

Thanks, merged.

avatar Christoph Feck | 2012-02-02 12:53:09 UTC

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository