Hi all, may I suggest that we put a
#ifdef HAVE_CONFIG_H # include "config.h" #endif
ahead of a few strategic platforms (and src) files so as to rescue the linux build? Currently, the builds fail because of #error directive in config.h...
For some files that already include sqConfig.h, just moving sqConfig.h ahead generally also solves the problem...
I've tried it with some success...
Hai
On 21. Apr 2021, at 20:37, Nicolas Cellier nicolas.cellier.aka.nice@gmail.com wrote:
Hi all, may I suggest that we put a
#ifdef HAVE_CONFIG_H # include "config.h" #endif
ahead of a few strategic platforms (and src) files so as to rescue the linux build? Currently, the builds fail because of #error directive in config.h...
I put it there because features.h depends on config.h
For some files that already include sqConfig.h, just moving sqConfig.h ahead generally also solves the problem...
I'm open to putting sqconfig.h back where it was, but it seems it broke the development of some windows features. It seems thats why https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/aafcb78371c7e576073... exists, which moved the .h's around.
So you should wait for Eliot before moving on here, as he explicitly asks: "I ask anyone wishing to alter the system here to please contact me and discuss before making changes. "
Best regards -Tobias
I've tried it with some success...
Hi Tobias, yes I remember this message, thanks for the links. My intention was to open a branch for fixing the issue and ask permission before merging.
I'm pretty sure that the right thing would be to let the CI tell whether we broke the MSVC/LLVM windows build or not. That would be more efficient than requesting review from single person for every single change. Maybe someone can help Eliot setting this up.
Constantly red builds cannot sustainably scale anyway. It would be great to integrate epoll fixes from David before they rot for example.
Nicolas
Le mer. 21 avr. 2021 à 21:03, Tobias Pape Das.Linux@gmx.de a écrit :
Hai
On 21. Apr 2021, at 20:37, Nicolas Cellier nicolas.cellier.aka.nice@gmail.com wrote:
Hi all, may I suggest that we put a
#ifdef HAVE_CONFIG_H # include "config.h" #endif
ahead of a few strategic platforms (and src) files so as to rescue the linux build? Currently, the builds fail because of #error directive in config.h...
I put it there because features.h depends on config.h
For some files that already include sqConfig.h, just moving sqConfig.h ahead generally also solves the problem...
I'm open to putting sqconfig.h back where it was, but it seems it broke the development of some windows features. It seems thats why https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/aafcb78371c7e576073... exists, which moved the .h's around.
So you should wait for Eliot before moving on here, as he explicitly asks: "I ask anyone wishing to alter the system here to please contact me and discuss before making changes. "
Best regards -Tobias
I've tried it with some success...
On 21. Apr 2021, at 21:35, Nicolas Cellier nicolas.cellier.aka.nice@gmail.com wrote:
Constantly red builds cannot sustainably scale anyway.
true. -t
vm-dev@lists.squeakfoundation.org