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.

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

Reactive python 3.6 in gitsome #135

Closed
wants to merge 3 commits into from
Closed

Conversation

szlin
Copy link

@szlin szlin commented Nov 1, 2017

This PR is trying to fix #105 and reactive python 3.6 in gitsome.

Since xonsh has fixed the issue in python 3.6, the gitsome should work fine with python 3.6. I had some tests done on my side, please review on it.

Thanks.

@codecov-io
Copy link

Codecov Report

Merging #135 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #135   +/-   ##
=======================================
  Coverage   79.23%   79.23%           
=======================================
  Files          36       36           
  Lines        2817     2817           
=======================================
  Hits         2232     2232           
  Misses        585      585

@donnemartin
Copy link
Owner

Hi @szlin, I recall there were issues running in Python 3.6 because gitsome has a bundled version of xonsh, instead of listing it as a dependency. I'm not seeing a change in this PR that addresses that?

@szlin
Copy link
Author

szlin commented Nov 10, 2017

@donnemartin Do you mind to remove embedded code copies and add dependency with xonsh? I am the gitsome package maintainer in Debian, and the copies of code need to be removed in Debian archive according to Debian policy 4.13, hence I had created a patch for removing xonsh - Removal xonsh patch in Debian

@donnemartin
Copy link
Owner

This is something I'm hoping to get fixed at some point with #25, I recall at the time there were issues trying to unbundle xonsh.

It sounds like you've already done this, did you face any issues?

@szlin
Copy link
Author

szlin commented Nov 11, 2017

@donnemartin I haven't seen any issues and received any bug reports so far since 2017-02-20. You may reference to following link: gitsome in Debian

@dbaio
Copy link
Contributor

dbaio commented Nov 30, 2017

+1

It would be nice to unbundle xonsh and have a new release of gitsome as well.

Regards.

@ryukinix
Copy link

ryukinix commented Dec 28, 2017

Looking for that getting fixed/merged for months! Great job guys :) Congrats. Waiting happily ~

@ryukinix
Copy link

Any news?

@donnemartin
Copy link
Owner

Hi everyone, I'm seeing issues with this, see #105.

@szlin
Copy link
Author

szlin commented Sep 4, 2018

@donnemartin
Sorry, I still don't get the point. The issue of #105 seems to be closed since upstream of xonsh had merged the patches.

The gitsome with my patches works fine for me, and got no bug report both in Debian [1] and Ubuntu [2] over past 10 months.

Could anyone provide the reproducible steps for python3.6 error in gitsome after applying my patches? I can have a look afterwards.

szlin@szlin-TW:/home/szlin/gitsome# python3 setup.py egg_info
running egg_info
creating gitsome.egg-info
writing gitsome.egg-info/PKG-INFO
writing dependency_links to gitsome.egg-info/dependency_links.txt
writing entry points to gitsome.egg-info/entry_points.txt
writing requirements to gitsome.egg-info/requires.txt
writing top-level names to gitsome.egg-info/top_level.txt
writing manifest file 'gitsome.egg-info/SOURCES.txt'
reading manifest file 'gitsome.egg-info/SOURCES.txt'
writing manifest file 'gitsome.egg-info/SOURCES.txt'
szlin@szlin-TW:/home/szlin/gitsome# python3 --version
Python 3.6.6+

[1] https://tracker.debian.org/pkg/gitsome
[2] https://launchpad.net/ubuntu/+source/gitsome

@iamareebjamal
Copy link

1.5 years since last release and a lot of people want 3.6 support. Contributors have sent a pull request. I request @donnemartin to please have a look, approve and merge it. If you see a problem with it, please convey it so that one the OP or someone else may address it. I understand that you might be busy with other things in your life, hence, you may think of adding some other maintainers on the project so it takes the load off your head, maybe?

People love this awesome tool but it is not moving forward

@donnemartin
Copy link
Owner

Hey all, sorry been really busy. I'm starting to free up and plan to look into updating gitsome to support the latest Python versions.

@donnemartin
Copy link
Owner

Fixed with #160, gitsome now support Python 3.7.

@donnemartin donnemartin closed this Apr 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Python 3.6: ply.yacc.YaccError: Unable to build parser
6 participants