Project

General

Profile

Bug #23314

make client compatible with both python2 and python3

Added by Dennis Box 8 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
09/23/2019
Due date:
% Done:

0%

Estimated time:
First Occurred:
Occurs In:
Stakeholders:
Duration:

Description

Many experiments are starting to test python 3 and need a jobsub_client that supports this.

We need to continue to support python2 for some time as well. There are libraries (six) and a body of knowledge dedicated to writing python that runs correctly under both language branches, see https://python-future.org/compatible_idioms.html

Move as much of the client code as possible to dual compatibility

History

#1 Updated by Dennis Box 7 months ago

  • Target version changed from v1.3 to v1.3.1

#2 Updated by Dennis Box 6 months ago

  • Assignee changed from Dennis Box to Shreyas Bhat
  • Status changed from New to Feedback

please see and review for branch 23314
This code is also the current 'test' ups jobsub_client, v1_3_1_rc02

I have tested this branch/ups product for both python2.7 and python3.6. The directions for installing python rpms in the release notes must be followed for it to work.

#3 Updated by Shreyas Bhat 6 months ago

  • Assignee changed from Shreyas Bhat to Dennis Box
  • Status changed from Feedback to Under Discussion

Looks good.

With the update to all the print statements (for example), do we also want to update the string formatting statements?

e.g. '"Hello %s" % "world"' becomes '"Hello {}".format("world")'

Or should that be a separate ticket that we do some other time?

#4 Updated by Dennis Box 5 months ago

  • Status changed from Under Discussion to Accepted

#5 Updated by Dennis Box about 2 months ago

  • Status changed from Accepted to Feedback

#6 Updated by Dennis Box about 2 months ago

  • Status changed from Feedback to Closed


Also available in: Atom PDF