Skip to content

Make sure filenames containing commas are quoted when writing RECORD file for wheel#61

Merged
abn merged 5 commits intopython-poetry:masterfrom
finswimmer:issue-2052-wheel-record-with-comma
Aug 14, 2020
Merged

Make sure filenames containing commas are quoted when writing RECORD file for wheel#61
abn merged 5 commits intopython-poetry:masterfrom
finswimmer:issue-2052-wheel-record-with-comma

Conversation

@finswimmer
Copy link
Copy Markdown
Member

This PR uses the csv module, to make sure filenames containing commas are quoted, when writing the RECORD file for the wheel packages.

Resolves: python-poetry/poetry#2052

  • Added tests for changed code.

@finswimmer finswimmer requested a review from a team August 13, 2020 12:11
Comment thread poetry/core/masonry/builders/wheel.py Outdated
Comment thread poetry/core/masonry/builders/wheel.py Outdated
* use csv.excel constants to define delimiter and quotechar
@finswimmer
Copy link
Copy Markdown
Member Author

Thanks for your review @abn. I've included your suggestions.

Comment thread poetry/core/masonry/builders/wheel.py Outdated
Comment thread poetry/core/masonry/builders/wheel.py Outdated
@abn abn merged commit faefb7d into python-poetry:master Aug 14, 2020
@sdispater sdispater mentioned this pull request Sep 18, 2020
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.

Filenames containing commas lead to malformed RECORD files in wheels, and consequent "more than three elements" warnings on install

2 participants