Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

robbat2
Copy link
Contributor

@robbat2 robbat2 commented Aug 3, 2020

The testcase w/ Python doesn't really need Python3, it just needs to validate python; so use ast instead of compiler, which is available in Py3.

Also support moreutils on other Linux distributions where the parallel binary might have a different location, e.g. Gentoo.

@cainejette cainejette requested a review from a team August 4, 2020 17:30
Copy link
Member

@ryansimmen ryansimmen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Only one small suggestion. Thank you for making this change.

@ryansimmen
Copy link
Member

ryansimmen commented Aug 7, 2020

@robbat2 please sync your fork with the latest master, merge master into your branch and push for the Actions tests to run in this PR. Thank you.

@robbat2
Copy link
Contributor Author

robbat2 commented Sep 19, 2020

@ryansimmen rebased on master now

@ryansimmen ryansimmen merged commit 2e71786 into github:master Sep 20, 2020
@jianghao0718 jianghao0718 added bug and removed bug labels Sep 23, 2020
@jianghao0718 jianghao0718 changed the title Improve Python3 & Finding Moreutils Parallel Improve python3 & finding moreutils parallel Sep 23, 2020
This was referenced Sep 23, 2020
gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Sep 23, 2020
FULL upstream Py3 support added (partial support was present before)

Reference: github/backup-utils#627
Fixes: https://bugs.gentoo.org/735200
Signed-off-by: Robin H. Johnson <[email protected]>
amildahl pushed a commit that referenced this pull request Nov 28, 2023
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.

3 participants