Summary: | dev-util/git: nocvs support in manner of noperl | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Bernd Lommerzheim <bernd> |
Component: | [OLD] Development | Assignee: | Robin Johnson <robbat2> |
Status: | RESOLVED UPSTREAM | ||
Severity: | enhancement | CC: | ricmm |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://www.git-scm.com | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
git 1.6.3 ebuild patch (on git-1.6.3_rc4.ebuild)
git 1.6.3 nocvs patch git 1.6.3 ebuild patch (on git-1.6.3_rc4.ebuild) git 1.6.3 nocvs patch |
Description
Bernd Lommerzheim
2009-05-07 16:18:44 UTC
Created attachment 190631 [details, diff]
git 1.6.3 ebuild patch (on git-1.6.3_rc4.ebuild)
Created attachment 190632 [details, diff]
git 1.6.3 nocvs patch
I'm sorry. Both patches had mistakes. I will attach both patches in a new version and I hope that they are now working without problems. By the way they are solving the following problem: If you are using "cvs -perl" in one of the current ebuilds the ebuild says that it will ignore the "cvs" flag but the cvs scripts are installed into the system. When executing them they just raise a error that git was compiled with NO_PERL (with "-perl"). Now the "cvs" flag is getting really ignored and no cvs scripts are installed. I hope the new patches are working for everyone. Have a nice day. Created attachment 190635 [details, diff]
git 1.6.3 ebuild patch (on git-1.6.3_rc4.ebuild)
Created attachment 190636 [details, diff]
git 1.6.3 nocvs patch
I had my commit in the tree before I saw this bug (my email is running slow today) I'm not sure that adding a nocvs patch like this is correctly, I'd rather just get upstream to update the noperl stuff to exclude CVS as well. oops, forgot to assign I think this should be seperated. Maybe a user want to install git with the perl stuff, but without the cvs scripts and server. I guess it was an upstream design decision to install these cvs shell scripts saying that the cvs server is not available because git was build with NO_PERL instead of just not installing these shell scripts. But why should the cvs use-flag just be used in src_test and not by the rest of the ebuild? And why does the ebuild says that the +cvs flag is ignored when using "-perl" (which generally means that the ebuild is building the programm without this support) but installs these cvs shell scripts anyway? I think the ebuild should not install these cvs shell scripts when using "-cvs" or "-perl". If we want these cvs shell scripts when using "+cvs -perl" my ebuild patch can be changed from > if ! use cvs || ! use perl ; then > myopts="${myopts} NO_CVS=YesPlease" > fi > to > use cvs || myopts="${myopts} NO_CVS=YesPlease" What do you think? And by the way: You removed the "files/50git-gentoo.el" file out of the portage which is still used in line 69 and 213 of the current git-1.6.3.ebuild. I restored that other file already. I'm strongly in favour of upstream's decision to install the wrapper saying the command is not available rather than not installing it at all. We could install those wrappers for USE=-cvs as well, would that be suitable? In that case it might be better to leave the ebuild like it is now. If the ebuild installs these cvs files in any case it can directly install the original files from the git release. That should be better than installing some custom shell scripts when using "-cvs perl" which are raising an error that git was built without cvs support and that the user should remerge git with the +cvs use-flag in order to use these scripts. It wasn't clear in your last message. - Can we close this bug, with acceptance of the upstream state? OR - Override/modify their "CVS is not available" message with one telling the user to emerge with USE=cvs? You can close this ticket. I think that the upstream message is clear and does not need to be modified. Thanks. Best regards. Thanks. |