Skip to content
Snippets Groups Projects

SPI-2541: Python path issues for Python with double digit minor version

Merged SPI-2541: Python path issues for Python with double digit minor version
SPI_2542_PYTHON_PATH_issues into master
1 unresolved thread
Merged Tim Nicolai Ehmann requested to merge SPI_2542_PYTHON_PATH_issues into master
1 unresolved thread

Branch name Ticket number and Title ticket number are not the same. The title is the correct one.

To run the testing, add a comment with:

Bot test PACKAGE[ PACKAGE2 ...]|all in TOOLCHAIN|all [(only|also) for LABEL-COMPILER-BUILDTYPE] please!

To ensure patches required for existing LCG branches are not lost, please add the respective labels: alsoTargeting:LCG_102-patches, alsoTargeting:master, etc.

Edited by Tim Nicolai Ehmann

Merge request reports

Pipeline #7132998 passed

Stage: test

    Pipeline passed for 6977f44c on SPI_2542_PYTHON_PATH_issues

    Merged by Tim Nicolai EhmannTim Nicolai Ehmann 1 year ago (Mar 27, 2024 10:24pm UTC)

    Loading

    Pipeline #7133013 passed

    Stage: test

      Pipeline passed for 28b7b31a on master

      Activity

      Filter activity
      • Approvals
      • Assignees & reviewers
      • Comments (from bots)
      • Comments (from users)
      • Commits & branches
      • Edits
      • Labels
      • Lock status
      • Mentions
      • Merge request status
      • Tracking
      cmake/scripts/create_lcg_view_setup_sh.in
      42 42 fi
      43 43
      44 44 if [ -x ${thisdir}/bin/python ]; then
      45 PYTHON_VERSION=`expr $(readlink ${thisdir}/bin/python) : '.*/Python/\([0-9].[0-9]\).*'`
      45 PYTHON_VERSION=`expr $(readlink ${thisdir}/bin/python) : '.*/Python/\([0-9].[0-9][0-9]?\).*'`
      • Andre Sailer
        Andre Sailer @sailer ·
        Maintainer
        Suggested change
        45 PYTHON_VERSION=`expr $(readlink ${thisdir}/bin/python) : '.*/Python/\([0-9].[0-9][0-9]?\).*'`
        45 PYTHON_VERSION=`expr $(readlink ${thisdir}/bin/python) : '.*/Python/\([0-9]\.[0-9]\+\).*'`

        Maybe? Also the dot probably should be escaped? Try running this with the different python versions 3.9 and 3.11

        Edit to use proper operator https://ss64.com/bash/expr.html

        Edited by Andre Sailer
      • Please register or sign in to reply
      Please register or sign in to reply