Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Win32 fixes #597

Merged
merged 2 commits into from
Feb 16, 2023
Merged

Win32 fixes #597

merged 2 commits into from
Feb 16, 2023

Conversation

jamsilva
Copy link
Contributor

@jamsilva jamsilva commented Feb 6, 2023

Well detailed description of the change :

Corrected some errors in compilation for Win32. Ability to compile Win32 version on Linux using MinGW.
Modified linenoise with support for Win32 and additional features not available in the currently included linenoise (Such as Ctrl-R history search).

Type of change:

  • Bug fix
  • New feature

@Jason2605
Copy link
Member

Thanks for this! I have a pretty busy week this week but will try my best to get round to them at the weekend, sorry!

@Jason2605
Copy link
Member

Just as a quick check, did you replace the linenoise files completely? Just as the linenoise file within Dictu does have a couple of custom additions to it

@jamsilva
Copy link
Contributor Author

Yes, I replaced it with a different fork. I did not notice any problems at compile-time but if you can tell me what custom additions you made, I can look into it.

@Jason2605
Copy link
Member

Actually sorry ignore me I'm talking rubbish, this looks great thank you very much!

@Jason2605 Jason2605 merged commit 0ea4647 into dictu-lang:develop Feb 16, 2023
Jason2605 added a commit that referenced this pull request Feb 16, 2023
Removed option to build without linenoise (depends on #597)
@Jason2605 Jason2605 mentioned this pull request Feb 17, 2023
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