Skip to content

Replace pipes.quote with shlex.quote on Python 3 - #75

Open
musicinmybrain wants to merge 1 commit into
xolox:masterfrom
musicinmybrain:pipes
Open

Replace pipes.quote with shlex.quote on Python 3#75
musicinmybrain wants to merge 1 commit into
xolox:masterfrom
musicinmybrain:pipes

Conversation

@musicinmybrain

Copy link
Copy Markdown

The shlex.quote() API is available from Python 3.3 on; pipes.quote() was never documented, and is removed in Python 3.13.

Fixes #73.

The shlex.quote() API is available from Python 3.3 on; pipes.quote() was
never documented, and is removed in Python 3.13.

Fixes xolox#73.
gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Jun 30, 2024
The test restriction is dropped as the test suite seems to work. It was
necessary to apply patch [1] in order to make py3.13 work.

[1] xolox/python-humanfriendly#75

Signed-off-by: Petr Vaněk <arkamar@gentoo.org>

@smoser smoser left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm not a maintainer, but I think this looks good.

Perhaps it fits in the 'compat.py'

This module exposes aliases and functions that make it easier to write Python
code that is compatible with Python 2 and Python 3.

and then used from there. But either way, it would be nice to have this as python 3.13 is eminent.

Thanks for the PR @musicinmybrain

mhy-pexip pushed a commit to pexip/os-humanfriendly that referenced this pull request Apr 28, 2025
humanfriendly (10.0-6) unstable; urgency=medium
.
  * Team upload.
  * Replace pipes.quote with shlex.quote on Python 3
    (xolox/python-humanfriendly#75, closes:
    #1084718).
  * Remove some Python 2 packaging leftovers.
.
humanfriendly (10.0-5) unstable; urgency=medium
.
  * Team upload.
  * add d/gbp.conf matching what is done on Salsa
  * patch-out trivial usage of python3-mock
.
humanfriendly (10.0-4) unstable; urgency=medium
.
  * remove myself
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet