Opened 16 years ago
Closed 16 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)
Change History (4)
Changed 16 years ago by
Attachment: | pkg-config.patch added |
---|
comment:1 follow-up: 2 Changed 16 years ago by
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 Changed 16 years ago by
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 16 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
484fff1afecd