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

appveyor tests: miss creation of non-wide non-utf8 #249

Open
GitMensch opened this issue Nov 6, 2022 · 2 comments
Open

appveyor tests: miss creation of non-wide non-utf8 #249

GitMensch opened this issue Nov 6, 2022 · 2 comments

Comments

@GitMensch
Copy link
Collaborator

Note: I haven't checked if this is part of the cmake definitions used (so just needs to be added to appveyor.yml) or are missing there, too.

Note: we may miss a MinGW GCC build in the cmake definitions and/or its use in appveyor.yml, too.

@Bill-Gray
Copy link
Owner

We do compile the OS/2 and DOS ports with 8-bit characters, so I don't think we've really left this completely unchecked. Theoretically speaking, at least, we could have compiles with

(1) 8-bit characters, (2) wide chars, (3) forced UTF8;
(1) 32-bit chtypes vs. (2) 64-bit chtypes
(1) DEBUG vs. (2) non-DEBUG
(1) DLL vs. static

Most platform support at least two different compilers (gcc and MSVC, or gcc and clang, or Digital Mars and Borland and Watcom 1.9 and Watcom 2.0)

...and that's for DOS, DOSVGA, fb, the DRM variant of fb, OS/2, Plan 9, SDL1 and 2, VT (DOS, Windows, and Linux), WinCon, WinGUI, and X11. Which would be roughly 3x2x2x2x2x14 = 672 possible compilation tests. (Kinda thought it might turn out to be more than that.)

I do think we should have at least a few tests of, for example, 32-bit chtypes (I don't think we have any right now), and there may be other variants that would be good stress tests. I'd hesitate to test everything, though.

@GitMensch
Copy link
Collaborator Author

I do think we should have at least a few tests of, for example, 32-bit chtypes (I don't think we have any right now), and there may be other variants that would be good stress tests. I'd hesitate to test everything, though.

Given that huge number I totally agree.

If possible I'd suggest to add the following for Windows, both to have tests and to have usable artifacts:

  • MinGW 32bit with 32bit chtype, non-wide
  • MinGW 32bit with 64bit chtype, non-wide
  • MinGW 32bit with 32bit chtype, utf8
  • MinGW 32bit with 64bit chtype, utf8

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

No branches or pull requests

2 participants