Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Upstream hexdump check breaks usage on SmartOS #65

Closed
teutat3s opened this issue Aug 5, 2024 · 1 comment · Fixed by #66
Closed

Upstream hexdump check breaks usage on SmartOS #65

teutat3s opened this issue Aug 5, 2024 · 1 comment · Fixed by #66
Assignees
@teutat3s

Comments

@teutat3s
Copy link
Member

teutat3s commented Aug 5, 2024

With dehydrated 0.7.1 hexdump was added to required binaries. hexdump is not (yet?) available on illumos. See dehydrated-io/dehydrated#910.

/opt/triton-dehydrated/dehydrated -c
# INFO: Using main config file /opt/triton-dehydrated/config
ERROR: This script requires hexdump.
exit_hook
@teutat3s teutat3s self-assigned this Aug 5, 2024
teutat3s added a commit that referenced this issue Aug 5, 2024
@teutat3s
Add patch to move hexdump check to function that
03eec97 
requires it #65
@teutat3s teutat3s mentioned this issue Aug 5, 2024
Merged
teutat3s added a commit that referenced this issue Aug 7, 2024
@teutat3s
Add patch to replace hexdump with od
8303351 
Apply patch early dehydrated-io/dehydrated#943
Fixes #65
@teutat3s
Copy link
Member Author

teutat3s commented Aug 7, 2024

Thanks to @bahamat, there's now a PR to fix this upstream as well dehydrated-io/dehydrated#943.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@teutat3s teutat3s

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add patch to replace hexdump with od TritonDataCenter/triton-dehydrated
1 participant
@teutat3s