files.put() does not accept pathlib.Path

[NedJWestern]

Describe the bug

A clear and concise description of what the bug is.

files.put() breaks when dest is a pathlib.Path instead of a string. I understand that a string is expected here, but this is not explicitly stated in the docs. Adding datatypes to all of the docs would be ideal.

To Reproduce

Steps to reproduce the behavior, please include where possible:

• Operation code & usage
• Target system information
• Example using the @docker connector (helps isolate the problem)

import os
from pathlib import Path
from pyinfra.operations import files


files.put(
    src="foo.txt",
    dest=Path("/data") / "tmp/",
)

Error output:

--> An internal exception occurred:

  File "/data/home/su_nwestern/repos/pkg_pyinfra/pyinfra/pyinfra/operations/util/files.py", line 11, in <listcomp>
    part_list[0:-1] = [part.rstrip("/") for part in part_list[0:-1]]
      ^^^^^^^^^^^^^^^^^
AttributeError: 'PosixPath' object has no attribute 'rstrip'

Expected behavior

A clear and concise description of what you expected to happen.

Behave the same as if dest was a string

Meta

• Include output of pyinfra --support.

• How was pyinfra installed (source/pip)?

• Include pyinfra-debug.log (if one was created)

• Consider including output with -vv and --debug.

  System: Linux
  Platform: Linux-4.18.0-553.22.1.el8_10.x86_64-x86_64-with-glibc2.28
  Release: 4.18.0-553.22.1.el8_10.x86_64
  Machine: x86_64
  pyinfra: v3.1.1
  black: v24.4.2
  black: v24.4.2
  click: v8.1.7
  configparser: v7.1.0
  coverage: v7.5.1
  coverage: v7.5.1
  distro: v1.9.0
  flake8: v7.0.0
  flake8: v7.0.0
  flake8-black: v0.3.6
  flake8-black: v0.3.6
  flake8-isort: v6.1.1
  flake8-isort: v6.1.1
  flake8-spellcheck: v0.12.1
  gevent: v24.10.1
  importlib-metadata: v8.5.0
  ipdb: v0.13.13
  ipdbplugin: v1.5.0
  ipython: v8.28.0
  isort: v5.13.2
  isort: v5.13.2
  jinja2: v3.1.4
  mypy: v1.11.2
  mypy: v1.11.2
  myst-parser: v2.0.0
  myst-parser: v2.0.0
  packaging: v23.2
  paramiko: v3.5.0
  pyinfra-guzzle-sphinx-theme: v0.16
  pyinfra-guzzle-sphinx-theme: v0.16
  pytest: v8.2.1
  pytest: v8.2.1
  pytest-cov: v5.0.0
  pytest-cov: v5.0.0
  python-dateutil: v2.9.0.post0
  pywinrm: v0.5.0
  redbaron: v0.9.2
  setuptools: v65.5.1
  sphinx: v6.2.1
  sphinx: v6.2.1
  twine: v5.1.1
  typeguard: v4.3.0
  types-PyYAML: v6.0.12.20240917
  types-PyYAML: v6.0.12.20240917
  types-cryptography: v3.3.23.2
  types-cryptography: v3.3.23.2
  types-paramiko: v3.5.0.20240928
  types-paramiko: v3.5.0.20240928
  types-python-dateutil: v2.9.0.20241003
  types-python-dateutil: v2.9.0.20241003
  types-setuptools: v75.1.0.20240917
  types-setuptools: v75.1.0.20240917
  typing-extensions: v4.12.2
  wheel: v0.44.0
  Executable: /data/home/su_nwestern/.local/bin/pyinfra
  Python: 3.11.9 (CPython, GCC 8.5.0 20210514 (Red Hat 8.5.0-22))

[simonhammes]

I've opened #1227 to address the docs part of this issue.

Accepting pathlib.Path can be handled later on 👍