# The form data below was edited by lester_cheung # Perforce Workshop Jobs # # Job: The job name. 'new' generates a sequenced job number. # Status: Job status; [open/closed/suspended]. Required # Project: The project this job is for. Required. # Severity: [A/B/C] (A is highest) Required. # ReportedBy The user who created the job. Can be changed. # ReportedDate: The date the job was created. Automatic. # ModifiedBy: The user who last modified this job. Automatic. # ModifiedDate: The date this job was last modified. Automatic. # OwnedBy: The owner, responsible for doing the job. Optional. # Description: Description of the job. Required. # DevNotes: Developer's comments. Optional. # Type: Type of job; [Bug/Feature]. Required. Job: job000033 Status: open Project: lester-cheung-p4review Severity: B ReportedBy: lester_cheung ReportedDate: 2013/10/13 21:12:59 ModifiedBy: lester_cheung ModifiedDate: 2015/07/16 17:35:16 Description: [p4review2] Python 3 support. Currently p4review2.py requires Python 2.7. Converting to Python 3 is tricky because we are currently supporting non-ASCII characters from a non-unicode-enabled server. One possible solution is to work with bytes (raw bytes) directly and avoid the automatic unicode conversion of Python 3. DevNotes: lester_cheung: Also need to handle all dirty data coming from the server. For example, umlauted chars in filenames, job names etc.. 2015/0717 lester_cheung: We also need to address module/class changes as pointed out by @riesman in https://swarm.workshop.perforce.com/changes/15011#comments Type: Feature