View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000048 | OpenFOAM | Bug | public | 2010-09-25 08:38 | 2010-10-07 18:02 |
Reporter | Assigned To | henry | |||
Priority | low | Severity | major | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Summary | 0000048: dirname computation bug in makeThirdPartyFunctions | ||||
Description | In the function cpMakeFiles the dirname of the file to copy is computed with ${i%%/*}, while it should be ${i%/*}. Otherwise everything up to the _first_ / will be deleted, instead of the _last_ slash. This causes the function to copy all the files into <package>/Make/ instead of <package>/<dirname>/Make/. | ||||
Steps To Reproduce | Just run Allwmake in the ThirdParty-1.7.x directory and observe how the */Make/{options,files} are created. | ||||
Additional Information | Attached patch fixes this. | ||||
Tags | Utility | ||||
2010-09-25 08:38
|
0001-Fix-dirname-computation-in-makeThirdPartyFunctions.patch (808 bytes)
From 794b58ec8257a40b173dc1bfbbfeeaa42117e3b3 Mon Sep 17 00:00:00 2001 From: Michael Wild <themiwi@users.sourceforge.net> Date: Sat, 25 Sep 2010 09:24:40 +0200 Subject: [PATCH] Fix dirname computation in makeThirdPartyFunctions Signed-off-by: Michael Wild <themiwi@users.sourceforge.net> --- tools/makeThirdPartyFunctions | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/makeThirdPartyFunctions b/tools/makeThirdPartyFunctions index 9106251..5abb092 100755 --- a/tools/makeThirdPartyFunctions +++ b/tools/makeThirdPartyFunctions @@ -99,7 +99,7 @@ cpMakeFiles() for i in $(cd $wmakeFiles && find . -type f) do - d=${i%%/*} # dirname + d=${i%/*} # dirname b=${i##*/} # basename mkdir -p $dst/$d/Make 2>/dev/null -- 1.7.2.3 |
|
Thanks for the big-fix, this will be included in the next release of the ThirdParty pack. |
Date Modified | Username | Field | Change |
---|---|---|---|
2010-09-25 08:38 |
|
New Issue | |
2010-09-25 08:38 |
|
File Added: 0001-Fix-dirname-computation-in-makeThirdPartyFunctions.patch | |
2010-09-28 20:03 | henry | Assigned To | => henry |
2010-09-28 20:03 | henry | Status | new => assigned |
2010-09-28 20:18 | henry | Note Added: 0000055 | |
2010-09-28 20:18 | henry | Status | assigned => resolved |
2010-09-28 20:18 | henry | Resolution | open => fixed |
2010-10-07 18:02 |
|
Tag Attached: Utility |