Opened 6 years ago

Closed 6 years 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: Gregg Young Owned by: Silvan Scherrer
Priority: minor Milestone:
Component: rpm Version:
Severity: low Keywords:
Cc:

Description


Attachments (2)

wipfc-config-files-for-RPM.zip (12.4 KB) - added by Gregg Young 6 years ago.
wipfc-support-files.png (36.7 KB) - added by Gregg Young 6 years ago.

Download all attachments as: .zip

Change History (20)

comment:1 Changed 6 years ago by Gregg Young

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 6 years ago by Silvan Scherrer

Owner: set to Silvan Scherrer
Status: newaccepted

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:3 Changed 6 years ago by Gregg Young

comment:4 Changed 6 years ago by Silvan Scherrer

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 6 years ago by Silvan Scherrer

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 6 years ago by Gregg Young

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 6 years ago by Silvan Scherrer

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 6 years ago by Gregg Young

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 6 years ago by Silvan Scherrer

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

comment:10 Changed 6 years ago by Gregg Young

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 6 years ago by Gregg Young

comment:11 Changed 6 years ago by Gregg Young

I have attached the configuration files for wipfc.

comment:12 Changed 6 years ago by Silvan Scherrer

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 years ago by Gregg Young

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 years ago by Gregg Young

Attachment: wipfc-support-files.png added

comment:14 Changed 6 years ago by Silvan Scherrer

please update your installed watcom rpm's and retest.

comment:15 Changed 6 years ago by Gregg Young

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 years ago by Silvan Scherrer

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

comment:17 Changed 6 years ago by Gregg Young

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

comment:18 Changed 6 years ago by Silvan Scherrer

Resolution: fixed
Status: acceptedclosed

no problem. Thx for testing.

Note: See TracTickets for help on using tickets.