View Issue Details

IDProjectCategoryView StatusLast Update
0001689OpenFOAM[All Projects] Bugpublic2015-05-14 12:15
ReporterTimm SeverinAssigned Tohenry 
PrioritylowSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
PlatformGNU/LinuxOSUbuntuOS Version14.04
Product Version 
Fixed in Version 
Summary0001689: RunFunctions not accepting tab-character
DescriptionThe 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 ReproduceReplace the spaces by a tab in the controlDict and execute an arbitrary Allrun
Additional InformationSuggested action:

Match for "\s*" (whitespace: space / tab - matching for sed) instead of " *".

Suitable patch attached
TagsNo tags attached.

Activities

Timm Severin

2015-05-11 14:19

reporter  

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()
RunFunctions.patch (516 bytes)

henry

2015-05-14 12:14

manager   ~0004739

Thanks for the patch
Resolved by commit 4baafccbe4e866bb9d088ec1aff29104f22e54a2

Issue History

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