#2539 closed defect (done)

MergeConfigs.cmd does not work correctly

Reported by: abeham Owned by: abeham
Priority: medium Milestone: HeuristicLab 3.3.14
Component: General Version: 3.3.13
Keywords: Cc:

Description

The MergeConfigs.cmd uses a variable Outdir which comes from the build macro $(Outdir) (a misspelled version of $(OutDir)). The variable was thus empty and the default ".\bin" was used. It is generally wrong to use only OutDir as this is a path relative to ProjectDir and TargetDir should be used instead which is an absolute path.

In addition, there is still a CustomPostBuildTemplate_UpdateLocalInstallation.cmd which is a leftover from a very early version of HeuristicLab. This and some other files are to be terminated.

Change History (5)

comment:1 Changed 22 months ago by abeham

  • Status changed from new to accepted

comment:2 Changed 22 months ago by abeham

  • Owner changed from abeham to ascheibe
  • Status changed from accepted to reviewing

r13428: fixed MergeConfigs.cmd and .sh files, removed obsolete files

comment:3 Changed 22 months ago by abeham

r13429: removed files from project

comment:4 Changed 22 months ago by ascheibe

  • Owner changed from ascheibe to abeham
  • Status changed from reviewing to readytorelease

Reviewed r13428 and r13429: Looks good, except that the post build event got duplicated in the project file. I removed the duplication in r13430, I think it's now correct.

comment:5 Changed 22 months ago by abeham

  • Resolution set to done
  • Status changed from readytorelease to closed

r13453: merged r13428:13430 to stable

Note: See TracTickets for help on using tickets.