Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 373439 - dev-vcs/git-1.7.6: automagic dep against dev-vcs/cvs
Summary: dev-vcs/git-1.7.6: automagic dep against dev-vcs/cvs
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-29 07:25 UTC by Fabio Erculiani (RETIRED)
Modified: 2011-06-29 08:45 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
kill cvs patch (git-1.7.6-kill-cvs.patch,798 bytes, patch)
2011-06-29 08:32 UTC, Fabio Erculiani (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fabio Erculiani (RETIRED) gentoo-dev 2011-06-29 07:25:56 UTC
I guess it's back again.

# USE="-cvs" emerge =dev-vcs/git-1.7.6

It gives me git-cvs for free.

/usr/bin/git-cvsserver
/usr/libexec/git-core/git-cvsexportcommit
/usr/libexec/git-core/git-cvsimport
/usr/libexec/git-core/git-cvsserver
/usr/share/doc/git-1.7.6/git-cvsexportcommit.txt.bz2
/usr/share/doc/git-1.7.6/git-cvsimport.txt.bz2
/usr/share/doc/git-1.7.6/gitcvs-migration.txt.bz2
/usr/share/doc/git-1.7.6/git-cvsserver.txt.bz2
/usr/share/man/man1/git-cvsexportcommit.1.bz2
/usr/share/man/man1/git-cvsimport.1.bz2
/usr/share/man/man1/git-cvsserver.1.bz2
/usr/share/man/man7/gitcvs-migration.7.bz2

Reproducible: Always
Comment 1 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2011-06-29 07:58:05 UTC
Try run them, and see if you get the warning or the actual command.
It SHOULD give you the warning iirc.
Comment 2 Fabio Erculiani (RETIRED) gentoo-dev 2011-06-29 08:32:17 UTC
I don't think I want to have some sort of warnings here, I don't want to see these files installed in any case.

The same has been done for git-svn stuff.
I have fixed this way here (see the attached patch).
Comment 3 Fabio Erculiani (RETIRED) gentoo-dev 2011-06-29 08:32:39 UTC
Created attachment 278613 [details, diff]
kill cvs patch
Comment 4 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2011-06-29 08:42:08 UTC
Upstream rejected this already, because under certain cases those wrappers get called by other stuff, and having them not present causes weirdness. The wrappers need to stay existing and return a certain error code instead.
Comment 5 Fabio Erculiani (RETIRED) gentoo-dev 2011-06-29 08:45:16 UTC
Then also Fedora is doing it wrong... Looking at their git.spec.