mapbox / mapbox/mason

Cmake integration todo

Open
#274 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
258
Forks
68
PR merge metrics
No merged PRs in 30d

Description

Issue to track improvements we can make to cmake integration:

- After https://github.com/mapbox/mason/pull/176, packages need to be rebuilt to contain a `mason.ini`. Otherwise `pkg-config` will be used as a fallback. The TODO is to rebuild packages to avoid the `pkg-config` dependency.

- At least one package was found with an empty `mason.ini` (expat 2.1.0). We should vet all packages to see if others like this remain.

- Per https://github.com/mapbox/mason/pull/176#issuecomment-235939189 we depend on curl for ssl support due to `Instead of file(DOWNLOAD ...), we are using curl as a subprocess because some versions of CMake out there don't include SSL/TLS support.`. Potentially we could auto-detect the cmake version to start using `file(DOWNLOAD ...` for recent versions to avoid the curl dependency.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the CMake integration changes and discussion in mason pull request 176, then inspect package builds for mason.ini coverage and the current curl dependency. Done means packages no longer require pkg-config as a fallback, empty mason.ini files have been vetted, and any CMake-version-based download change is validated.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake
Domain
build-system
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.