Skip to content

Conversation

@Rutpiv
Copy link
Contributor

@Rutpiv Rutpiv commented Dec 24, 2025

Testing the changes

  • I tested the changes in this PR: briefly

Local build testing

  • I built this PR locally for my native architecture (x86_64)
  • I built this PR locally for these architectures using specific masterdirs:
    • x86_64-musl
  • I built this PR locally for these architectures (crossbuilds):
    • aarch64
    • aarch64-musl

@Rutpiv
Copy link
Contributor Author

Rutpiv commented Dec 24, 2025

During testing, the build and install process appears to require the patterns directory to be named exactly ImHex-Patterns in order to complete correctly.

For this reason, the extracted ImHex-Patterns-ImHex-v${version} directory is renamed to ImHex-Patterns in post_extract, matching what upstream expects during installation.

This update also installs a few additional resources from ImHex-Patterns that are not installed by default upstream, but are present in the patterns repository and can be used by the application at runtime:

  • disassemblers
  • plugins
  • scripts
  • themes
  • tips

The SDK directory (/usr/share/imhex/sdk) is no longer installed.
It significantly increases the package size and appears to be primarily intended for external plugin development rather than regular application usage. Other distributions also seem not to ship the SDK as part of the default installation.

The SDK could be split into a dedicated package, for example imhex-plugin-sdk, keeping the main package lean while still providing support for plugin developers. I can add this to the template if it makes sense. Alternatively, we can revert to the previous state and keep the SDK bundled together if that is preferred.

@classabbyamp
Copy link
Member

split the sdk, it's useful

@Rutpiv
Copy link
Contributor Author

Rutpiv commented Dec 25, 2025

Thanks. I’ve split the plugin SDK into a separate subpackage, imhex-plugin-sdk.

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