search

aspiers / git-deps

git commit dependency analysis tool
GNU General Public License v2.0
298 stars 47 forks source link

need proper installation mechanism #51

Closed aspiers closed 6 years ago

aspiers commented 9 years ago

Currently it's expected that git-deps will be run from the top of the git repo (see #27 and #22), and this only works by following the installation instructions which say to do:

ln -s /path/to/git-deps/repo/git-deps ~/bin

However that's a really lame installation process, which also assumes that the files under html/ will be retrieved from relative to the git-deps scripts. Instead it should be parametrized to facilitate clean packaging. @bmwiedemann has already written a Makefile in https://build.opensuse.org/package/show/home:bmwiedemann/git-deps which supports the usual $DESTDIR and $PREFIX variables, although maybe we should go the Pythonic route and write a setup.py instead - I'm not sure.

aspiers commented 8 years ago

I have converted it into a Python module - currently in the module branch but will release to PyPi soon-ish.

nicolasbock commented 8 years ago

@aspiers Would you mind tagging a release? I am packaging it for gentoo, and a release would help.

aspiers commented 8 years ago

@nicolasbock Will do but I still have a bit of work left to do before the conversion to a module is complete. Perhaps you could help me test it? By now the only problem I know about is that click and drag doesn't work in the visualization.

nicolasbock commented 8 years ago

I have it installed now without the graphical UI. I will install the dependencies today so that I can play around with it more.

aspiers commented 6 years ago

It's finally hackweek so I'm revisiting this now, and being reminded how much Python's ecosystem miserably sucks :angry:

aspiers commented 6 years ago

This is now done: https://pypi.org/project/git-deps/

There will no doubt be a few teething problems. For instance it seems that gitfile-handler.desktop is not being installed correctly yet. But I'll open a separate issue for that.