June 2018 - Wine-Bugs - list.winehq.org

[Bug 44132] Luajit’s os.execut e() does not work well with quoted arguments

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=44132 Jason Edmeades <us(a)edmeades.me.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |us(a)edmeades.me.uk --- Comment #1 from Jason Edmeades <us(a)edmeades.me.uk> --- This doesnt work on windows either, and I believe needs to be resolved invalid. (The root cause of your problem based on the description of the mkdir issue is probably bug#40694 (just attached a patch) in that the "C:<space>&" fails) (Windows 10:) C:\luajit>luajit -e "print(os.execute(\"\\\"C:\\luajit\\luajit\\\" -e \\\"print(1)\\\"\"))" 'C:\luajit\luajit" -e "print' is not recognized as an internal or external command, operable program or batch file. nil exit 1 In effect this launches: cmd /c "C:\luajit\luajit.exe" -e "print(1)" Windows states (see cmd.exe /?): If /C or /K is specified, then the remainder of the command line after the switch is processed as a command line, where the following logic is used to process quote (") characters: 1. If all of the following conditions are met, then quote characters on the command line are preserved: - no /S switch - exactly two quote characters - no special characters between the two quote characters, where special is one of: &<>()@^| - there are one or more whitespace characters between the two quote characters - the string between the two quote characters is the name of an executable file. 2. Otherwise, old behavior is to see if the first character is a quote character and if so, strip the leading character and remove the last quote character on the command line, preserving any text after the last quote character. You hit condition 2, so the first and last quote are removed, and it tries to run the rest as a single command, which obviously fails. -- 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.

June 26, 2018

[Bug 40694] Invalid behavior of "C: && echo haha" or "C: & echo haha"

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=40694 --- Comment #13 from Jason Edmeades <us(a)edmeades.me.uk> --- Created attachment 61701 --> https://bugs.winehq.org/attachment.cgi?id=61701 patch -- 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.

June 26, 2018

[Bug 40694] Invalid behavior of "C: && echo haha" or "C: & echo haha"

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=40694 --- Comment #12 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- (In reply to Jason Edmeades from comment #11) I couldn't find any replies to the patch, so it seems it just got lost without review. -- 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.

June 26, 2018

[Bug 40617] Allow users to change passwords and more

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=40617 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fgouget(a)codeweavers.com Summary|Allow users to change |Allow users to change |passwords |passwords and more --- Comment #1 from François Gouget <fgouget(a)codeweavers.com> --- Users should also be allowed to change their email address (the current workaround is for them to ask a TestBot administrator), or, gasp, delete their account (again, ask an administrator). -- 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.

June 26, 2018

[Bug 40694] Invalid behavior of "C: && echo haha" or "C: & echo haha"

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=40694 --- Comment #11 from Jason Edmeades <us(a)edmeades.me.uk> --- Was there any follow on as to why it was rejected? Looking at it I dont like the " && !(strlenW(cmd) >= 3 && cmd[2] != ' '))" part I just was experimenting and came up with a similar (slightly cleaner) patch (attaching in a few mins). In the absence of any other comments, I'll send it next time I'm submitting patches. -- 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.

June 26, 2018

[Bug 32216] WineTestBot should skip some tests on some platforms

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=32216 Bug 32216 depends on bug 33065, which changed state. Bug 33065 Summary: Make it possible to build a winetest executable with a single test https://bugs.winehq.org/show_bug.cgi?id=33065 What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |WONTFIX -- 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.

June 26, 2018

[Bug 31609] TestLauncher says a required DLL is missing, but the test runs fine without TestLauncher

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=31609 Bug 31609 depends on bug 33065, which changed state. Bug 33065 Summary: Make it possible to build a winetest executable with a single test https://bugs.winehq.org/show_bug.cgi?id=33065 What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |WONTFIX -- 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.

June 26, 2018

[Bug 33065] Make it possible to build a winetest executable with a single test

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=33065 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |WONTFIX --- Comment #6 from François Gouget <fgouget(a)codeweavers.com> --- This issue was created when the goal was to eliminate TestLauncher. However this does not seem desirable or necessary anymore. See this wine-devel discussion for the rationale: https://www.winehq.org/pipermail/wine-devel/2017-April/117163.html Note that all the issues mentioned at the end of that email have been resolved, except for the dll check. And regarding that WineTest is actually worse (see bug 45032). So I'm closing this bug. -- 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.

June 26, 2018

[Bug 45032] WineTest does not run the vcomp tests

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=45032 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|vcomp |testcases --- Comment #5 from François Gouget <fgouget(a)codeweavers.com> --- Changed to component to testcases since the bug is in WineTest. -- 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.

June 26, 2018

[Bug 40694] Invalid behavior of "C: && echo haha" or "C: & echo haha"

by wine-bugs＠winehq.org

https://bugs.winehq.org/show_bug.cgi?id=40694 --- Comment #10 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- (In reply to Jason Edmeades from comment #9) Just for reference, there was another patch to fix this bug here: https://www.winehq.org/pipermail/wine-patches/2017-September/165017.html -- 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.

June 26, 2018