Skip to content
Snippets Groups Projects

Fix PyEval_InitThreads() and PyEval_ThreadsInitialized() deprecated warnings . Part II

All threads resolved!

Same changeset as !45559 (merged)

In all devLCG4 releases we see warnings

cvmfs/sft.cern.ch/lcg/releases/Python/3.9.5-82945/x86_64-centos7-gcc8-opt/include/python3.9/ceval.h:129:36: note: declared here
 Py_DEPRECATED(3.9) PyAPI_FUNC(int) PyEval_ThreadsInitialized(void);
                                    ^~~~~~~~~~~~~~~~~~~~~~~~~
/build/atnight/localbuilds/nightlies/master--dev4LCG/athena/Control/AthenaServices/src/initPyInterpreter.cxx:66:34: warning: 'int PyEval_ThreadsInitialized()' is deprecated [-Wdeprecated-declarations]
   if (!PyEval_ThreadsInitialized()) {

note that : https://docs.python.org/3/whatsnew/3.9.html

The PyEval_InitThreads() and PyEval_ThreadsInitialized() functions are now deprecated and will be removed in Python 3.11. Calling PyEval_InitThreads() now does nothing. The GIL is initialized by Py_Initialize() since Python 3.7. (Contributed by Victor Stinner in bpo-39877.)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Removing L1-pending label until discussions are resolved. Cheers lino (L1)

  • Christos Anastopoulos resolved all threads

    resolved all threads

  • added 1 commit

    Compare with previous version

  • This merge request affects 1 package:

    • HLT/Trigger/TrigControl/TrigPSC

    This merge request affects 1 file:

    • HLT/Trigger/TrigControl/TrigPSC/src/Psc.cxx

    Adding @bernius ,@wiedenma ,@rbielski ,@fwinkl as watchers

  • :white_check_mark: CI Result SUCCESS (hash 59ed72a3)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 37691]

  • Changes look fine and the CI is good. Approving. Clara (L1)

  • added review-approved label and removed review-pending-level-1 label

  • Walter Lampl mentioned in commit 11a75e63

    mentioned in commit 11a75e63

  • merged

  • Please register or sign in to reply
    Loading