Fix missing distutils module error during build #672
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
👋 Thanks for contributing @Wingless-Archangel! We will review the pull request and get back to you soon. |
|
@microsoft-github-policy-service agree |
Wingless-Archangel
force-pushed
the
main
branch
from
b56759c to
10d3c92
Compare
Vamsidarling
approved these changes
Feb 3, 2025
Wingless-Archangel
force-pushed
the
main
branch
from
10d3c92 to
770fd1d
Compare
|
Hi @koreyspace, Is there something wrong with my pull request? |
Wingless-Archangel
force-pushed
the
main
branch
from
0f7791f to
a9d520d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While trying to spawn the devcontainer in VSCode and Codespace, I encountered an error during the build process related to the missing
distutilspackage for some reason.After investigating further,
I discovered that the build tool failed to read the configuration from the(after have another look this file is for conda to use but the build doesn't use conda at all) it was using a base Python version higher than 3.10, which is deprecated and no longer includes the distutils package as outlined in PEP-632 . Also, some packages already included in the container image..devcontainer/environment.ymlfile.This pull request addresses this issue by removing some dependencies instead and add rust compiler to satisfy tiktoken requirement.
Please have a look.
EDIT: change the approach since I couldn't reproduce the build anymore.