Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add --skip-existing option to publish #444

Merged
merged 1 commit into from
Feb 25, 2021
Merged

Add --skip-existing option to publish #444

merged 1 commit into from
Feb 25, 2021

Conversation

messense
Copy link
Member

Without --skip-existing

🍹 Building a mixed python/rust project
🔗 Found pyo3 bindings with abi3 support for Python ≥ 3.6
🐍 Not using a specific python interpreter (With abi3, an interpreter is only required on windows)
    Finished release [optimized] target(s) in 0.05s
📦 Built wheel for abi3 Python ≥ 3.6 to /Users/messense/Projects/rjieba-py/target/wheels/rjieba-0.1.7-cp36-abi3-macosx_10_7_x86_64.whl
📦 Built source distribution to /Users/messense/Projects/rjieba-py/target/wheels/rjieba-0.1.7.tar.gz
🚀 Uploading 2 packages
💥 maturin failed
  Caused by: 💥 Failed to upload "rjieba-0.1.7-cp36-abi3-macosx_10_7_x86_64.whl" (2.8 MB)
  Caused by: File already exists: <html>
 <head>
  <title>400 File already exists. See https://pypi.org/help/#file-name-reuse for more information.</title>
 </head>
 <body>
  <h1>400 File already exists. See https://pypi.org/help/#file-name-reuse for more information.</h1>
  The server could not comply with the request since it is either malformed or otherwise incorrect.<br/><br/>
File already exists. See https://pypi.org/help/#file-name-reuse for more information.


 </body>
</html>

With --skip-existing

🍹 Building a mixed python/rust project
🔗 Found pyo3 bindings with abi3 support for Python ≥ 3.6
🐍 Not using a specific python interpreter (With abi3, an interpreter is only required on windows)
    Finished release [optimized] target(s) in 0.07s
📦 Built wheel for abi3 Python ≥ 3.6 to /Users/messense/Projects/rjieba-py/target/wheels/rjieba-0.1.7-cp36-abi3-macosx_10_7_x86_64.whl
📦 Built source distribution to /Users/messense/Projects/rjieba-py/target/wheels/rjieba-0.1.7.tar.gz
🚀 Uploading 2 packages
⚠  Skipping "rjieba-0.1.7-cp36-abi3-macosx_10_7_x86_64.whl" because it appears to already exist
⚠  Skipping "rjieba-0.1.7.tar.gz" because it appears to already exist
✨ Packages uploaded successfully

Closes #354

messense
messense commented Feb 24, 2021
View reviewed changes
src/upload.rs Show resolved Hide resolved
@messense messense force-pushed the upload-skip-existing branch 2 times, most recently from 6980e6e to 10db884 Compare February 24, 2021 08:22
@messense messense force-pushed the upload-skip-existing branch from 10db884 to a22d539 Compare February 25, 2021 16:06
@konstin
Copy link
Member

konstin commented Feb 25, 2021

Thank you!

@konstin konstin merged commit a2c5fd5 into PyO3:master Feb 25, 2021
@messense messense deleted the upload-skip-existing branch February 25, 2021 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@konstin konstin konstin left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Adding --skip-existing option to publish
2 participants
@messense @konstin