Skip to content

Comments

Rm win32 sgtty take 2#213

Open
n1ywb-1 wants to merge 10 commits intojrincayc:masterfrom
n1ywb-1:rm-win32-sgtty
Open

Rm win32 sgtty take 2#213
n1ywb-1 wants to merge 10 commits intojrincayc:masterfrom
n1ywb-1:rm-win32-sgtty

Conversation

@n1ywb-1
Copy link
Contributor

@n1ywb-1 n1ywb-1 commented Mar 25, 2025

Redo on #208

remind me not to pr off my master branch

n1ywb-1 added 10 commits March 25, 2025 02:39
This avoids mutating libloc.c during the windows build which avoids accidentally comitting the mutated file while developing on windows.

Tested on windows 11.
sgtty.h non-termios stuff
non-ioctl longjmp stuff

as a side-effect this fixes Linux no-graphics builds

that probably helps with jrincayc#92
also helps with jrincayc#203

tested on linux mint 22 with and without WX

X11 builds remain broken
tested on windows 11
Copy link
Owner

@jrincayc jrincayc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some comments.

git.c
logo_win.res
.vscode/browse.*
.*
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, I think I would rather add .????* than private space unicode characters.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or maybe .[!:alnum:][!:alnum:][!:alnum:][!:alnum:]* ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have a strong opinion

@@ -0,0 +1,111 @@
{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am hesitant to approve adding this files. However, I think I would be fine with adding a defaults directory or similar with them inside, so people could just copy them into the main directory.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That works for me. It needs documentation to be really useful anyway.

I was thinking of asking if we could add a contrib directory for stuff like that.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

contrib sounds fine to me.

@jrincayc jrincayc mentioned this pull request Aug 26, 2025
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.

2 participants