Maybe you can try cross-compilation.

https://github.com/mstorsjo/llvm-mingw/releases

If you do not have ubuntu 18.04, building llvm-mingw for you linux distro should be relatively simple....

It could be interesting experiment. Actually, we can even add a simple support in ide / umake to prepend binary with "wine" for execution...

Mirek

mirek wrote on Mon, 01 June 2020 17:32

Maybe you can try cross-compilation.

https://github.com/mstorsjo/llvm-mingw/releases

If you do not have ubuntu 18.04, building llvm-mingw for you linux distro should be relatively simple....

It could be interesting experiment. Actually, we can even add a simple support in ide / umake to prepend binary with "wine" for execution...

Mirek

First attempt is unsuccessful.
umk completely ignores all my attempts to set path to a different clang version.
I changed PATH and even put absolute path to clang into COMPILER.
Everything was ignored.

BTW, you put in PATH folders with dll-s ... What is that for?

Can you post .bm?

/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/Core/Ops.h:165:29: error: use of undeclared identifier 'HASH64_CONST3'
        return (dword)SwapEndian64(HASH64_CONST3 * h);

mirek wrote on Tue, 02 June 2020 11:40

Can you post .bm?

I fixed problem with clang++ by using x86_64-w64-mingw32-clang++ instead of it. Before that I was trying to use /home/ssg/.local/soft/llvm-mingw/llvm-mingw-20200325-ubuntu- 18.04/bin/clang++ as a compiler ... Plain clang++ didn't work by definition.

A new problem is that umk is passing POSIX and LINUX flags to x86_64-w64-mingw32-clang++ because it runs on Linux ...
Basically, this makes cross-compilation impossible with the current umk ...

Novo wrote on Tue, 02 June 2020 18:56

mirek wrote on Tue, 02 June 2020 11:40

Can you post .bm?

I fixed problem with clang++ by using x86_64-w64-mingw32-clang++ instead of it. Before that I was trying to use /home/ssg/.local/soft/llvm-mingw/llvm-mingw-20200325-ubuntu- 18.04/bin/clang++ as a compiler ... Plain clang++ didn't work by definition.

A new problem is that umk is passing POSIX and LINUX flags to x86_64-w64-mingw32-clang++ because it runs on Linux ...
Basically, this makes cross-compilation impossible with the current umk ...

Hm, we probably need to add flags override to .bm, right?

mirek wrote on Tue, 02 June 2020 15:07

Novo wrote on Tue, 02 June 2020 18:56

mirek wrote on Tue, 02 June 2020 11:40

Can you post .bm?

I fixed problem with clang++ by using x86_64-w64-mingw32-clang++ instead of it. Before that I was trying to use /home/ssg/.local/soft/llvm-mingw/llvm-mingw-20200325-ubuntu- 18.04/bin/clang++ as a compiler ... Plain clang++ didn't work by definition.

A new problem is that umk is passing POSIX and LINUX flags to x86_64-w64-mingw32-clang++ because it runs on Linux ...
Basically, this makes cross-compilation impossible with the current umk ...

Hm, we probably need to add flags override to .bm, right?

IMHO, we need an option for umk. Something like "target platform", "-t", I guess. Possible values should be "windows", "linux", "mac", "native", e.t.c.

Another useful option for umk on Windows is "use POSIX-like configuration files". I'm already doing this, but I manually change source code for that.

Well, we already can add flags via "Common fixed flags". I think all we need is "override" option that would avoid adding platform flags before adding these fixed. And then of replace any PLATFORM_ #ifdefs in GCC builder with if(HasFlag(...

Well, we already can add flags via "Common fixed flags". I think all we need is "override" option that would avoid adding platform flags before adding these fixed.

./umk uppsrc ide CLANG -r -M=Makefile +WIN32,GUI

./umk uppsrc ide CLANG -rs -M=BSDMakefile +POSIX,BSD,FREEBSD,GUI

./umk uppsrc ide CLANG -rs -M=GNUMakefile +POSIX,LINUX,GUI

IMHO, we need an option for umk. Something like "target platform", "-t", I guess. Possible values should be "windows", "linux", "mac", "native", e.t.c.

mirek wrote on Tue, 02 June 2020 19:27

Well, we already can add flags via "Common fixed flags". I think all we need is "override" option that would avoid adding platform flags before adding these fixed.

As far as I know, there is a possibility to override platform related flags. This feature is implemented in LocalHost::AddFlags and LocalHost::HasPlatformFlag methods ^{[1, 2, 3]}.
For example, possible to generate Makefile(s) for WIN32 platform, e.g.:

./umk uppsrc ide CLANG -r -M=Makefile +WIN32,GUI

or FreeBSD:

./umk uppsrc ide CLANG -rs -M=BSDMakefile +POSIX,BSD,FREEBSD,GUI

or Linux:

./umk uppsrc ide CLANG -rs -M=GNUMakefile +POSIX,LINUX,GUI

etc.

There is just a need to use correct build method for them. But I didn't test this for cross-platform/remote build(s), just for Makefile generation.

May suggest to add "OSX" flag to platformFlags also (based on order of appearance in LocalHost::AddFlags method). Patch attached.

reference/GLDrawDemo is still broken.

/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/GLDraw/GLShaders.cpp:30:3: error: expected expression
                DLOG(error);
                ^

Novo wrote on Wed, 03 June 2020 01:30

reference/GLDrawDemo is still broken.

/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/GLDraw/GLShaders.cpp:30:3: error: expected expression
                DLOG(error);
                ^

Thanks, fixed.

Yes, but that ADDs flags. The problem is that there are default, platform defined flags too. We need a method to clear those...

if(HasPlatformFlag(cfg))
	return;

IMHO, we need an option for umk. Something like "target platform", "-t", I guess.

There is another unexpected problem with Clang on Windows.
If I add "-std=c++17" to cpp options, then in case of GUI app (example: tutorial/Gui01) in Debug configuration I get this (Release is fine):
[code]Linking...
lld-link: error: duplicate symbol: std::__throw_bad_alloc()
>>> defined at c:\local\apps\upp\uppsrc\Core\heap.cpp:315
>>> c:/local/apps/upp/out/tutorial/Core/CLANGx64cpp17.Debug.Debu g_Full.Gui\heap.o
>>> defined at libc++.a(new.cpp.obj)

lld-link: error: duplicate symbol: operator new(unsigned long long)
>>> defined at c:\local\apps\upp\uppsrc\Core\heap.cpp:301
>>> c:/local/apps/upp/out/tutorial/Core/CLANGx64cpp17.Debug.Debu g_Full.Gui\heap.o
>>> defined at libc++.a(new.cpp.obj)

I have removed all #ifdef PLATFORM_* from GCC builder (except one case where this was not possible, but that one should not affect POSIX -> WIN32 compilation).

Linking...
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/.cache/upp.out/ide/Builders/CLANG.Debug.Debug_Full.Noblitz.Shared/Builders.a(Cocoa.o): in function `GccBuilder::CocoaAppBundle()':
/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:18: undefined reference to `Upp::StreamRaster::LoadFileAny(char const*, Upp::Function<bool (int, int)>)'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:26: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:26: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:48: undefined reference to `Upp::Image::Image(Upp::Image const&)'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:51: undefined reference to `Upp::PNGEncoder::PNGEncoder(int, Upp::ImageKind, bool)'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:51: undefined reference to `Upp::Rescale(Upp::Image const&, int, int, Upp::Function<bool (int, int)>)'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:51: undefined reference to `Upp::StreamRasterEncoder::SaveFile(char const*, Upp::Image const&)'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:51: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:51: undefined reference to `Upp::PNGEncoder::~PNGEncoder()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:52: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:51: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:51: undefined reference to `Upp::PNGEncoder::~PNGEncoder()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/ide/Builders/Cocoa.cpp:52: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/.cache/upp.out/ide/Builders/CLANG.Debug.Debug_Full.Noblitz.Shared/Builders.a(Cocoa.o): in function `PNGRaster__initialize_struct':
/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/plugin/png/png.h:8: undefined reference to `Upp::PNGRaster__initializer()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/.cache/upp.out/ide/Builders/CLANG.Debug.Debug_Full.Noblitz.Shared/Builders.a(Cocoa.o): in function `Upp::SortedVectorMap<int, Upp::Image, Upp::StdLess<int> >::Add(int const&, Upp::Image const&)':
/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/Core/InVector.h:650: undefined reference to `Upp::Image::operator=(Upp::Image const&)'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/Core/InVector.h:650: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/Core/InVector.h:650: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/.cache/upp.out/ide/Builders/CLANG.Debug.Debug_Full.Noblitz.Shared/Builders.a(Cocoa.o): in function `void Upp::Destroy<Upp::Image>(Upp::Image*, Upp::Image const*)':
/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/Core/Vcont.h:28: undefined reference to `Upp::Image::~Image()'
/usr/bin/ld: /home/ssg/.local/soft/bb-worker/worker/l-upp/build/.cache/upp.out/ide/Builders/CLANG.Debug.Debug_Full.Noblitz.Shared/Builders.a(Cocoa.o): in function `Upp::Image Upp::clone<Upp::Image>(Upp::Image const&)':
/home/ssg/.local/soft/bb-worker/worker/l-upp/build/uppsrc/Core/Defs.h:398: undefined reference to `Upp::Image::Image(Upp::Image const&)'

Sorry, fixed at the price of another #ifdef, which will mean you cannot build OSX GUI appliaction in LINUX (which is impossible anyway).

Mirek

A minor issue with mingw on Linux.
umk calls x86_64-w64-mingw32-windres.exe, which is called x86_64-w64-mingw32-windres (no extension) on Linux, otherwise +WIN32 seems to work ...

String windres = "windres.exe";

String windres = "windres";

Another interesting observation. Size of executable.

mingw:

:~/.local/soft/bb-worker/worker/mingw-upp/build/.cache/upp.out/CLANG.Blitz.Gui.Win32$ ll AK.exe 
-rwx------ 1 ssg ssg 2541056 Jun  4 14:56 AK.exe*

Novo wrote on Thu, 04 June 2020 11:35

A minor issue with mingw on Linux.
umk calls x86_64-w64-mingw32-windres.exe, which is called x86_64-w64-mingw32-windres (no extension) on Linux, otherwise +WIN32 seems to work ...

Could you please fix that?
You just need to replace

String windres = "windres.exe";

with

String windres = "windres";

TIA

Problem with mingw & linking when compiling reference/GuiWebCrawler.
Command line contains "-lcrypto -lssl".
Linker is looking for libcrypto.a & libssl.a, which are named crypto.lib & ssl.lib.

Novo wrote on Thu, 04 June 2020 21:14

Another interesting observation. Size of executable.

mingw:

:~/.local/soft/bb-worker/worker/mingw-upp/build/.cache/upp.out/CLANG.Blitz.Gui.Win32$ ll AK.exe 
-rwx------ 1 ssg ssg 2541056 Jun  4 14:56 AK.exe*

Not sure what is "mingw" here... Do you mean llvm-mingw toolchain in crosscompile mode?

If yes, could you activate Verbose mode and compare all compiler/linker options?

Mirek

Novo wrote on Fri, 05 June 2020 04:59

Problem with mingw & linking when compiling reference/GuiWebCrawler.
Command line contains "-lcrypto -lssl".
Linker is looking for libcrypto.a & libssl.a, which are named crypto.lib & ssl.lib.

Can you try to add .lib in package organizer? Just to know it that helps.

- Try with GIT head, as Martin claims this is now fixed. This would be most interesting option for me (Being there, you could check msvcrt version too).

Mirek

mirek wrote on Fri, 05 June 2020 03:44

Novo wrote on Fri, 05 June 2020 04:59

Problem with mingw & linking when compiling reference/GuiWebCrawler.
Command line contains "-lcrypto -lssl".
Linker is looking for libcrypto.a & libssl.a, which are named crypto.lib & ssl.lib.

Can you try to add .lib in package organizer? Just to know it that helps.

Adding .lib to crypto & ssl doesn't affect anything.

BTW, precompiled ssl is already included into Clang distribution bundled with Upp.

mirek wrote on Thu, 04 June 2020 04:37

- Try with GIT head, as Martin claims this is now fixed. This would be most interesting option for me (Being there, you could check msvcrt version too).

Mirek

Checked against GIT head. The problem is still there.

What do you mean by msvcrt version?

Novo wrote on Fri, 05 June 2020 22:24

mirek wrote on Thu, 04 June 2020 04:37

- Try with GIT head, as Martin claims this is now fixed. This would be most interesting option for me (Being there, you could check msvcrt version too).

Mirek

Checked against GIT head. The problem is still there.

Just to be sure: I mean llvm-mingw head (not U++)....

By default, llvm-mingw builds for universal CRT, which is the new Windows OS C library in Win10 (and via system updates, since Win7). There is older one, "msvcrt.dll" which is present sice WinNT. However, situation is more complicated:

https://github.com/mstorsjo/llvm-mingw/issues/104

Still, it would probably be better to ship with msvcrt version... maybe next release.

Mirek

Anyway, I'll try to recompile llvm-mingw with --with-default-msvcrt=msvcrt on the build-mingw-w64.sh line.

Frankly, it is dancing between blades.... I was really happy to find working solution for genuine windows for all small issues involved. It is really stupid that it has wine problem.

Mirek