Skip to content

Force tcl files to have lf line endings inside tclkit. #2

Open
wtschueller wants to merge 1 commit into
rkeene:masterfrom
wtschueller:master
Open

Force tcl files to have lf line endings inside tclkit. #2
wtschueller wants to merge 1 commit into
rkeene:masterfrom
wtschueller:master

Conversation

@wtschueller

Copy link
Copy Markdown

Otherwise, fcopy would produce crlf if on Windows.

Files may be read directly using mk4tcl/uplevel commands without automatic <newline> substitution.
<newline> occur inside mk4vfs.tcl. The resulting error is silently suppressed, but leads to a not
functional access to the vfs like the one decribed here http://kitcreator.rkeene.org/fossil/tktview?name=3428b6ae06

…py would produce crlf if on Windows.

Files may be read directly using mk4tcl/uplevel commands without automatic \<newline> substitution.
\<newline> occur inside mk4vfs.tcl. The resulting error is silently suppressed, but leads to a not
functional access to the vfs like the one decribed here http://kitcreator.rkeene.org/fossil/tktview?name=3428b6ae06
@mayd

mayd commented Jul 28, 2023

Copy link
Copy Markdown

I had this exact problem building Tclkits with MSYS2/MINGW on Windows. This patch worked for me.

@mtmcp

mtmcp commented Dec 14, 2023

Copy link
Copy Markdown

This fix worked for me as well using MSYS2 UCRT64.

@rkeene please merge this in when you are able. Or if you'd like another approach on Fossil please let me know and I'd be happy to submit a patch there. Thank you!

@wtschueller Thank you for fixing this issue it would have taken a while for me to track it down otherwise.

@mtmcp

mtmcp commented Dec 15, 2023

Copy link
Copy Markdown

@rkeene I have included the fix with the drive-by patch associated with the issue TclKits with Tk targetting Windows blurry due to improper high-dpi scaling:
https://kitcreator.rkeene.org/fossil/tktview/90ea2a3a93

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants