Print

Print


  Branch: refs/heads/master
  Home:   https://github.com/xrootd/xrootd
  Commit: 991fcb63e7fbaa9485cda0bdbdcacb5dadf69f58
      https://github.com/xrootd/xrootd/commit/991fcb63e7fbaa9485cda0bdbdcacb5dadf69f58
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-04-26 (Tue, 26 Apr 2022)

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

  Log Message:
  -----------
  [FIX] Use python's sys.version_info.minor to ensure correct library

To avoid finding multiple directories that have 'python' in the name
under /usr/local/lib/ on the GitHub Actions runners, use Python's
sys.version_info.minor to get the minor version of python3. As we know
that we're using Python 3, the minor version is the only piece of information
that needs to be determined.


  Commit: 1c885e0ad71cd70fb9822c01a942a9c1eac0177a
      https://github.com/xrootd/xrootd/commit/1c885e0ad71cd70fb9822c01a942a9c1eac0177a
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-04-26 (Tue, 26 Apr 2022)

  Changed paths:
    M packaging/wheel/install.sh

  Log Message:
  -----------
  [FIX] Fix import syntax to enable shutil.which check

In PR 1585 a syntax error was introduced  as `shutil.which` is a method of the Python
'shutil' module and not a module, so the proper import is `from shutil import which`
as regardless of the CPython version `import shutil.which` will result in a

ModuleNotFoundError: No module named 'shutil.which'; 'shutil' is not a package

This error was causing all runs of packaging/wheel/install.sh to use the legacy behavior
of `python setup.py install` regardless if `shutil.which` was available or not.


  Commit: 03d1ac849433de1f41c059c7904b2747b8a61416
      https://github.com/xrootd/xrootd/commit/03d1ac849433de1f41c059c7904b2747b8a61416
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-04-26 (Tue, 26 Apr 2022)

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

  Log Message:
  -----------
  [FIX] Remove brew install of git for cmake-macos job

The brew installed git version conflicts with the installed version in the
macOS environment on the GitHub Action runners.


  Commit: 5791b277ab8e0d472b3f9e0a4025fdaa3d721c42
      https://github.com/xrootd/xrootd/commit/5791b277ab8e0d472b3f9e0a4025fdaa3d721c42
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-04-26 (Tue, 26 Apr 2022)

  Changed paths:
    M packaging/wheel/install.sh

  Log Message:
  -----------
  [pip] Remove manual renaming of egg-info to dist-info

As all CPython versions that have 'shutil.which' (Python 3.3+) are able to run
packaging/wheel/setup.py then they will be able to take advantage of having pip
be able to create a valid wheel during install time. In the event that a very old
version of pip is used, an egg distribution will be installed but it will still work
fine so there is no real reason to rename it in a manner that mocks the dist-info of a
wheel.


  Commit: 5088d6e47e3763bdcfbcaa35206b1f8aa5eddf47
      https://github.com/xrootd/xrootd/commit/5088d6e47e3763bdcfbcaa35206b1f8aa5eddf47
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-04-26 (Tue, 26 Apr 2022)

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

  Log Message:
  -----------
  [CI] Update checkout action to v3

* Use actions/checkout v3 for workflows jobs
   - c.f. https://github.com/actions/checkout/releases/tag/v3.0.0


  Commit: 3f1340aebeae9367532d77bddbef730d5d214481
      https://github.com/xrootd/xrootd/commit/3f1340aebeae9367532d77bddbef730d5d214481
  Author: Matthew Feickert <[log in to unmask]>
  Date:   2022-04-26 (Tue, 26 Apr 2022)

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

  Log Message:
  -----------
  [CI] Add ubuntu sdist build workflow

In light of checks needed for Issue 1668 for building Python bindings across
OS versions, add a GitHub Actions workflow job that builds the built sdist
on Ubuntu (ubuntu-latest in GHA).


  Commit: 6520868d75f8da1d0563386b08d0c4747a6b7852
      https://github.com/xrootd/xrootd/commit/6520868d75f8da1d0563386b08d0c4747a6b7852
  Author: simonmichal <[log in to unmask]>
  Date:   2022-04-26 (Tue, 26 Apr 2022)

  Changed paths:
    M .github/workflows/build.yml
    M packaging/wheel/install.sh

  Log Message:
  -----------
  Merge pull request #1672 from matthewfeickert/ci/add-ubuntu-sdist-job

[FIX] Fix import syntax to enable shutil.which check


Compare: https://github.com/xrootd/xrootd/compare/9db73ec71bec...6520868d75f8

########################################################################
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