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

Remove git-url-parse requirement #73

Merged
merged 3 commits into from
Mar 14, 2020

Conversation

philipsd6
Copy link
Contributor

@philipsd6 philipsd6 commented May 4, 2019

Fixes #72

  • replace giturlparse.parse with local implementation based on
    urllib.parse.urlparse
  • Strip tilde from owner for Bitbucket Server user repos
  • put src_dir and lock dir under directory named after the repo
    hostname
  • remove _get_lock_file()

gilt/config.py Show resolved Hide resolved
Copy link
Owner

@retr0h retr0h left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline

@philipsd6 philipsd6 force-pushed the fix/remove-giturlparse branch from 1c6e54c to 69177bd Compare May 6, 2019 17:08
@philipsd6
Copy link
Contributor Author

@retr0h I've added all the repo URIs from the git-url-parse tests, along with a couple more specific to Bitbucket Server. I added a new test to validate the repo URIs, and updated the existing tests to take into account the lock and clone files being in a hostname subdirectory. All tests pass.

philipsd6 added 3 commits May 6, 2019 13:56
Fixes #72

- replace giturlparse.parse with local implementation based on
  urllib.parse.urlparse
- Strip tilde from owner for Bitbucket Server user repos
- put src_dir and lock dir under directory named after the repo
  hostname
- remove _get_lock_dir()
@philipsd6 philipsd6 force-pushed the fix/remove-giturlparse branch from 69177bd to 744bfae Compare May 6, 2019 17:56
@jayvdb
Copy link

jayvdb commented Jul 12, 2019

Not wanting to change your course, however there is discussion about new maintenance of git-url-parse at https://github.com/retr0h/git-url-parse/pull/39

@coala / @moremoban / @GitMateIO depends on it in too many different projects for vendoring to be the ideal solution, and we do have a large-ish team, but we'd also be happy if someone else has a special interest in maintaining it. fwiw, coala also needs the BitBucket support improved.

@philipsd6
Copy link
Contributor Author

I don't much care one way or the other, but it seemed to me that if it was easy to remove the dependency on an unmaintained project then why not. But given that @retr0h isn't reviewing this PR after I made the requested changes, I've dropped my interest in this project entirely.

@retr0h
Copy link
Owner

retr0h commented Jul 19, 2019

But given that @retr0h isn't reviewing this PR after I made the requested changes, I've dropped my interest in this project entirely.

I haven't had a chance to look at this. Sorry about that.

@philipsd6
Copy link
Contributor Author

But given that @retr0h isn't reviewing this PR after I made the requested changes, I've dropped my interest in this project entirely.

I haven't had a chance to look at this. Sorry about that.

It's not personal -- we all have day jobs and sometimes other things take over our lives. I just think this project is getting very low attention, and we can't rely on it, so we'll come up with a different solution for our needs.

@jayvdb
Copy link

jayvdb commented Jul 20, 2019

@philipsd6 , depending on your needs, you may be interested in https://github.com/moremoban/moban , which isnt as mature as gilt at the things gilt does well, but it is maintained and has a few different features. Another one I have seen but not tried is https://github.com/jpscaletti/copier . Or you may someone more suitable in the lists at ksdme/org-status#6 and https://gitlab.com/coala/package_manager/issues/237

@retr0h
Copy link
Owner

retr0h commented Jul 21, 2019

@philipsd6 , depending on your needs, you may be interested in https://github.com/moremoban/moban , which isnt as mature as gilt at the things gilt does well, but it is maintained and has a few different features. Another one I have seen but not tried is https://github.com/jpscaletti/copier . Or you may someone more suitable in the lists at ksdme/org-status#6 and https://gitlab.com/coala/package_manager/issues/237

Gilt is maintained. However, I am in the process of moving Gilt into my own repo. I somewhat recently moved teams, and no longer have access to merge Gilt PRs. I am hoping to straighten that out soon, and get the project back in line.

Again, my apologies.

@ryandaugherty
Copy link

ryandaugherty commented Nov 24, 2019

@philipsd6 your fix works great! I installed it in pip with the following command (somehow got errors, but it installed), and now gilt works with Bitbucket. Thanks much!

pip3 install git+https://github.com/philipsd6/gilt.git@fix/remove-giturlparse#egg=gilt

@retr0h do you have access to merge yet? Or move the repo? :)

@retr0h
Copy link
Owner

retr0h commented Dec 2, 2019

@retr0h do you have access to merge yet? Or move the repo? :)

Hoping I will :)

/cc @cburgess

@retr0h
Copy link
Owner

retr0h commented Dec 19, 2019

@ryandaugherty ok just got access back :)

@retr0h retr0h force-pushed the master branch 3 times, most recently from 7f69361 to 54f2a58 Compare March 14, 2020 18:49
@retr0h retr0h merged commit 5d31d6a into retr0h:master Mar 14, 2020
@ryandaugherty
Copy link

Yay! thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

git-url-parse doesn't properly parse Bitbucket Server URIs
4 participants