Skip to content

GNU autotools based build - #8

Open
wookietreiber wants to merge 5 commits into
zhaoyanswill:masterfrom
idiv-biodiversity:wip-gnu-autotools-build
Open

GNU autotools based build#8
wookietreiber wants to merge 5 commits into
zhaoyanswill:masterfrom
idiv-biodiversity:wip-gnu-autotools-build

Conversation

@wookietreiber

Copy link
Copy Markdown
  • builds/install with the standard procedure:

    autoreconf -i
    ./configure
    make
    make install
    
  • required autoconf version may be lowered (this is just the one I am
    using), needs to be tested, though

  • removes all included boost files, these need not to be redistributed,
    but can / should be provided by the system (boost is pretty portable
    and provided by the distros)

  • includes the appropriate checks, boost libs are missing appropriate
    functions to check

- builds/install with the standard procedure:

    autoreconf -i
    ./configure
    make
    make install

- required autoconf version may be lowered (this is just the one I am
  using), needs to be tested, though

- removes all included boost files, these need not to be redistributed,
  but can / should be provided by the system (boost is pretty portable
  and provided by the distros)

- includes the appropriate checks, boost libs are missing appropriate
  functions to check
@wookietreiber

Copy link
Copy Markdown
Author

This is also an opportunity to easily fix #3, since configure can pick up the external boost deps easily.

@zhaoyanswill

Copy link
Copy Markdown
Owner

Thanks, Christian, I've been planing to do this for a while!

Sincerely,
Y. Z.

On 09/24/2014 07:23 AM, Christian Krause wrote:

This is also an opportunity to easily fix #3
#3, since configure
can pick up the external boost deps easily.


Reply to this email directly or view it on GitHub
#8 (comment).

@wookietreiber

Copy link
Copy Markdown
Author

I just merged in your latest changes.

Can you merge this PR or is there anything staying in the way of it?

@wookietreiber

Copy link
Copy Markdown
Author

This has been laying around for quite a while now. Is there any feedback?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants