cli icon indicating copy to clipboard operation
cli copied to clipboard

Use standard zsh path for rpm/RHEL releases

Open mariaWitch opened this issue 3 years ago • 2 comments

Due to a quirk with OSTree releases, the rpm releases should place the doppler zsh completion file in /usr/share/zsh/, which is the default for Fedora/RHEL/CentOS based systems. In theory this should fix #331 but I will need to perform testing to confirm. There may be a more elegant way to accomplish this, but this seems to be the most straight forward way that I can find for the moment.

mariaWitch avatar Sep 13 '22 17:09 mariaWitch

Validated on Fedora CoreOS. zsh also autocompletes, so functionality is fully working.

mariaWitch avatar Sep 13 '22 19:09 mariaWitch

Validated on AlmaLinux. Going to do Fedora and then CentOS next. (These are non-OSTree based distros).

Update: Confirmed working on zsh on Fedora and CentOS Stream as well.

mariaWitch avatar Sep 14 '22 23:09 mariaWitch

@Piccirello Looking for review on this/the related issue.

mariaWitch avatar Sep 28 '22 17:09 mariaWitch

Thanks for putting up this fix, and for testing across those different OSes! Could you squash the two merge commits so that this PR only has one commit? We can then get this merged in and released.

Piccirello avatar Oct 05 '22 19:10 Piccirello

@Piccirello Done.

mariaWitch avatar Oct 06 '22 02:10 mariaWitch

I believe I have solved the issue for why the e2e test was failing (it wasn't related to any of the changes, and was environment specific). If you could manually re-run the test on your end, it should pass now. @Piccirello

mariaWitch avatar Oct 07 '22 18:10 mariaWitch

I was able to get it to work on the forked repository with the correct secret values. So I don't understand why it isn't working here. It seems to be a github environment related issue though. image

mariaWitch avatar Oct 07 '22 18:10 mariaWitch

Same as before, just updated again.

mariaWitch avatar Oct 10 '22 21:10 mariaWitch

This change has been released in CLI v3.45.0.

Piccirello avatar Oct 12 '22 19:10 Piccirello