search

binpash / pash

PaSh: Light-touch Data-Parallel Shell Processing
MIT License
547 stars 37 forks source link

Replace python with python3 package #637

Closed sethsabar closed 1 year ago

sethsabar commented 1 year ago

Package python causes issues on Ubuntu Linux when running sudo pash/scripts/distro-deps.sh during installation, python3 works.

angelhof commented 1 year ago

Thanks for this change! Unfortunately our CI infrastructure is down at the moment and I would like to avoid merging anything until it is fixed. When it is, we will push your PR through it and if everything goes smoothly we will merge it :)

angelhof commented 1 year ago

This seems to be related to this issue (#334).

This PR looks good! Now that the CI is running we could try to merge it. Could you:

  1. retarget it to future
  2. remove the rest of the python3 package references in that file (e.g., https://github.com/binpash/pash/blob/future/scripts/distro-deps.sh#L35)

After that, we can merge the PR if CI passes :)

angelhof commented 1 year ago

Closing this PR since I did the changes in #661