tree-hugger icon indicating copy to clipboard operation
tree-hugger copied to clipboard

Added windows compiling ability to tree-hugger

Open sarthakforwet opened this issue 4 years ago • 4 comments

Following scripts have been updated -

  • [x] download_libs.py
  • [x] create_libs.py

Both of these files belong to the tree_hugger package and hence updation in the module is necessary on PyPI. Until the next version it would be good to run pip install -e . to work on windows.

I worked in a virtual environment on Windows and did not get requests module pre-installed with it but it is required for the installation and working of the tree-hugger package. If maintainers would agree, I would update the requirements.txt file regarding the installation of requests module.

The Windows .dll that we are working with is being built using a 32 bit Python Interpretor. Running the same with 64 bit Interpreter abruptly shuts down the current session. This could be possibly due to a data type overflow error. #18 deals with this issue in detail.

sarthakforwet avatar Dec 05 '20 21:12 sarthakforwet

Also, yes, you can add the requests part in requirements.txt please (and also in setup.py as this will install dependencies while installing from pypi)

rcshubhadeep avatar Dec 06 '20 13:12 rcshubhadeep

I am getting issues installing dependencies from requirements.txt. The problem is occurring in the setup file of pygit2. Please have a look as why is this happening. Not sure but it could be OS problem.

image

sarthakforwet avatar Dec 08 '20 04:12 sarthakforwet

I am getting issues installing dependencies from requirements.txt. The problem is occurring in the setup file of pygit2. Please have a look as why is this happening. Not sure but it could be OS problem.

image

Please try to run pip install --upgrade pip first. Pygit2 has a binary wheel but unfortunately before pip 19.0+ it was not getting picked up. If it still remains then we have to look into it again.

rcshubhadeep avatar Dec 08 '20 08:12 rcshubhadeep

Had a discussion with @sarthakforwet and there is an issue with modern 64 bit Windows compilation. We hold on this issue for the moment. I am going to look into original py-tree-sitter and will come back to it.

rcshubhadeep avatar Dec 17 '20 11:12 rcshubhadeep