#2050: cctk_startupxxx is a valid group ----------------------+----------------------------------------------------- Reporter: sbrandt | Owner: sbrandt Type: defect | Status: new Priority: unset | Milestone: Component: Other | Version: development version Resolution: | Keywords: ----------------------+-----------------------------------------------------
Comment (by rhaas):
Sounds like an important fix to have. Your proposed patch is missing a ")" in the English language hint it seems. Why is there a check for level not being 0 in line 973?
Re-stating Erik's worry about MPI (and this seems like a less trivial change to me): are we *sure* that this will not break any valid schedule.ccl files (anywhere, not just the one in the ET)?
The alternative is to leave it current code in place which does introduce a new bug (which we can document) but at least the bug is not possibly preventing builds (making it major rather than a possible blocker).