Opened 7 years ago

Closed 7 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 7 years ago.
wipfc-support-files.png (36.7 KB ) - added by Gregg Young 7 years ago.

Download all attachments as: .zip

Change History (20)

comment:1 by Gregg Young, 7 years ago

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

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

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

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

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

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

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

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

comment:10 by Gregg Young, 7 years ago

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?

by Gregg Young, 7 years ago

comment:11 by Gregg Young, 7 years ago

I have attached the configuration files for wipfc.

comment:12 by Silvan Scherrer, 7 years ago

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

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).

by Gregg Young, 7 years ago

Attachment: wipfc-support-files.png added

comment:14 by Silvan Scherrer, 7 years ago

please update your installed watcom rpm's and retest.

comment:15 by Gregg Young, 7 years ago

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

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

comment:17 by Gregg Young, 7 years ago

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

comment:18 by Silvan Scherrer, 7 years ago

Resolution: fixed
Status: acceptedclosed

no problem. Thx for testing.

Note: See TracTickets for help on using tickets.