[Bug 16511] New: Google Sketchup rejects some ruby scripts containing CRLF
http://bugs.winehq.org/show_bug.cgi?id=16511 Summary: Google Sketchup rejects some ruby scripts containing CRLF Product: Wine Version: 1.1.10 Platform: Other URL: http://sketchup.google.com OS/Version: other Status: NEW Keywords: download Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs(a)winehq.org ReportedBy: dank(a)kegel.com Oddly, under Wine, the example ruby scripts from http://sketchup.google.com/download/rubyscripts.html are rejected by Sketchup unless you remove the CRs, i.e. run them through dos2unix. The one-liner "puts 'hello'" program is ok, though. The smallest script I've created that exibits the problem is a hello, world app that starts with a ten line comment. This script does not cause the error when run in Sketchup on MS Windows. I'll attach the script. -- 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=16511 Dan Kegel <dank(a)kegel.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |msvcrt --- Comment #1 from Dan Kegel <dank(a)kegel.com> 2008-12-15 08:34:44 --- Native msvcrt works around this (winetricks vcrun6, then add override for msvcrt in winecfg) so changing category. -- 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=16511 --- Comment #2 from Dan Kegel <dank(a)kegel.com> 2008-12-15 11:13:01 --- Created an attachment (id=17965) --> (http://bugs.winehq.org/attachment.cgi?id=17965) Patch that seems to fix the problem In ascii mode, if we decide we need one more byte, it looks like sometimes we grab it from the file instead of the buffer! The attached patch fixes this by continuing to use buffered data longer. -- 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=16511 Dan Kegel <dank(a)kegel.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch --- Comment #3 from Dan Kegel <dank(a)kegel.com> 2008-12-15 11:13:27 --- I'll submit the patch once I have a conformance test. -- 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=16511 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #4 from Austin English <austinenglish(a)gmail.com> 2008-12-16 09:08:46 --- Patch was committed. -- 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=16511 --- Comment #5 from Dan Kegel <dank(a)kegel.com> 2008-12-16 09:59:29 --- And it even works, too. Bonus :-) -- 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=16511 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #6 from Alexandre Julliard <julliard(a)winehq.org> 2008-12-20 09:06:46 --- Closing bugs fixed in 1.1.11. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=16511 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |focht(a)gmx.net Summary|Google Sketchup rejects |Google Sketchup 7 rejects |some ruby scripts |some ruby scripts |containing CRLF |containing CRLF OS|other |Linux URL|http://sketchup.google.com |https://web.archive.org/web | |/20120726023942/http://dl.g | |oogle.com/sketchup/gsu7/PW- | |2-1-6860-EN.exe Fixed by SHA1| |72ffe1da676b673f6caa84f1859 | |a601f6935dd99 Hardware|Other |x86 -- 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 (2)
-
wine-bugs@winehq.org -
WineHQ Bugzilla