-
-
Notifications
You must be signed in to change notification settings - Fork 596
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
Replaced vendored openpyxl by a dependency #221
Conversation
Should we un-vendor every dependency? it's very weird in this way :) It seems this project is pre- |
The aim (for me as a simple contributor) is to unvendor other dependencies too. I'm just segmenting the patch initiated by Tommy Anthony in #195, to fix issues separately. |
The intention behind the vendoring was for user simplicity. I didn't want users of Tablib to have to install a number of other dependencies in order to use the software. This was compounded by the fact that I had to port the codebases to Python 3 myself, and make other modifications. If all of this software is now properly maintained and packaged for both Python 2 and Python 3, un-vendoring is a possibility. The large number of sub-dependencies that this would pass on to the user is quite unfortunate, but it would also reduce maintenance burden (ideally). |
That was certainly a good design when it was decided. I think now that Python dependency management has improved a lot during the recent years, and it may be time to switch to build on a dependency system again. |
Agreed, as long as everything is now available for Python 3 (I haven't checked). |
Travis should tell us that :-) |
Travis is broken (for this project) at the moment, unfortunately. I sent a message to their support a few days ago, but haven't heard anything yet. I may take the time to just switch this project over to my http://ci.kennethreitz.org/ |
There seems to be an issue with the develop branch. But for other branches like: I think that at some time, your repo was setup with the develop branch as the main branch, instead of master. It might have been the source of various problems. |
Yes, I made some adjustments a few days ago. There is no longer a develop branch (it was the default branch). I am annoyed with Travis :) |
These are all tablib Python 3 vendored packages:
These claim to support Python 3: https://pypi.python.org/pypi/xlwt The dbfpy is not clear about Python 3 support. It is a simple package, I guess they support, but used it only in Python 2.7. |
For dbf support, there's this on pypi that has many recent downloads: https://pypi.python.org/pypi/dbf, and says it's multiversion (2 and 3) |
Good. I don't know which package is the vendored one. Maybe it is this |
cdbb78f
to
59b1d71
Compare
I've just rebased the branch to check if tests are still happy. Is there anything blocking this? |
I would LOVE to have this merged. |
I'll rebase my patch ASAP. |
Recent dependencies are dropping Python 3.2 too.
Thanks Tommy Anthony for the initial patch.
Rebase done. |
Merged. It is part of #273 work. |
Wow, that was fast. Thanks guys :) |
Re-opened #217 against master.