[Dune] [#861] duneproject: Generates invalid Makefile.am if project name has more than two dashes

[Dune]

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

A new Flyspray task has been opened.  Details are below. 

User who did this - Ansgar Burchardt (ansgar) 

Attached to Project - Dune
Summary - duneproject: Generates invalid Makefile.am if project name has more than two dashes
Task Type - Bug Report
Category - Common
Status - Unconfirmed
Assigned To - 
Operating System - Unspecified / All
Severity - Very Low
Priority - Normal
Reported Version - SVN (pre2.1)
Due in Version - Undecided
Due Date - Undecided
Details - duneproject generates a Makefile.am if the project name has more than two dashes in it.

Example session:
% ./dune-common/bin/duneproject
[...]
1) Name of your new Project? (e.g.: dune-grid): dune-a-b-c
2) Which modules should this module depend on?
   Following modules are found:
   dune-common dune-pdelab dune-grid dune-istl dune-localfunctions dune-grid-dev-howto dune-grid-howto dune-pdelab-howto 
   Enter space separated list: dune-common
3) Project/Module version? 0.1
4) Maintainers email address? me at example.com

creating Project "dune-a-b-c", version 0.1 
[...]
writing initial files:
- dune-a-b-c/configure.ac
[... These are the problematic files: ]
- dune-a-b-c/dune/a_b-c
- dune-a-b-c/dune/Makefile.am
- dune-a-b-c/dune/a_b-c/Makefile.am
- dune-a-b-c/dune/a_b-c/a_b-c.hh
[...]

Running `dunecontrol --only=dune-a-b-c all' gives this error:
dune/a_b-c/Makefile.am:1: bad characters in variable name `a_b-cincludedir'
dune/a_b-c/Makefile.am:2: bad characters in variable name `a_b-cinclude_HEADERS'

The bug is present in both 2.0 and the current SVN trunk (r6303).

A simple solution would be to just convert all minuses "-" to an underscore "_".

Regards,
Ansgar


More information can be found at the following URL:
http://www.dune-project.org/flyspray/index.php?do=details&task_id=861

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.