Closed Bug 337826 Opened 19 years ago Closed 19 years ago

CCK Wizard - Creation process after 'Finish now' failed (OS/2)

Categories

(Other Applications :: CCK, defect)

x86
OS/2
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ak120, Assigned: mkaply)

Details

Attachments

(1 file)

User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.8.0.2) Gecko/20060420 Firefox/1.5.0.2 Build Identifier: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.8.0.2) Gecko/20060420 Firefox/1.5.0.2 Running CCK Wizard with Firefox 1.5.0.2 for OS/2. The creation process beyond 'Finish now' failed. cck.config and a jar-directory are present. Reproducible: Always Steps to Reproduce: 1. XPI-Installation of CCK 2. Restarting Firefox 3. Launch CCK Wizard 4. Press 'Finish Now'-Button on the last screen Actual Results: file 'cck.config' and directory 'jar' are present in the right directory
(In reply to comment #0) > file 'cck.config' and directory 'jar' are present in the right directory are you sure there is an executable zip in your path or specified?
(In reply to comment #1) > (In reply to comment #0) > > file 'cck.config' and directory 'jar' are present in the right directory > > are you sure there is an executable zip in your path or specified? > YES - Zip 2.31
This is a bug in my code. I did a bad win32 check so OS/2 is getting a shell script instead of a CMD file. argh.
Status: UNCONFIRMED → NEW
Ever confirmed: true
I'm not sure I am going to be able to make this work on OS/2. I'm still thinking. Basically the problem is that on OS/2 you can't wait for a child process if the child process as started with DosStartSession (as is the case here) So when we try to do the zipping at the end, things return too quickly and files don't get created properly. I might end up having to completely special case OS/2...
Attached patch Ugly workaroundSplinter Review
In order to workaround the process bug on OS/2, I display message boxes. bleh.
Fix checked in.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: