Opened 7 months ago

Closed 6 months ago

#302 closed task (fixed)

I have updated WRC to work with rc files containing the WIDOW keyword I would like to get the RPM updated

Reported by: gyoung Owned by: diver
Priority: minor Milestone:
Component: rpm Version:
Severity: low Keywords:
Cc:

Description


Attachments (2)

wipfc-config-files-for-RPM.zip (12.4 KB) - added by gyoung 7 months ago.
wipfc-support-files.png (36.7 KB) - added by gyoung 6 months ago.

Download all attachments as: .zip

Change History (20)

comment:1 Changed 7 months ago by gyoung

The changes also updated wlink. So I would like to update that RPM also. The zip file containing them is too large to upload. I might be able to attach them individually here or I can upload them to netlabs incoming. I can upload them some where else if that is more convenient. Thanks

comment:2 Changed 7 months ago by diver

  • Owner set to diver
  • Status changed from new to accepted

please zip the changed exe and put it on netlabs.
of course also put the link here. so we will update the rpm later.

comment:4 Changed 7 months ago by diver

was/is there a reason you changed the name from wl.exe to wlink.exe? I also see no lvl file, which describes the changes.

comment:5 Changed 7 months ago by diver

and your build includes all fixed Steven did in 2014? According to the version I assume so, but I want to be sure.

comment:6 Changed 7 months ago by gyoung

The builds include Steven's code. The Watcom build system renames it to wlink but I now see you are packaging the static build. I will upload the static exes with the lvl files. Would you like to include the help files? Also it would be nice to package wipfc.exe as an RPM.

comment:7 Changed 7 months ago by diver

if you provide a static or non static build is not really an issue. But the name is, as gcc needs wl as the name.
I don't think we need the help files right now.

If you could add the needed files for wipfc, I add it to the rpm.
see also ticket:202

comment:8 Changed 7 months ago by gyoung

Hopefully I got all the files uploaded correctly. See ftp://ftp.netlabs.org/incoming/Open-watcom-wl-wrc-wipfc-for-RPM-packages.zip

comment:9 Changed 7 months ago by diver

I rebuilt the rpm and uploaded it to exp repo. please test

comment:10 Changed 7 months ago by gyoung

The wl and wrc packages appear to work fine. The wipfc package is incomplete (my bad). It requires a set of configuration files. These files are found using an environmental variable in a standard Open Watcom install (SET WIPFC=U:\Watcom\Wipfc) I will add the files to the lvl file and upload them here. Do you need anything else for this addition?

Changed 7 months ago by gyoung

comment:11 Changed 7 months ago by gyoung

I have attached the configuration files for wipfc.

comment:12 Changed 6 months ago by diver

hmm are you really sure the files are not there? as the watcom-wipfc package installs the files and also adds a config.sys entry. Of course it needs a reboot after the install.
Could you please double check first?

comment:13 Changed 6 months ago by gyoung

Some of them are there and misnamed versions of others are there. Wipfc traps with the set included. Sometime between 2010 and 2014 these got changed. I didn't have anything to do with that change and wasn't aware it had occurred. The files attached here are the current versions. I have added a screenshot comparing the current OW install (left side) with what the RPM installs (right side).

Changed 6 months ago by gyoung

comment:14 Changed 6 months ago by diver

please update your installed watcom rpm's and retest.

comment:15 Changed 6 months ago by gyoung

It still traps but I know why. RPM puts the following line in config.sys:
SET WIPFC=X:\usr\share\os2\wipfc;
Wipfc.exe doesn't tolerate the ";" on the end of this line. Once I removed it everything works fine. Sorry for all the hassle with this. Thanks

comment:16 Changed 6 months ago by diver

ok I updated the rpm once more. please test on a system w/o set wipfc already in.

comment:17 Changed 6 months ago by gyoung

It now works as expected. Sorry for the delayed response. I have been traveling. Thanks

comment:18 Changed 6 months ago by diver

  • Resolution set to fixed
  • Status changed from accepted to closed

no problem. Thx for testing.

Note: See TracTickets for help on using tickets.