[Bug 25963] New: Dead Space 2 dvd installer fails to switch discs
http://bugs.winehq.org/show_bug.cgi?id=25963 Summary: Dead Space 2 dvd installer fails to switch discs Product: Wine Version: 1.3.12 Platform: x86 OS/Version: Linux Status: NEW Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs(a)winehq.org ReportedBy: dank(a)kegel.com The first disc starts installing nicely, but installation aborts after a few minutes. This happens regardless of whether you use physical discs or just mount iso's. Here's a bit of +msi, suitably trimmed: trace:msi:msi_get_property returning L"I:\\" for property L"SourceDir" trace:msi:resolve_file_source Working to resolve source of file L"DS2DAT4_DAT" trace:msi:resolve_folder Working to resolve L"INSTALLLOCATION" trace:msi:resolve_folder (source)already resolved to L"I:\\" trace:msi:resolve_file_source file L"DS2DAT4_DAT" source resolves to L"I:\\DS2DAT4.DAT" trace:msi:ACTION_InstallFiles copying L"I:\\DS2DAT4.DAT" to L"C:\\Program Files\\EA Games\\Dead Space 2\\DS2DAT4.DAT" ... err:msi:ACTION_InstallFiles Failed to copy L"I:\\DS2DAT4.DAT" to L"C:\\Program Files\\EA Games\\Dead Space 2\\DS2DAT4.DAT" (2) err:msi:ITERATE_Actions Execution halted, action L"InstallFiles" returned 1603 ds2dat4.dat is on the second disc. Dunno why it didn't try to change discs...? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 Dan Kegel <dank(a)kegel.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Installer -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 --- Comment #1 from Dan Kegel <dank(a)kegel.com> 2011-02-02 09:10:06 CST --- Copying contents of both discs to a directory on hard drive lets you work around this, and the installer then reports success. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |focht(a)gmx.net Component|-unknown |msi --- Comment #2 from Anastasius Focht <focht(a)gmx.net> 2011-10-14 16:18:52 CDT --- Hello, --- snip --- 0026:Call KERNEL32.CreateProcessW(00000000,0033d618 L"msiexec.exe /i \"D:\\autorun.dat\" TRANSFORMS=:en-us.mst",00000000,00000000,00000000,00000000,00000000,00000000,0033d5d4,0033d5c4) ret=0040fdd2 0026:Ret KERNEL32.CreateProcessW() retval=00000001 ret=0040fdd2 --- snip --- Not sure why the transform command line parameter for english is explicitly given, its the default and data is embedded in package. Maybe they wanted multi-language MSI package one day and prepared for that. The boostrapper has to decide which language the installer package should use. For example if german was decided, the parameter "TRANSFORMS=:de.mst" would be passed to msiexec, which directs it to apply the embedded transform named "de.mst"before running the package. Anyway ... dumping "autorun.dat" with Orca: "Component" table: --- snip --- Group0 {89D09903-93C2-45A4-9D70-D4D7444AA412} INSTALLLOCATION 0 MAINENTRYPOINTFILEID Group1 {95CC93FF-F788-498B-8776-BF7CB0D61151} INSTALLLOCATION 0 DS2DAT4_DAT --- snip --- "File" table: --- snip --- DS2DAT0_DAT Group0 DS2DAT0.DAT 2146199456 8704 5 ... MAINENTRYPOINTFILEID Group0 0zascoph.exe|deadspace2.exe 63504024 1.0.0.0 1033 8704 10 ... SupportEADMEadm_installer_exe SupportEADMEADM arfyp7im.exe|eadm-installer.exe 19086936 7.1.3.3 1033 8705 1322 DS2DAT4_DAT Group1 DS2DAT4.DAT 1653587968 8704 1323 DS2DAT5_DAT Group1 DS2DAT5.DAT 1733548032 8704 1324 moviesEnBloopers_vp6 moviesEnMoviesEn bloopers.vp6 200324 8704 1325 --- snip --- "Media" table: --- snip --- DiskId LastSequence DiskPrompt Cabinet VolumeLabel Source i2 i4 L64 S255 S32 S72 1 0 Disc1 2 10 Disc1 3 11 Disc1 4 12 Disc1 5 0 Disc1 6 0 Disc1 7 0 Disc1 8 0 Disc1 9 13 Disc1 10 1321 Disc1 11 1322 Disc1 12 0 Disc1 13 0 Disc1 14 1324 Disc2 15 1386 Disc2 16 1448 Disc2 --- snip --- The file in question "DS2DAT4_DAT" has sequence id 1323. The highest LastSequence id for "Disc1" is 1322. File "DS2DAT5.DAT" with sequence id 1324 which is also located on CD2 is referenced as LastSequence id in DiskId 14. MSDN: http://msdn.microsoft.com/en-us/library/windows/desktop/aa370558.aspx --- quote --- For example, suppose a file has a sequence number of 92 entered in the Sequence column of the File table. To determine on which source disk this file resides, the installer checks the record of the Media table for the entry with the smallest LastSequence value that is larger than 92. The DiskId column is the primary key for the Media table and this field uniquely identifies the disk in the table. --- quote --- 1323 -> 1324 -> Disc2 Regards -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Dead Space 2 dvd installer |Dead Space 2 dvd installer |fails to switch discs |fails to switch discs (no | |ready_media/msi_change_medi | |a for normal files) --- Comment #3 from Anastasius Focht <focht(a)gmx.net> 2011-10-15 07:35:03 CDT --- Hello, looking at the bug again, the sequence handling is fine. The problem is actually ACTION_InstallFiles() -> ready_media(). ACTION_InstallFiles: http://source.winehq.org/git/wine.git/blob/cda63b8a3820de7864c663ab79d47c431... ready_media: http://source.winehq.org/git/wine.git/blob/7b7c95425a8d23ca0a30756017018aca3... ready_media() doesn't do anything if the source file is a _normal_ file. They put several large files "DS2DATx.DAT" on DVD1 and DVD2 to be simply copied to target folder. I refactored the code to allow volume id/media change detection also for normal files and it helped. The installer asked for second DVD and completed successfully. Regards -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 André H. <nerv(a)dawncrow.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW CC| |nerv(a)dawncrow.de --- Comment #4 from André H. <nerv(a)dawncrow.de> 2011-11-22 13:55:22 CST --- fixing status, i doubt that should be assigned to AF -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 --- Comment #5 from Hans Leidekker <hans(a)meelstraat.net> 2011-12-02 06:58:06 CST --- Created attachment 37743 --> http://bugs.winehq.org/attachment.cgi?id=37743 ~/msi_fix_changing_media_for_normal_files. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 --- Comment #6 from Hans Leidekker <hans(a)meelstraat.net> 2011-12-02 06:58:55 CST --- Please try this patch. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 --- Comment #7 from Austin English <austinenglish(a)gmail.com> 2011-12-23 15:08:10 CST --- http://source.winehq.org/git/wine.git/commitdiff/ad4f747fa68fe185854e6fbbc6f... -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |ad4f747fa68fe185854e6fbbc6f | |0fec62e762e06 Status|NEW |RESOLVED Resolution| |FIXED --- Comment #8 from Anastasius Focht <focht(a)gmx.net> 2011-12-27 06:50:32 CST --- Hello, yes it helped - partially. At least a dialog indicating the disk change is now shown. So this part of bug is fixed. Unfortunately the overall installer is still not fixed - that deserves a new bug. --- snip --- 0031:trace:msi:MSI_ProcessMessage (nil) (nil) (nil) 0 200 L"File: DS2DAT4.DAT, Directory: INSTALLLOCATION, Size: 1653587968" ... 0031:trace:msi:msi_get_property returning L"D:\\" for property L"SourceDir" 0031:Call shlwapi.PathStripToRootW(0033f0f4 L"D:\\") ret=683cc13b 0031:Ret shlwapi.PathStripToRootW() retval=00000001 ret=683cc13b 0031:Call shlwapi.PathAddBackslashW(0033f0f4 L"D:\\") ret=683cc14c 0031:Ret shlwapi.PathAddBackslashW() retval=0033f0fa ret=683cc14c 0031:Call KERNEL32.GetVolumeInformationW(0033f0f4 L"D:\\",0033f2fe,00000105,00000000,00000000,00000000,00000000,00000000) ret=683cc197 0031:Ret KERNEL32.GetVolumeInformationW() retval=00000001 ret=683cc197 ... 0031:trace:msi:MSI_DatabaseOpenViewW L"SELECT `Message` FROM `Error` WHERE `Error` = 1302" 0x33f478 ... 0031:trace:msi:MSI_DatabaseOpenViewW L"UPDATE `Control` SET `Text` = 'Please insert Dead Space\2122 2: Disc2' WHERE `Dialog_` = 'ErrorDlg' AND `Control` = 'ErrorText'" 0x33f038 ... 0031:Call user32.CreateWindowExW(00000000,68421f60 L"MsiDialogCloseClass",007ddeac L"ErrorDlg",10000000,80000000,80000000,80000000,80000000,00000000,00000000,00000000,007dde68) ret=683bc13c ... 0031:Call user32.CreateWindowExW(00000000,68421fce L"BUTTON",007de9a8 L"&No",50010000,000000c8,0000006b,0000004b,00000017,000300e6,00000000,00000000,00000000) ret=683b3222 ... 0031:Ret user32.CreateWindowExW() retval=000200f4 ret=683b3222 0031:trace:msi:msi_dialog_create_window Dialog L"ErrorDlg" control L"&No" hwnd 0x200f4 ... 0031:Call user32.CreateWindowExW(00000000,68421fce L"BUTTON",007deb70 L"&Yes",50010000,000000c8,0000006b,0000004b,00000017,000300e6,00000000,00000000,00000000) ret=683b3222 ... 0031:Ret user32.CreateWindowExW() retval=000200f0 ret=683b3222 0031:trace:msi:msi_dialog_create_window Dialog L"ErrorDlg" control L"&Yes" hwnd 0x200f0 ... 0031:trace:msi:msi_dialog_add_control L"Static", L"ErrorText", 00000003, L"Please insert Dead Space\2122 2: Disc2", 00020000 ... 0031:Ret user32.CreateWindowExW() retval=000200ee ret=683b3222 0031:trace:msi:msi_dialog_create_window Dialog L"ErrorDlg" control L"Please insert Dead Space\2122 2: Disc2" hwnd 0x200ee ... 0031:Call user32.CreateWindowExW(00000000,68421fce L"BUTTON",007deff8 L"Cancel",50010000,000000c8,0000006b,0000004b,00000017,000300e6,00000000,00000000,00000000) ret=683b3222 ... 0031:Ret user32.CreateWindowExW() retval=000200ea ret=683b3222 0031:trace:msi:msi_dialog_create_window Dialog L"ErrorDlg" control L"Cancel" hwnd 0x200ea ... 0031:Call user32.CreateWindowExW(00000000,68421fce L"BUTTON",007df3d8 L"&Ignore",50010000,000000c8,0000006b,0000004b,00000017,000300e6,00000000,00000000,00000000) ret=683b3222 ... 0031:Ret user32.CreateWindowExW() retval=000200d6 ret=683b3222 0031:trace:msi:msi_dialog_create_window Dialog L"ErrorDlg" control L"&Ignore" hwnd 0x200d6 ... 0031:Call user32.CreateWindowExW(00000000,68421fce L"BUTTON",007df6c8 L"OK",50010000,000000c8,0000006b,0000004b,00000017,000300e6,00000000,00000000,00000000) ret=683b3222 ... 0031:Ret user32.CreateWindowExW() retval=000200e4 ret=683b3222 0031:trace:msi:msi_dialog_create_window Dialog L"ErrorDlg" control L"OK" hwnd 0x200e4 ... 0031:Call user32.CreateWindowExW(00000000,68421fce L"BUTTON",007dfba8 L"&Retry",50010000,000000c8,0000006b,0000004b,00000017,000300e6,00000000,00000000,00000000) ret=683b3222 ... 0031:Ret user32.CreateWindowExW() retval=000200e2 ret=683b3222 0031:trace:msi:msi_dialog_create_window Dialog L"ErrorDlg" control L"&Retry" hwnd 0x200e2 ... 0031:Call window proc 0x6879aec7 (hwnd=0x200f4,msg=WM_LBUTTONUP,wp=00000000,lp=00090042) ... 0031:trace:msi:msi_dialog_oncommand 0x7dde68 0x200f4 00000000 ... 0031:trace:msi:MSI_EvaluateConditionW 1 <- L"1" 0031:trace:msi:msi_dialog_send_event Sending control event L"EndDialog" L"ErrorNo" ... 0031:trace:msi:msi_set_property 0x146750 L"MSIErrorDialogResult" L"ErrorAbort" ... 0031:trace:msi:msi_dialog_end_dialog 0x7dde68 ... 0031:err:msi:ACTION_InstallFiles Failed to ready media for L"DS2DAT4_DAT" ... 0031:trace:msi:MSI_ProcessMessage (nil) (nil) (nil) 0 10 L"Action ended 13:01:29: InstallFiles. Return value 1627." ... --- snip --- For next bug: The disk change dialog is somehow borked. It seems all buttons created as child below dialog parent are overlaying each other (same rectangle -> x,y,w,h). The last button "retry" text is shown. When you move the main installer main window over the disk change dialog, the button text cycles through all buttons text (each control is updated once). Moving foreign windows over the dialog will cause the "no" button to be active. Clicking the button will always use first button control, which is unfortunately "No", hence the installer aborts. Regards -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=25963 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #9 from Alexandre Julliard <julliard(a)winehq.org> 2011-12-30 12:57:10 CST --- Closing bugs fixed in 1.3.36. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
participants (1)
-
wine-bugs＠winehq.org