[Patches] [ python-Patches-614055 ] OpenVMS patches

noreply@sourceforge.net noreply@sourceforge.net
Wed, 25 Sep 2002 06:02:10 -0700


Patches item #614055, was opened at 2002-09-24 23:34
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=614055&group_id=5470

Category: Core (C code)
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: Piéronne Jean-François (pieronne)
Assigned to: Nobody/Anonymous (nobody)
Summary: OpenVMS patches

Initial Comment:
The attach file contains the patches of the Core files necessary to build Python for OpenVMS.
All the patches used conditional compilation test on __VMS, so I may not  be disruptive for the initial 
code.

11 files have been updated:

Python/exceptions.c
Python/import.c
Modules/getbuildinfo.c
Modules/getpatch.c
Modules/main.c
Modules/posixmodule.c
Modules/pwdmodule.c
Modules/socketmodule.c
Modules/stringobject.c
Modules/_hotshot.c
include/pyerrors.h


Jean-François

----------------------------------------------------------------------

>Comment By: Martin v. Löwis (loewis)
Date: 2002-09-25 15:02

Message:
Logged In: YES 
user_id=21627

I have a number of comments and questions:

- What is the purpose of VMSerror? It appears to be unused.
Can you use just OSError instead? If not, it should be VMSError.

- There are various formatting problems (indentation, spaces
after function names).

- Is it really necessary to compute the link time?

- Why do you need psxmod_gat_psxpath? Copying into
psxmod_gt_psxpath appears to do the same thing?

- what is the purpose of psxmod_to_vms_action?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=614055&group_id=5470