Remove MinGW support #51301

Closed
opened 2017-04-23 20:56:02 +02:00 by Aaron Carlisle · 11 comments
Member

The Issue


For a long time now MinGW has been unsupported and unmaintained and at this point, it looks like something that we should just leave behind and move on.

Why Remove


One of the big motivations for MinGW back in the day is that it was free compared to MSVC which was licensed based.
However, now that this is no longer true we have basically stopped updating the need CMake files.
Along with the CMake files, there are several patches to the extern libs needed to make this work. For example, see:
https://developer.blender.org/diffusion/B/browse/master/extern/carve/patches/mingw_w64.patch

If we wanted to keep MinGW then we would need to make more custom patches to the external libs and this is not something our platform maintainers are willing to do.
So for the same reasons we removed Scons I think we should also remove MinGW.

TODO


Remove from git -- D2648
Remove from SVN
Rewrite Build docs for windows

The Issue **** For a long time now MinGW has been unsupported and unmaintained and at this point, it looks like something that we should just leave behind and move on. Why Remove **** One of the big motivations for MinGW back in the day is that it was free compared to MSVC which was licensed based. However, now that this is no longer true we have basically stopped updating the need CMake files. Along with the CMake files, there are several patches to the extern libs needed to make this work. For example, see: https://developer.blender.org/diffusion/B/browse/master/extern/carve/patches/mingw_w64.patch If we wanted to keep MinGW then we would need to make more custom patches to the external libs and this is not something our platform maintainers are willing to do. So for the same reasons we removed Scons I think we should also remove MinGW. TODO **** Remove from git -- [D2648](https://archive.blender.org/developer/D2648) Remove from SVN Rewrite Build docs for windows
Author
Member

Changed status to: 'Open'

Changed status to: 'Open'
Author
Member
Added subscribers: @Blendify, @LazyDodo, @brecht, @mont29, @MartijnBerger, @Sergey
Member

Added subscriber: @MikeErwin

Added subscriber: @MikeErwin
Member

+1 for removing it

MinGW itself seems stuck on an old version of gcc. I get the impression it's not being updated. On Windows I'm more and more pleased with MSVC over the years.

It's not just about CMake files; we can also streamline the "how to build" instructions & remove the MinGW libs. Fewer options = less confusion for people setting up a new dev system.

+1 for removing it MinGW itself seems stuck on an old version of gcc. I get the impression it's not being updated. On Windows I'm more and more pleased with MSVC over the years. It's not just about CMake files; we can also streamline the "how to build" instructions & remove the MinGW libs. Fewer options = less confusion for people setting up a new dev system.

Added subscriber: @fablefox

Added subscriber: @fablefox

+1

I agree. Until there are changes in MSVC licensing, MinGW can be removed.

+1 I agree. Until there are changes in MSVC licensing, MinGW can be removed.

Added subscriber: @ideasman42

Added subscriber: @ideasman42

Some things to consider:

  • @MikeErwin There are Mingw builds for recent GCC versions (6.0) , but for some reason http:*www.mingw.org main download link points to a release from 2013.
  • The build configuration for Mingw could be useful for building with Clang on Windows (although Clang has some support for MSVC ABI... I never tested this).
  • Removing MINGW support might cause some annoying conflicts with master/blender2.8 branches (probably not too many), whatever the case, whoever removes mingw should manage syncing changes between branches.
  • @Blendify re: "Rewrite Build docs for windows" - AFAICS we only need to remove a single wiki page.

For now suggest to mail bf-committers, general +1 for removal but don't think its really a priority either.

Some things to consider: - @MikeErwin There are Mingw builds for recent [GCC versions (6.0) ](https:*sourceforge.net/projects/mingw-w64/), but for some reason http:*www.mingw.org main download link points to a release from 2013. - The build configuration for Mingw could be useful for building with Clang on Windows *(although Clang has some support for MSVC ABI... I never tested this).* - Removing MINGW support might cause some annoying conflicts with master/blender2.8 branches *(probably not too many)*, whatever the case, whoever removes mingw should manage syncing changes between branches. - @Blendify re: *"Rewrite Build docs for windows"* - AFAICS we only need to remove a single wiki page. For now suggest to mail bf-committers, general +1 for removal but don't think its really a priority either.
Member

I think the main issue for mingw support it's the 3rd party deps, nobody has stepped up in years to care for them. understandably so, this is the patchset just to be able to build python with mingw. It's not impossible to do, someone just has to step up. That being said, if someone were to step up, it's probably easier to bring back mingw support from a completely removed state than a neglected for years state. so +1 for the removal of it.

In #51301#430261, @ideasman42 wrote:
Some things to consider:

  • The build configuration for Mingw could be useful for building with Clang on Windows (although Clang has some support for MSVC ABI... I never tested this).

Clang is a special kind of animal on windows, it doesn't have a c-runtime or any windows related headers so it borrows all of that from msvc, i did some preliminary work to build blender with clang on windows a while ago, but kinda lost interest in working on it, The only motivation we have to switch compilers is to get rid of the dependency of having msvc installed and the use of the binary ms c runtime dll's, and clang doesn't solve either issue.

I think the main issue for mingw support it's the 3rd party deps, nobody has stepped up in years to care for them. understandably so, [this ](https://github.com/Alexpux/MINGW-packages/tree/master/mingw-w64-python3) is the patchset just to be able to build python with mingw. It's not impossible to do, someone just has to step up. That being said, if someone were to step up, it's probably easier to bring back mingw support from a completely removed state than a neglected for years state. so +1 for the removal of it. > In #51301#430261, @ideasman42 wrote: > Some things to consider: > > - The build configuration for Mingw could be useful for building with Clang on Windows *(although Clang has some support for MSVC ABI... I never tested this).* Clang is a special kind of animal on windows, it doesn't have a c-runtime or any windows related headers so it borrows all of that from msvc, i did some preliminary work to build blender with clang on windows a while ago, but kinda lost interest in working on it, The only motivation we have to switch compilers is to get rid of the dependency of having msvc installed and the use of the binary ms c runtime dll's, and clang doesn't solve either issue.

This issue was referenced by 9f044cb422

This issue was referenced by 9f044cb422c1fc9ad79278092445f612342abb59
Author
Member

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Sign in to join this conversation.
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
6 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#51301
No description provided.