Summary: | dev-vcs/git: missing runtime dependency for cvsimport | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Nekun <nekokun> |
Component: | Current packages | Assignee: | Robin Johnson <robbat2> |
Status: | UNCONFIRMED --- | ||
Severity: | normal | CC: | polynomial-c |
Priority: | Normal | Keywords: | PullRequest |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://github.com/gentoo/gentoo/pull/20607 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Nekun
2021-04-30 00:32:54 UTC
Created PR: https://github.com/gentoo/gentoo/pull/20607 I doubt this is necessary to be a hard dependency when USE="cvs" is set. I used git-cvsimport multiple times without rsh being installed on my system. I suggest to add a conditional pkg_postinst message informing users about possible rsh necessity instead. (In reply to Lars Wendler (Polynomial-C) from comment #2) > I doubt this is necessary to be a hard dependency when USE="cvs" is set. I > used git-cvsimport multiple times without rsh being installed on my system. > > I suggest to add a conditional pkg_postinst message informing users about > possible rsh necessity instead. Please note that rsh invokes only when ':ext:' protocol is used, and as I can see, this invocation was implemented in git starting from initial revision: https://git.kernel.org/pub/scm/git/git.git/commit/?id=34155390a576d8124e0adc864aaf2f11bbf5168b. Consider it an apparent evidence. I got an error while trying to clone the OpenBSD repo. > I suggest to add a conditional pkg_postinst message informing users about possible rsh necessity instead.
Well, if we consider that RSH is actually dead this might be enough, but error while invoking a tool with default setup looks somewhat `unclear` to me. As an alternative approach, I can suggest adding CVS_RSH="ssh" as a good default to env.d, in addition to the pkg_postinst message.
|