COIN-OR::LEMON - Graph Library

Opened 10 years ago

Closed 10 years ago

#198 closed enhancement (wontfix)

Building only the tutorial requires LEMON to be installed

Reported by: Akos Ladanyi Owned by: Alpar Juttner
Priority: minor Milestone:
Component: documentation Version: other
Keywords: Cc:
Revision id:

Description

Currently in the tutorial tree the configure script calls pkg-config. In case you do not have LEMON installed, you will get error messages when running configure.

The attached patch modifies Makefile.in and configure so that the Makefile will call pkg-config and only in case the demo programs are built (and not when the documentation is generated).

Attachments (1)

pkg-config.patch (1.3 KB) - added by Akos Ladanyi 10 years ago.
484fff1afecd

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by Akos Ladanyi

Attachment: pkg-config.patch added

comment:1 in reply to:  description ; Changed 10 years ago by Alpar Juttner

Replying to ladanyi:

The attached patch modifies Makefile.in and configure so that the Makefile will call pkg-config and only in case the demo programs are built (and not when the documentation is generated).

I don't like this change too much. Assume your lemon is installed to a non-standard place. Currently, it's enough to tell its location only once (to the configure script) like this:

PKG_CONFIG_PATH=/path/to/lemon:$PKG_CONFIG_PATH ./configure

then a simple

make demos

command will work fine. Using the attached path, one should type

PKG_CONFIG_PATH=/path/to/lemon:$PKG_CONFIG_PATH make demos

all the time.

Btw. what is the problem with the current behavior? Is it a too strong assumption that we have a LEMON installed somewhere?

comment:2 in reply to:  1 Changed 10 years ago by Akos Ladanyi

Replying to alpar:

Btw. what is the problem with the current behavior? Is it a too strong assumption that we have a LEMON installed somewhere?

Nothing serious. I was writing that script which automatically builds the tutorial and did not like the pkg-config error messages and thought I could improve the situation.

Now I see my approach has drawbacks too, so feel free to close this as 'wontfix'.

comment:3 Changed 10 years ago by Alpar Juttner

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.