Print

Print


  Branch: refs/heads/master
  Home:   https://github.com/xrootd/xrootd
  Commit: 6ba2d73ca9486cdcb8f2636dfc4704ba60344fa5
      https://github.com/xrootd/xrootd/commit/6ba2d73ca9486cdcb8f2636dfc4704ba60344fa5
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-01-21 (Fri, 21 Jan 2022)

  Changed paths:
    M packaging/rhel/xrootd.spec.in

  Log Message:
  -----------
  [RPM] Add python3-setuptools to BuildRequires

For setuptools to be picked up during the RPM build BuildRequires needs
to specify either python2-setuptools (for Python 2) or python3-setuptools
(for Python 3).


  Commit: 80890de66f66208346c3f3e81aca8209fb60a852
      https://github.com/xrootd/xrootd/commit/80890de66f66208346c3f3e81aca8209fb60a852
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-01-21 (Fri, 21 Jan 2022)

  Changed paths:
    M packaging/wheel/setup.py

  Log Message:
  -----------
  [Python] Use CPython libraries over distuils

As distutils and Easy Install are deprecated

c.f.:
* https://setuptools.pypa.io/en/stable/deprecated/distutils-legacy.html
* https://setuptools.pypa.io/en/stable/deprecated/easy_install.html

it is preferable to convert use of distutils to either use existing CPython
libraries or use setuptool's versions of them. This patch applies

* distutils.spawn.find_executable -> shutil.which

Henry Schreiner pointed out that shutil.which (added in Python 3.3) is preferable
to setuptools._distutils.spawn.find_executable.

This patch was meant to also change:

* distutils.setup -> setuptools.setup
* distutils.sysconfig -> CPython's sysconfig

but it seems that these are still required given the current state of the build
and will need to be revised in a later PR.

The change

distutils.core.Extension -> setuptools.Extension

could be applied in:

* bindings/python/setup_pypi.py
* bindings/python/setup_standalone.py

but isn't as these files are obsolete and should be removed.

Co-authored-by: Henry Schreiner <[log in to unmask]>


  Commit: 538ed5c78a0bc848dfa919446be5395c05a46424
      https://github.com/xrootd/xrootd/commit/538ed5c78a0bc848dfa919446be5395c05a46424
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-01-21 (Fri, 21 Jan 2022)

  Changed paths:
    M bindings/python/setup_pypi.py
    M packaging/wheel/setup.py

  Log Message:
  -----------
  [Python] Remove unused Python imports


  Commit: 5fe4f9db7d818cdf96af97cf221205baa2e6a856
      https://github.com/xrootd/xrootd/commit/5fe4f9db7d818cdf96af97cf221205baa2e6a856
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-01-21 (Fri, 21 Jan 2022)

  Changed paths:
    M packaging/wheel/install.sh

  Log Message:
  -----------
  [Python] Try to install sdist with pip during packaging, fall back to setup.py


  Commit: 111662e7c52ce3d4d5ecbe0d8054d45d6063d954
      https://github.com/xrootd/xrootd/commit/111662e7c52ce3d4d5ecbe0d8054d45d6063d954
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-01-21 (Fri, 21 Jan 2022)

  Changed paths:
    M packaging/wheel/install.sh
    M packaging/wheel/publish.sh

  Log Message:
  -----------
  [PyPI] Try to use build to make sdist, else fall back to setup.py sdist

Also add check for wheel to avoid error during publish.sh


  Commit: df3188c7669d7f8fdbc7ecd6996ae71d38dc79a3
      https://github.com/xrootd/xrootd/commit/df3188c7669d7f8fdbc7ecd6996ae71d38dc79a3
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-01-21 (Fri, 21 Jan 2022)

  Changed paths:
    M .github/workflows/build.yml

  Log Message:
  -----------
  [CI] Add sdist building with publishing to CentOS 7 RPM stage


  Commit: 65950acd7ab15dc91e31fdc4de089e14d474ecf9
      https://github.com/xrootd/xrootd/commit/65950acd7ab15dc91e31fdc4de089e14d474ecf9
  Author: simonmichal <[log in to unmask]>
  Date:   2022-01-26 (Wed, 26 Jan 2022)

  Changed paths:
    M .github/workflows/build.yml
    M bindings/python/setup_pypi.py
    M packaging/rhel/xrootd.spec.in
    M packaging/wheel/install.sh
    M packaging/wheel/publish.sh
    M packaging/wheel/setup.py

  Log Message:
  -----------
  Merge pull request #1585 from matthewfeickert/fix/use-sysconfig-over-distutils

fix: Use `shutil.which` over `distutils.spawn.find_executable` when possible


Compare: https://github.com/xrootd/xrootd/compare/48c31961838a...65950acd7ab1

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1