View Issue Details

IDProjectCategoryView StatusLast Update
0001872FreeCADBugpublic2014-12-22 09:50
Reporterpeterl94 Assigned Towmayer  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
OSWindows 
Summary0001872: HAVE_CONFIG_H regression
DescriptionCommit f0507331 reintroduced defining HAVE_CONFIG_H when OCE is found. This causes the build to fail on Windows, and is redundant on *nix.
TagsNo tags attached.
FreeCAD Information

Activities

shoogen

2014-12-22 07:53

developer   ~0005433

It seems that this command has been around since using cmake
(commit of 2009-06-26T11:56)

Peter, are you suggesting to just remove the line
add_definitions (-DHAVE_CONFIG_H)
as you did in f13f5dc52098c02bc53ac7a4e625c8e0d8e1ae2e ?

peterl94

2014-12-22 07:58

developer   ~0005434

Yes, I am. Correct me if I'm wrong, but isn't it redundant since it gets set if GCC or Clang is used?

Related Changesets

FreeCAD: master f0507331

2014-11-21 23:43:51

jobermayr


Committer: shoogen Details Diff
Fix OCE / OCC version detection. Affected Issues
0001872
mod - CMakeLists.txt Diff File
mod - cMake/FindOpenCasCade.cmake Diff File
mod - cMake/FreeCadMacros.cmake Diff File

FreeCAD: master 5478c0fd

2014-12-22 10:24:26

wmayer

Details Diff
+ fixes 0001872: HAVE_CONFIG_H regression Affected Issues
0001872
mod - cMake/FindOpenCasCade.cmake Diff File

Issue History

Date Modified Username Field Change
2014-12-22 00:42 peterl94 New Issue
2014-12-22 07:39 shoogen Changeset attached => FreeCAD Master master f0507331
2014-12-22 07:53 shoogen Note Added: 0005433
2014-12-22 07:58 peterl94 Note Added: 0005434
2014-12-22 09:50 wmayer Changeset attached => FreeCAD Master master 5478c0fd
2014-12-22 09:50 wmayer Assigned To => wmayer
2014-12-22 09:50 wmayer Status new => closed
2014-12-22 09:50 wmayer Resolution open => fixed