Menu

#1077 Support mingw compilation under Linux

Committed
closed
build (4)
5
2014-12-03
2014-10-03
No

Hello,

I tried to compile Scite for Windows, using Mingw64 on Linux host. Since makefiles in win32 directories are for compiling on Windows host, I made some changes to make these makefiles universal, e.g. now they work in Mingw both on Linux and Windows hosts. I am attaching them here. Both produced against current trunk of scintilla and scite projects.

Or, if you prefer patches I am attaching them too.

Compilation tested in Windows 7 with TDM-GCC-64 and on Fedora Linux 64-bit with mingw64, both with and without LUA.

Please review,

Thank you

4 Attachments

Related

Bugs: #1993

Discussion

  • Neil Hodgson

    Neil Hodgson - 2014-10-03

    Trying with MinGW32 g++ 4.8.1 Windows 7:

    :::text
    >mingw32-make -f makefile.scintilla clean
    *.exe *.o *.obj *.dll *.res *.map *.plist
    makefile.scintilla:53: recipe for target 'clean' failed
    process_begin: CreateProcess(NULL, *.exe *.o *.obj *.dll *.res *.map *.plist, ...) failed.
    make (e=2): The system cannot find the file specified.
    
    mingw32-make: *** [clean] Error 2
    
    >mingw32-make --version
    GNU Make 3.82.90
    Built for i686-pc-mingw32
    Copyright (C) 1988-2012 Free Software Foundation, Inc.
    License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
    This is free software: you are free to change and redistribute it.
    There is NO WARRANTY, to the extent permitted by law.
    
     
    • Baurzhan Muftakhidinov

      It looks like I didn't hit the Reply button) Anyways, my answer is below

       
  • Baurzhan Muftakhidinov

    Hi,

    Now I see that these DEL defines shouldn't have been indented.

    please try the updated version in attachment.

     
  • Neil Hodgson

    Neil Hodgson - 2014-10-07

    Committed as [76ef0b] and [668818]. The Scintilla changes are fairly simple although I don't understand why indenting the commands failed for Scintilla when they work for SciTE. I have memories of major problems trying to use '/' instead of '\' but I haven't seen anything go wrong with this. Repeating the 5 line copy block 3 times is ugly though.

     

    Related

    Commit: [668818]
    Commit: [76ef0b]

    • Baurzhan Muftakhidinov

      Hi Neil,

      Regarding the "5 line copy block 3 times", could you please review the attached patch?
      It:
      1) Unifies *dll rules, merging two rules in one
      2) Gets rid of ifeq $(OS) block on copy rules, by defining path substitution directly in COPY variable. Don't know whether it will look healthy to you.

      Thanks,

       
  • Neil Hodgson

    Neil Hodgson - 2014-10-07
    • labels: --> build
    • Group: Completed --> Committed
     
  • Neil Hodgson

    Neil Hodgson - 2014-12-03
    • status: open --> closed
    • assigned_to: Neil Hodgson
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.