ADMConstraints param.ccl fails to parse

Create issue
Issue #2028 closed
Ian Hinder created an issue

arrangements/EinsteinAnalysis/ADMConstraints/param.ccl fails to parse with the new parser, giving the error

CST ERROR IN FILE '/home/ianhin/Cactus/EinsteinToolkitGit/arrangements/EinsteinAnalysis/ADMConstraints/param.ccl' ERROR ON LINE 70:
string excision_mask_name "The name of the bitfield chosen for excision"
  :: "Anything goes"
  | here
EXPECTED CHARACTER(S): DOUBLE_QUOTE, 'A' to 'Z', '\', '_', 'a' to 'z', '}'
Parse Error at /home/ianhin/Cactus/EinsteinToolkitGit/lib/sbin/ line 51.
    main::create_parameter_database('TwoPunctures', '/home/ianhin/Cactus/EinsteinToolkitGit/arrangements/EinsteinI...', 'pciutils', '/home/ianhin/Cactus/EinsteinToolkitGit/arrangements/ExternalL...', 'CoordBase', '/home/ianhin/Cactus/EinsteinToolkitGit/arrangements/CactusBas...', 'ADMConstraints', '/home/ianhin/Cactus/EinsteinToolkitGit/arrangements/EinsteinA...', 'OpenBLAS', ...) called at /home/ianhin/Cactus/EinsteinToolkitGit/lib/sbin/CST line 142
make[1]: *** [/home/ianhin/Cactus/EinsteinToolkitGit/configs/gw150914/config-data/make.thornlist] Error 255
make: *** [gw150914] Error 2

I'm pretty sure this violates the description of the allowed values for "parameter values" in I have corrected the param.ccl file. I am mentioning it here in case anyone thinks that the parser should be extended to support this incorrect usage, since it was accepted by the previous version. I don't think that this is worth doing.


Comments (3)

  1. Roland Haas
    • removed comment

    Yes, this violates the docs which state that empty strings (which is what is used for a regex that matches anything) need to be given as "" ie the quotes are required here (the docs say something like quotes are required for empty strings and strings containing space). This has been documented since 2005 (git hash c3e9e0714d45f550b9190d3e204432de41944cb9).

  2. Log in to comment