Opened 7 years ago

Closed 6 years ago

#50 closed defect (fixed)

RPM package attemps to change config.sys in C:

Reported by: guzzi Owned by:
Priority: minor Milestone: 0.8.9
Component: odin Version: 0.8.4
Severity: low Keywords:
Cc:

Description

My boot partition is E: There isn't even a C: partition on this system. Is it even necessary to change the config.sys? It looks like the .lib files were installed in usr\lib and the .exe. files in usr\bin, which are already in the path.

Installing : odin-win32k-0.7.1-3.i386 14 18

NOTE:

The file 'C:\config.sys' has been changed. You need to reboot your computer in order to activate these changes.

Installing : odin-exe-tools-0.7.1-3.i386 2/3

Installing : odin-doc-0.7.1-3.noarch 3/3

odin-doc-0.7.1-3.noarch was supposed to be installed but is not! odin-exe-tools-0.7.1-3.i386 was supposed to be installed but is not! odin-win32k-0.7.1-3.i386 was supposed to be installed but is not!

Installed:

odin-doc.noarch 0:0.7.1-3 odin-exe-tools.i386 0:0.7.1-3

odin-win32k.i386 0:0.7.1-3

Complete!

Change History (10)

comment:1 Changed 7 years ago by dmik

  • Priority changed from major to Feedback Pending

Since you installed win32k, your config.sys should have been changed indeed (win32k.sys needs a DEVICE statement). If it was not, it's an RPM bug. Are you sure your boot drive is E: and not C:?

comment:2 Changed 7 years ago by guzzi

Very sure. I don't even have a C: partition. I just noticed that there is a DEVICE=f:\usr\lib\win32k.sys in config.sys on E:\ . Looks like it's a small defect in RPM not displaying the right drive letter in the message.

comment:3 follow-up: Changed 7 years ago by dmik

Hmm, f:\usr\... or E:\usr\...?

comment:4 in reply to: ↑ 3 Changed 7 years ago by guzzi

Replying to dmik:

Hmm, f:\usr\... or E:\usr\...?

The unix tree is in F:, boot drive is E:. so, f:\usr\ is correct

comment:5 Changed 7 years ago by dmik

Okay, I got the problem. The boot drive is expanded at RPM generation time, not at installation time. As a result, you got my boot drive. The trick must be that the tool we use for changing config.sys (cube) is smart enough to pick up the right config sys despite the fact that "C:\config.sys" is given. This doesn't relate to the printed message of course.

We will have to fix that in the next version of Odin RPM. Thanks for reporting.

comment:6 Changed 7 years ago by diver

  • Milestone changed from general enhancement to next

comment:7 Changed 7 years ago by dmik

  • Milestone changed from next to 0.8.4

comment:8 Changed 6 years ago by diver

  • Priority changed from Feedback Pending to minor
  • Version set to 0.8.4

comment:9 Changed 6 years ago by dmik

  • Milestone changed from next to 0.8.9

comment:10 Changed 6 years ago by dmik

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

This was fixed in r 330 in the rpm/spec repo. It will now say the phrase "The boot configuration has been changed..." instead of referring to the config.sys file.

Note: See TracTickets for help on using tickets.