Opened 9 years ago

Closed 7 years ago

#29 closed enhancement (fixed)

WINXP version to SP3

Reported by: abwillis Owned by:
Priority: minor Milestone: 0.8.6
Component: odin Version: 0.8.5
Severity: low Keywords:
Cc:

Description

There was some chatter on OS2World about updating the reported version of WinXP in Odin to SP3. I have run into an app that wanted that as well, though so far increasing the version only got rid of the message that I needed SP3 and did not fix the problem or I would just commit the change.

Index: version.cpp =================================================================== --- version.cpp (revision 21552) +++ version.cpp (working copy) @@ -88,14 +88,14 @@

2, 0, 0, 0

}

},

  • Windows XP

+ Windows XP (SP3)

{

0x05005F03, /* Assuming DOS 5 like the other NT */ 0x0A280105, {

sizeof(OSVERSIONINFOA), 5, 1, 0xA28,

  • VER_PLATFORM_WIN32_NT, "",
  • 0, 0, 0, 0

+ VER_PLATFORM_WIN32_NT, "Service Pack 3", + 3, 0, 0, 0

}

}

};

Change History (8)

comment:1 Changed 9 years ago by diver

  • Milestone set to general enhancement

comment:2 Changed 8 years ago by diver

  • Severity set to low

higher the Win version to XP SP3 could make sense anyway.

comment:3 Changed 7 years ago by diver

  • Milestone changed from general enhancement to next
  • Version set to 0.8.5

we should decide on that for the next odin version

comment:4 Changed 7 years ago by diver

  • Milestone changed from next to 0.8.6

comment:5 Changed 7 years ago by dmik

Well, we may do that of course but I'm not sure if our current Odin implements all additions SP3 should have.

But we can still do it. Since right now there are still some unimplemented functions that must be present in NT strictly speaking.

comment:6 Changed 7 years ago by abwillis

It hasn't seemed to hurt anything that I can tell but we could add it as an option instead of a replacement as I have currently so that we could choose XP or XP sp3. I don't know that it would be helpful for that or not and would require quite a bit more to do that.

comment:7 Changed 7 years ago by dmik

I think that we should provide a parameter in ODIN.INI that sets the default Windows version for the Odin runtime and add command line switches for PE/PEC to override it for a specific application. For cases when Odin is used from OS/2 applications (like Java or Mozilla), there is an OdinSetVersion? API (each application may hard code a particular version and also provide its own ways of configure if needed).

The first two things are easily doable. I won't mind if you implement them -)

comment:8 Changed 7 years ago by dmik

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

Applied the patch (with important corrections from Silvan) in r22030. Made XP SP3 the default Windows version in r22031. Closing this ticket.

I'm ready for the release and will do it either later today or tomorrow morning.

Note: See TracTickets for help on using tickets.