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

Shyaml Ubuntu 24 fixes #2740

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Shyaml Ubuntu 24 fixes #2740

wants to merge 2 commits into from

Conversation

tomjn
Copy link
Member

@tomjn tomjn commented Oct 20, 2024

Cherry picked work from @pentatonicfunk to fix shyaml installation on Ubuntu 24

Checks

  • I've updated the changelog.
  • I've tested this PR
  • This PR is for the develop branch not the stable branch.
  • This PR is complete and ready for review.

Copy link

update-docs bot commented Oct 20, 2024

Thanks for opening this pull request! Make sure CHANGELOG.md gets updated with this change, additionally any docs that need updated can be found at https://github.com/Varying-Vagrant-Vagrants/varyingvagrantvagrants.org

GitHub
The VVV docs and website. Contribute to Varying-Vagrant-Vagrants/varyingvagrantvagrants.org development by creating an account on GitHub.

@tomjn
Copy link
Member Author

tomjn commented Oct 20, 2024

Related to #2739

@tomjn tomjn marked this pull request as ready for review October 29, 2024 17:35
@tomjn
Copy link
Member Author

tomjn commented Oct 29, 2024

@pentatonicfunk could the pipx tool solve this?

@pentatonicfunk
Copy link
Contributor

@pentatonicfunk could the pipx tool solve this?

could be. i think i tried with pipx, the install went okay but somehow shyaml isn't callable. i'll try to play around with it again soon

@pentatonicfunk
Copy link
Contributor

so the problem with pipx, turns out about path-ing, by default it goes to user home bin

vagrant@vvv:~$ source ~/.bashrc
vagrant@vvv:~$ which shyaml
/home/vagrant/.local/bin/shyaml

there is --global parameter in their documentation, but somehow its not recognized in my test

vagrant@vvv:~$ sudo pipx ensurepath --global
usage: pipx [-h] [--quiet] [--verbose] [--version] {install,uninject,inject,upgrade,upgrade-all,uninstall,uninstall-all,reinstall,reinstall-all,list,run,runpip,ensurepath,environment,completions} ...
pipx: error: unrecognized arguments: --global

@pentatonicfunk
Copy link
Contributor

pentatonicfunk commented Oct 29, 2024

right, so seems ubuntu repo has pipx in version 1.4.3 https://launchpad.net/ubuntu/noble/+package/pipx
meanwhile --global only added in 1.5.0 https://github.com/pypa/pipx/releases/tag/1.5.0

Launchpad
pipx allows you to... . * Run the latest version of a CLI application from a package in a temporary virtual environment, leaving your system untouched after it finishes. * Install packages to isolated virtual environments, while globally exposing their CLI applications so you can run them from anywhere. * Easily list, upgrade, and uninstall packages that were installed with pipx. . pipx runs with regular user permissions, never calling "sudo pip install".
GitHub
What's Changed

[pre-commit.ci] pre-commit autoupdate by @pre-commit-ci in #1209
fix: allow older setuptools_scm versions when building by @henryiii in #1208
ci: update to artifacts v2 by @henryiii...

@tomjn
Copy link
Member Author

tomjn commented Oct 29, 2024

meh lets add it in the future once we've figured that out, and focus on getting this reviewed and merged

@tomjn tomjn requested a review from a team October 29, 2024 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants