Skip to content

Remove obsolete legacy headers - #110

Draft
skilledwolf wants to merge 1 commit into
masterfrom
agent/remove-deprecated-headers
Draft

Remove obsolete legacy headers#110
skilledwolf wants to merge 1 commit into
masterfrom
agent/remove-deprecated-headers

Conversation

@skilledwolf

Copy link
Copy Markdown
Collaborator

Summary

  • remove the four opt-in deprecated compatibility headers under legacy/alps
  • remove the unused ALPS_INSTALL_DEPRECATED CMake option and install rule
  • stop the line-count helper from scanning the removed directory

These headers were deprecated in 2010, are not used within ALPS, are disabled by default, and have no test coverage. The alps/vectorio.h shim was also ineffective because it included itself.

Migration

  • alps/copyright.halps/utility/copyright.hpp
  • alps/osiris/os.halps/utility/os.hpp
  • alps/vectorio.halps/utility/vectorio.hpp
  • alps/math.hpp → the required headers under alps/numeric/

Checks

  • fresh minimal CMake configuration
  • script/linecount.sh
  • no remaining references to legacy/alps or ALPS_INSTALL_DEPRECATED
  • git diff --check

@skilledwolf skilledwolf changed the title Remove obsolete legacy headers Update legacy license headers Jul 23, 2026
@skilledwolf skilledwolf changed the title Update legacy license headers Remove obsolete legacy headers Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant