mapbox / mapbox/mason

Tests and documentation MASON_SYSROOT variable

Open
#372 2 comments 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

Per chat with @kkaefer the `MASON_SYSROOT` variable [here](https://github.com/mapbox/mason/blob/bcbb1d98ff3f30cafd5b5591ecce882426b49d9c/mason.sh#L577-L585) was added to allow a run-time override of incorrect pkg-config paths. But we think this functionality is both no longer needed or used, so this ticket is to track removing this variable in mason v2.

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 with mason.sh at the linked lines and search the repository for MASON_SYSROOT references. Check the related tests and documentation before removing the variable; done means the obsolete runtime override is no longer referenced and the affected tests and documentation reflect its removal.

Written by the indexing model from the issue text.

Assessment

Tech stack
shell
Domain
build-system, tooling
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.