https://bugs.winehq.org/show_bug.cgi?id=49439
Bug ID: 49439 Summary: Printing and scanning software for Brother L2540DW fails to install (VBScript parser can't handle a "case else" statement with a comment and no statements to execute) Product: Wine Version: 5.11 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: vbscript Assignee: wine-bugs@winehq.org Reporter: alexhenrie24@gmail.com Distribution: ---
Created attachment 67536 --> https://bugs.winehq.org/attachment.cgi?id=67536 WINEDEBUG=+vbscript
To reproduce:
1. Run `winetricks mfc42`
2. Run the Brother Printing Device setup program
3. When asked for the connection type, select "Wired Network Connection (Ethernet)" and be sure to check "Custom Install"
4. When asked how to locate the printer, type in the IP address 127.0.0.1 (this is just for debugging, any IP address is fine)
After that, a message pops up that says "An error occurred during the installation" and the software to scan a document over the network is not installed.
Console output with WINEDEBUG=+vbscript is attached which indicates failure parsing a VBScript file. I dumped the full text of the file and this is the problematic part:
select case lcase(PortType)
case "raw" oPort.Protocol = 1
case "lpr" oPort.Protocol = 2
case else
'
' PutInstance will attempt to get the configuration of
' the device based on its IP address. Those settings
' will be used to add a new port
' end select
oPort.HostAddress = oParamDict.Item(kHostAddress)
It seems that Wine's VBScript parser can't handle a "case else" clause that contains nothing but a comment.
$ sha256sum DCP-L2540DW-inst-C1-US.EXE 29d58844815a030ae22297363dbc707b887da3d742b0a60de17e4c610c5bd31c
https://bugs.winehq.org/show_bug.cgi?id=49439
Alex Henrie alexhenrie24@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, hardware URL| |https://support.brother.com | |/g/b/downloadhowto.aspx?c=u | |s&lang=en&prod=dcpl2540dw_u | |s_as&os=92&dlid=dlf100994_0 | |00&flang=8&type3=11
https://bugs.winehq.org/show_bug.cgi?id=49439
Molk dimaki@rocketmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dimaki@rocketmail.com
--- Comment #1 from Molk dimaki@rocketmail.com --- Empty "case else" parsing is now fixed by the commit below which allows the installer to proceed further before running into a different issue.
https://source.winehq.org/git/wine.git/?a=commit;h=18e7e07a77a754c4b48c6c9bd...
https://bugs.winehq.org/show_bug.cgi?id=49439
Alex Henrie alexhenrie24@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |18e7e07a77a754c4b48c6c9bde2 | |25ad5c7295944 Resolution|--- |FIXED Status|NEW |RESOLVED
--- Comment #2 from Alex Henrie alexhenrie24@gmail.com --- Thank you!
https://bugs.winehq.org/show_bug.cgi?id=49439
--- Comment #3 from Dmitry Kislyuk dimaki@rocketmail.com --- Glad I could be helpful with this one!
https://bugs.winehq.org/show_bug.cgi?id=49439
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #4 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 5.12.
https://bugs.winehq.org/show_bug.cgi?id=49439
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |5.0.x
https://bugs.winehq.org/show_bug.cgi?id=49439
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|5.0.x |---
--- Comment #5 from Michael Stefaniuc mstefani@winehq.org --- Removing the 5.0.x milestone from bug fixes included in 5.0.3.