Skip to content

ENH: Patch versioneer files into manifest at runtime#309

Merged
effigies merged 2 commits intopython-versioneer:masterfrom
effigies:fix/cleanup_manifest
Aug 9, 2022
Merged

ENH: Patch versioneer files into manifest at runtime#309
effigies merged 2 commits intopython-versioneer:masterfrom
effigies:fix/cleanup_manifest

Conversation

@effigies
Copy link
Copy Markdown
Contributor

@effigies effigies commented Jul 27, 2022

Hack egg_info to modify SOURCES.txt, adding versioneer.py and versionfile_source. versionfile_source is necessary in cases where there's a single module.py file in the root. Admittedly a boutique case, but one we test for.

This removes the need to add these files to MANIFEST.in, which has been a source of annoyance (#40, #247, probably others...).

Closes #40.
Addresses part of #247.

Hack egg_info to modify SOURCES.txt, adding versioneer.py and
versionfile_source. versionfile_source is necessary in cases where
there's a single module.py file in the root. Admittedly a boutique case,
but one we test for.
@effigies effigies force-pushed the fix/cleanup_manifest branch from bae3128 to b44f75d Compare July 28, 2022 00:53
@effigies effigies changed the title MNT: Remove versionfile_source from MANIFEST.in ENH: Patch versioneer files into manifest at runtime Jul 28, 2022
@effigies
Copy link
Copy Markdown
Contributor Author

Thinking about this again, I think using the manifest_maker class leaves us more vulnerable to changes in implementation details. By updating egg_info.filelist and doing our own write, I think we're less likely to get broken by upstream fixes.

I would appreciate a review, if anybody has the time and interest.

@effigies effigies merged commit f55246d into python-versioneer:master Aug 9, 2022
@effigies effigies deleted the fix/cleanup_manifest branch August 9, 2022 20:07
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.

MANIFEST.in lines are sometimes (often? usually?) redundant

1 participant