-
Notifications
You must be signed in to change notification settings - Fork 13
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.
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
Publish artifacts #16
Comments
And for linux, it is better to use an old version linux distribution to avoid glibc issues, for example, centos6 for manylinux2010 in https://github.com/pypa/manylinux. (Compiling on high version glibc can not run on lower version glibc linux) |
@levyfan would you be willing to cut a 0.0.2 release? If so I can submit another PR for 0.0.3-SNAPSHOT. |
@erip The release fails to upload multiple sentencepiece-0.0.2.pom to same place. Error: Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.7:deploy (default-deploy) on project sentencepiece: Failed to deploy artifacts: Could not transfer artifact com.github.levyfan:sentencepiece:pom:0.0.2 from/to github (https://maven.pkg.github.com/levyfan/sentencepiece-jni): Transfer failed for https://maven.pkg.github.com/levyfan/sentencepiece-jni/com/github/levyfan/sentencepiece/0.0.2/sentencepiece-0.0.2.pom 409 Conflict -> [Help 1] |
It seems like the classifier isn't being added to the pom name. I'd expect separate poms for the separate platforms. Let me dig into.it. |
@erip any update? |
With the newly merged work in #12 and #14, it's possible to publish cross-platform artifacts to Maven central or even GitHub Packages for broad consumption. I think there may be a few other things to consider before doing this (e.g., should the sentencepiece version == package version? should it be a classifier?), but this issue tracks that.
The text was updated successfully, but these errors were encountered: