Opened 3 years ago
Closed 8 months ago
#63830 closed defect (fixed)
cmake: migrate variants 'qt5' and 'docs', to subports 'cmake-gui' and 'cmake-docs', respectively
Reported by: | mascguy (Christopher Nielsen) | Owned by: | mascguy (Christopher Nielsen) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | michaelld (Michael Dickens) | |
Port: | cmake |
Description (last modified by mascguy (Christopher Nielsen))
When enabling the variants for the CMake GUI and/or Docs, that results in a tremendous number of requisite dependencies built/installed first.
Given that CMake is very much a foundational port, it should always have the bare minimum of dependencies. In order to ensure that, let's migrate the GUI and Docs to two separate subports. (Perhaps named cmake-gui
and cmake-docs
.)
By making this change, we can also simplify the main port: It will eliminate most of the logic preventing circular dependencies, when the variants are enabled.
@michaelld, I'll do the work, if you support the idea!
Change History (10)
comment:1 Changed 3 years ago by mascguy (Christopher Nielsen)
Description: | modified (diff) |
---|
comment:2 Changed 3 years ago by mascguy (Christopher Nielsen)
Summary: | cmake: migrate variants '+gui' and '+docs' to subports → cmake: migrate variants '+qt' and '+docs', to subports 'cmake-gui' and 'cmake-docs', respectively |
---|
comment:3 Changed 3 years ago by mascguy (Christopher Nielsen)
Summary: | cmake: migrate variants '+qt' and '+docs', to subports 'cmake-gui' and 'cmake-docs', respectively → cmake: migrate variants '+qt5' and '+docs', to subports 'cmake-gui' and 'cmake-docs', respectively |
---|
comment:4 Changed 3 years ago by Christopher Nielsen <mascguy@…>
comment:5 Changed 3 years ago by mascguy (Christopher Nielsen)
Summary: | cmake: migrate variants '+qt5' and '+docs', to subports 'cmake-gui' and 'cmake-docs', respectively → cmake: migrate variants 'qt5' and 'docs', to subports 'cmake-gui' and 'cmake-docs', respectively |
---|
comment:6 Changed 2 years ago by Christopher Nielsen <mascguy@…>
comment:7 Changed 2 years ago by Christopher Nielsen <mascguy@…>
comment:8 Changed 2 years ago by mascguy (Christopher Nielsen)
Michael, while I wouldn't consider these subports production-worthy yet, we're getting warmer! Lol
comment:10 Changed 8 months ago by kencu (Ken)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
this is now done in the cmake port
In b98cd0efc774b55cb3a03976766c54ea6940f662/macports-ports (master):