Make the comments a tiny bit more meaningful.

Add the 'Suggests' field, to make it more obvious that optional
dependencies are possible.

Fixes:
- Use unambiguous truth values (Closes: #134)
- When checking with pkg-config, look for the dependency rather than the last
  module from the list of found modules

Modernizations:
- Scope variables
- Use a simple `if` rather than a loop to check for presence of a depency in
  list of existing modules
- Make sure the function does not depend on the status of the `-e` option, or
  whether it is called from within the condition of an `if` or a `||` or `&&`.

Documentation:
- Document calling convention for the function and assumptions on the
  parameters.

As a shell script, duneproject is not covered by the scripts preventing tabs
in indentation (or simply was exempt from the Great Reformatting).  Also, the
tab width was inconsistent.

This expands all tabs and uses a consistent indentation of 2.  It also
instructs both emacs and vi to use these settings, since their default
indentation for shell scripts seems to differ.

Markus Blatt authored 7 years ago and Christoph Grüninger committed 7 years ago

The script uses dune_enable_all_packages which has issues with older
CMake versions.

* dune-common requires CMake 2.8.12 anyway
* confusing option
* better remove line by hand after module created

The old documentation in buildsystem.pdf is gone.  It has been
replaced by Sphinx-based on on the web.  Refer to that now.

Ansgar Burchardt authored 8 years ago and Christoph Grüninger committed 8 years ago

Remove the `autogen` command from `dunecontrol`.  After the removal of
autotools support, running `dunecontrol autogen` would just return an
error anyway:

```
    $ ./dunecontrol --only=dune-common autogen
    [...]
    ./dune-common/bin/dunecontrol: line 341: run_default_autogen: command not found
    [...]
```

In addition the `dune-autogen` script was already removed in
4935e077.

Elias Pipping authored 9 years ago and Christoph Grüninger committed 9 years ago

On FreeBSD, e.g., bash is installed to /usr/local

The 'macroname' variable is still used by the code, but its definition
was accidentally removed together with the AutoTools-generating code.
Therefore, duneproject would fail with

/home/sander/dune/dune-common/bin/duneproject: Zeile 695: dune-foo/cmake/modules/: Ist ein Verzeichnis

This patch puts the missing definition back in.

dune_enable_all_packages() now requires CMake 2.8.12, so make sure
duneproject generates a correct template.

Also output a warning to tell the user about this change in required
CMake version.

The shell script logic for switching dune_enable_all_packages() on or
off in duneproject was broken and always turned the feature on, ignoring
the user input.

Fixed by testing for the actual contents of the variable instead of its
length and by converting the contents to lowercase before.

Viz., the question on whether all packages should be enabled by default.

This fixes FS 1647.

A typo made it impossible to say 'no'.

Furthermore, I changed the text shown to the user to point to the
module documentation.

Commit 	29405741 accidentally changed the file
permissions to non-executable.

Felix Gruber authored 10 years ago and Christoph Grüninger committed 10 years ago

The install path for header files can be configured in CMake with the
variable ${CMAKE_INSTALL_INCLUDEDIR}.

Felix Gruber authored 10 years ago and Christoph Grüninger committed 10 years ago

Otherwise UNAME_HH was seen by the shell as the name of the variable,
but actually, the variable is called UNAME and _HH should be appended
after expansion of the variable.

Follow up to the previous patch: if the source file contains hyphens, then
the compiled executable should not have them replaced by underscores.

Each module constructed by duneproject contains a little Hello-world example
program.  The name of the program used to be the name of the module with .cc
appended AND all hyphens replaced by underscores.  Hence a module dune-foo
would containt the program dune_foo.cc.  This patch removes that displacement.
Rationale:
- I see no reason for the replacement in the first place
- File names with hyphens appear to be much more common in Dune world than
  file names with underscores.

Felix Gruber authored 10 years ago and Christoph Grüninger committed 10 years ago

add missing @

mention cmake and CMAKE_FLAGS. Add an example for CMAKE_FLAGS in the
example opts file. Also update the compiler version in the example opts
file to a version that is actually supported by dune.

This propbably the best spot to safe the user from himself!