Skip to content

fix: changed hyde-shell execution to use bashInteractive#183

Merged
richen604 merged 1 commit intorichen604:mainfrom
de-abreu:fix/missing-compgen
Jan 23, 2026
Merged

fix: changed hyde-shell execution to use bashInteractive#183
richen604 merged 1 commit intorichen604:mainfrom
de-abreu:fix/missing-compgen

Conversation

@de-abreu
Copy link
Contributor

Description

Changed hyde-shell execution to use bashInteractive

Type of change

This prevents that the following error be triggered:

~/.config/NixOS feat/hydenix*
❯ hyde-shell
/nix/store/41p02idzg7qib93isxqyqr7zdn0yijz1-hyde-modified/Configs/.local/bin/hyde-shell: line 615: compgen: command not found

compgen is not available for the minimal bach instalations that nix uses for its writeShellScript funciton.

  • [ x ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

  • [ x ] My commits follow conventional commit format
  • [ x ] I have updated the documentation accordingly
  • [ x ] My changes generate no new warnings

@richen604
Copy link
Owner

looks good! just reword the commit to fix: ... and i can merge

@de-abreu
Copy link
Contributor Author

Hello, Richen. Sorry for the late reply. Maybe the commit can go through now?

@richen604 richen604 force-pushed the fix/missing-compgen branch from 5e51b73 to f1ae65d Compare January 23, 2026 00:43
@richen604 richen604 merged commit 98140fd into richen604:main Jan 23, 2026
2 checks passed
@richen604
Copy link
Owner

Hello, Richen. Sorry for the late reply. Maybe the commit can go through now?

I fixed your commits and merged

@github-actions
Copy link
Contributor

🎉 This PR is included in version 6.0.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants