Skip to content

BSD Baby!#50

Merged
japanoise merged 7 commits intojapanoise:mainfrom
nicholascarroll:main
Jul 11, 2025
Merged

BSD Baby!#50
japanoise merged 7 commits intojapanoise:mainfrom
nicholascarroll:main

Conversation

@nicholascarroll
Copy link
Contributor

@nicholascarroll nicholascarroll commented Jul 9, 2025

  • Make no longer autodetects platform
    • for android and darwin you have to specify eg make android
    • for solaris and MSYS2 the targets can be specified but actually just make does work.
    • removed platform.h
  • Bugfix E.windows init for BSD. Which was causing the segfault
  • Resolved various compiler warnings
  • Confirmed build without warnings on Oracle Redhat, Alpine, Ubuntu, OSX, OpenBSD, Slackware, MSYS2, Android. Some warnings on Solaris which can be ignored.
  • Fixed VMIN/VEOF slot sharing issue affecting Solaris
  • Fixed an Android bug.

The Makefile can be further improved in a future commit: more POSIX compliant would be write "emsys" instead of $(PROGNAME). And may be able to eliminate need for separate build targets per platform. I would want to get it compiling without warnings on AIX first and then the Makefile itself will tell the full story.

What does this change?

Testing done

Did some basic edits like typing , moving cursor, mark, open file, save file.

Yeah Baby!

  - Make no longer autodetects platform
    - for android and darwin you have to specify
    - removed platform.h
  - Bugfix E.windows init for BSD. Which was causing the segfault
  - Resolved various compiler warnings
  - Confirmed build without warnings on Oracle Redhat, Alpine, Ubuntu, OSX,
    OpenBSD, Slackware.
Copy link
Owner

@japanoise japanoise left a comment

Choose a reason for hiding this comment

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

Great job! Some minor cleanup here which I'll take care of myself.

@japanoise japanoise merged commit 44a159f into japanoise:main Jul 11, 2025
7 checks passed
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