Skip to content

fix: add setup.py fallback for legacy setuptools to resolve UNKNOWN metadata warnings - #67

Merged
Qubitium merged 2 commits into
mainfrom
devin/fix-pip-install-logbar
Aug 5, 2026
Merged

fix: add setup.py fallback for legacy setuptools to resolve UNKNOWN metadata warnings#67
Qubitium merged 2 commits into
mainfrom
devin/fix-pip-install-logbar

Conversation

@Qubitium

@Qubitium Qubitium commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

pip install logbar (and pip install PyPcre logbar datasets) was emitting repeated WARNING: Generating metadata for package logbar produced metadata for project name unknown warnings when the build ran against legacy setuptools that could not read pyproject.toml [project] metadata (e.g. system setuptools 59.6 on Ubuntu 22.04 or no-build-isolation installs). Without a setup.py fallback, the metadata came back as UNKNOWN, causing pip to discard each candidate version.

Changes

  • Added setup.py with the canonical LogBar metadata (name, version, license, packages, python_requires, etc.) so legacy setuptools can build the package correctly even when it does not parse pyproject.toml.
  • Marked license as dynamic in pyproject.toml and provided it through setup.py (license="Apache-2.0"). This avoids the setuptools 83+ deprecation warning for project.license as a TOML table while still being valid for setuptools>=61.
  • Lowered the build-system setuptools lower bound from >=77.0.1 to >=61 since the PEP 639 SPDX feature is no longer used.

Verification

  • pip install . succeeds with modern pip/setuptools build isolation.
  • pip install . --no-build-isolation succeeds on fresh venvs with setuptools 59.6, 60, 61, 65, 75, 80, 82, and 83, with Name: LogBar and no license/UNKNOWN warnings.
  • Local test suite passes: 157 passed, 6 skipped.

Link to Devin session: https://app.devin.ai/sessions/fd150f4dd61f470e96ff670b2dfe7649
Requested by: @Qubitium

@Qubitium Qubitium self-assigned this Aug 5, 2026
@devin-ai-integration

Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@Qubitium
Qubitium merged commit b537d60 into main Aug 5, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant