Skip to content

Consolidate dependency on OpenSSH headers.#65

Open
blunderer wants to merge 1 commit into
mainfrom
better-includes
Open

Consolidate dependency on OpenSSH headers.#65
blunderer wants to merge 1 commit into
mainfrom
better-includes

Conversation

@blunderer
Copy link
Copy Markdown
Contributor

@blunderer blunderer commented May 26, 2025

Create a new openssh/ folder and add a wrapper to OpenSSH includes. This
allows the code to:

  • Rely on includes.h from OpenSSH for better compatibility (see
    github.com/google/hiba/commit/642d64ffec5719178644078867bcd4ff88298621).
  • Fix pre-processor conflicts on config.h symbols that are used in both
    projects.

This also remove uneeded includes in some source files.

@blunderer blunderer force-pushed the better-includes branch 5 times, most recently from 3663d38 to 256dd3d Compare May 28, 2025 08:46
@blunderer blunderer force-pushed the better-includes branch 2 times, most recently from 4a81c24 to 270d343 Compare May 28, 2025 10:01
Create a new openssh/ folder and add a wrapper to OpenSSH includes. This
allows the code to:

* Rely on includes.h from OpenSSH for better compatibility (see
  github.com/google/hiba/commit/642d64ffec5719178644078867bcd4ff88298621).
* Fix pre-processor conflicts on config.h symbols that are used in both
  projects.

This also remove uneeded includes in some source files.
@blunderer blunderer changed the title Isolate OpenSSH includes. Consolidate dependency on OpenSSH headers. May 28, 2025
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.

2 participants