Skip to content

Comments

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
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
* use csv.excel constants to define delimiter and quotechar
@finswimmer
Copy link
Member Author

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

@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