Skip to content
Snippets Groups Projects

"Fixed" scan_dict_deps.py for not including directories

Merged Sebastien Ponce requested to merge sponce_scan_dict_deps into master

This was discovered thanks to a name collision between Eigen and DD4hep, Eigen having an header file named "Core" (with no extension indeed) and DD4hep a directory with same name. DD4hep happened to come first in the dependency list and thus the directort was used...

Note that this reveal a deeper problem in the scanning made by this tool : no scoping and thus no cleaning of the list of directories when going to next file. This is the actual root of the problem that lead to DD4hep being first in the dependency list

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading