Differential D30238
devel/git, devel/git-cvs, and 2 more: Split git ports Authored by grembo on May 13 2021, 2:49 AM. Tags None Referenced Files
Details
This moves CVS, P4, and SVN support to three subports, Things were done in a way, so that the port can be Did some testing, could use some more love. Made sure things build. Only minimal test if p4, svn, Ran these poudriere tests successfully (using default options):
Diff Detail
Event TimelineComment Actions Minor changes. It's a bit ugly how options are defined and excluded right now, Comment Actions Looks OK to me. It is as you say a bit ugly, but I think it gets us where we want to be and is the best we can do until sub-packages are supported. Comment Actions I'm facing some issues on the system I use to test all my ports related stuff. Hopefully it will be fixed today yet and then I'm going to be able to test it and get it committed. Comment Actions @grembo Patch is not applying after latest changes on git port. Could you please rebase it? I finally found the time to work on it, sorry for the delay Comment Actions Should apply now (there were a couple of bigger changes to the port just two days ago, so it was a bit of work to get it back in line). I tested git/git-p4/git-cvs in poudriere testport, git-svn is still running as it takes forever (lots of dependencies, including cmake). Once this is done, we might consider turning git-lite into git-core and base the git port on it (that way, git-core/lite could become a meaningful port to depend on). But that's some future business. For now, let's focus on landing this change, which should already improve the situation for folks like @bdrewery who need an easy/painless way to have git-svn support. Comment Actions @garga There is a MOVED entry on git-cvs, seems like there was something python based of that name before. So you would need to take care of this to allow the port to build in bulk (non-testport) mode. Comment Actions Hello! I'm working on it and making more changes to create more sub packages and make it as much flexible as I can. Unfortunately I don't have lots of free hours as I would like but I'm on it and don't plan to make any other changes on this port before this one Comment Actions @grembo I've submitted a new review https://reviews.freebsd.org/D32369 with final changes I made during tests and also with new git-gui subpackage. Thank you! Comment Actions D32369 landed in https://cgit.freebsd.org/ports/commit/devel/git?id=ff5ded75bcda28c580623504e01f5db8eb6be9cf Abandoning, as closing isn't an option at this point. Very happy it made it to the tree. |