View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001689 | OpenFOAM | Bug | public | 2015-05-11 14:19 | 2015-05-14 12:15 |
Reporter | Timm Severin | Assigned To | henry | ||
Priority | low | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | GNU/Linux | OS | Ubuntu | OS Version | 14.04 |
Summary | 0001689: RunFunctions not accepting tab-character | ||||
Description | The RunFunctions tools included in 2.3.x (and probably others) do not allow tab-characters as separators between "application" and the solver name. Same is valid for the getNumberOfProcessors function. | ||||
Steps To Reproduce | Replace the spaces by a tab in the controlDict and execute an arbitrary Allrun | ||||
Additional Information | Suggested action: Match for "\s*" (whitespace: space / tab - matching for sed) instead of " *". Suitable patch attached | ||||
Tags | No tags attached. | ||||
|
RunFunctions.patch (516 bytes)
--- RunFunctions 2015-05-11 15:03:41.861477689 +0200 +++ RunFunctions.new 2015-05-11 15:04:20.613669851 +0200 @@ -30,12 +30,12 @@ getNumberOfProcessors() { - sed -ne 's/^numberOfSubdomains *\(.*\);/\1/p' system/decomposeParDict + sed -ne 's/^numberOfSubdomains\s*\(.*\);/\1/p' system/decomposeParDict } getApplication() { - sed -ne 's/^ *application *\([a-zA-Z]*\) *;.*$/\1/p' system/controlDict + sed -ne 's/^ *application\s*\([a-zA-Z]*\)\s*;.*$/\1/p' system/controlDict } runApplication() |
|
Thanks for the patch Resolved by commit 4baafccbe4e866bb9d088ec1aff29104f22e54a2 |
Date Modified | Username | Field | Change |
---|---|---|---|
2015-05-11 14:19 | Timm Severin | New Issue | |
2015-05-11 14:19 | Timm Severin | File Added: RunFunctions.patch | |
2015-05-14 12:14 | henry | Note Added: 0004739 | |
2015-05-14 12:14 | henry | Status | new => resolved |
2015-05-14 12:14 | henry | Resolution | open => fixed |
2015-05-14 12:14 | henry | Assigned To | => henry |