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.

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

fall back to rage if age is unavailable #31

Merged
merged 1 commit into from
Jun 23, 2024
Merged

fall back to rage if age is unavailable #31

merged 1 commit into from
Jun 23, 2024

Conversation

arcxio
Copy link
Collaborator

@arcxio arcxio commented Jun 21, 2024

I've tested pa with rage and it does indeed work without issues just by changing the binary name, so maybe there's no reason to limit the options. I'd leave it as an undocumented option though, age would still be recommended as a reference implementation with a more active development (and it's still faster AFAIU - str4d/rage#57). Passage provides PASSAGE_AGE option to change the binary name, but I think that solution is worse: in addition to being a redundant configuration option that can just be inferred from what's installed, age spec doesn't specify API for CLIs, so there's no guarantee that any CLI that's going to implement age protocol is going to be interoperable in terms of interface with age(1) the way rage is.

@arcxio arcxio requested a review from biox June 21, 2024 13:39
@arcxio arcxio force-pushed the rage branch 2 times, most recently from b90efc5 to 404255d Compare June 21, 2024 23:01
@biox
Copy link
Owner

biox commented Jun 23, 2024

lgtm! i don't see why we shouldn't fall back to rage, especially given that rage aims to be interoperable with age. if rage ever changes this stance, maybe we reconsider!

@biox biox merged commit 9514016 into main Jun 23, 2024
@arcxio arcxio deleted the rage branch June 23, 2024 03:27
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