Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 366133 - app-portage/repo-commit-0.3.1-r1 does not support VisualHostKey=yes in ssh_config
Summary: app-portage/repo-commit-0.3.1-r1 does not support VisualHostKey=yes in ssh_co...
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords: NeedPatch
Depends on: 373669
Blocks:
  Show dependency tree
 
Reported: 2011-05-05 19:24 UTC by Nathan Phillip Brink (binki) (RETIRED)
Modified: 2019-03-17 15:46 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nathan Phillip Brink (binki) (RETIRED) gentoo-dev 2011-05-05 19:24:24 UTC
I'm crazy and I thought these VisualHostKey things might be cool. They end up just polluting one's screen and breaking sunrise-commit.

When sunrise-commit breaks in this way, it should make it more obvious to the user what is happening. sunrise-commit is seeing 1. output from cvs on stderr instead of stdout 2. something that doesn't look like the output of ``cvs up -nq''. sunrise-commit should say something more like ``I could not parse the output of <command>.''. Also, sunrise-commit might be able to ignore this particular output from cvs, add -o VisualHostKey=no somewhere, or just tell the user to make his ssh quieter.

ohnobinki@ohnopublishing ~/gentoo-x86/net-irc/unrealircd $ sunrise-commit "Test commit (newline)."
Host key fingerprint is 84:db:48:d1:39:45:a4:c8:f4:fd:6c:0d:fa:18:81:19
+[RSA 2048]-------+
|      o.E=+      |
|     o =+*       |
|      = *.o .    |
|     . =   = o   |
|      o S o + .  |
|           =     |
|          . .    |
|                 |
|                 |
+-----------------+

M ChangeLog
M unrealircd-3.2.8.1-r1.ebuild
--- ChangeLog   2011-05-05 15:25:02.000000000 -0400
+++ ChangeLog.new       2011-05-05 15:25:09.000000000 -0400
@@ -4,6 +4,10 @@
 
   05 May 2011; Nathan Phillip Brink <binki@gentoo.org>
   unrealircd-3.2.8.1-r1.ebuild:
+  Test commit (newline).
+
+  05 May 2011; Nathan Phillip Brink <binki@gentoo.org>
+  unrealircd-3.2.8.1-r1.ebuild:
   Fix bug 316379 reported by shimi by adding buildtime depend on pkgconfig.
 
   12 Jun 2010; Alex Legler <a3li@gentoo.org> -unrealircd-3.2.8.1.ebuild,


[ERROR] !!! Please fix the following issues reported from cvs: (U,P,M,A,R,D are ok)
[ERROR] !!! Note: This is a pretend/no-modify pass...
[ERROR] |      o.E=+      |
|     o =+*       |
|      = *.o .    |
|     . =   = o   |
|      o S o + .  |
|           =     |
|          . .    |
|                 |
|                 |
M ChangeLog
M unrealircd-3.2.8.1-r1.ebuild
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2011-06-29 10:33:23 UTC
I think this particular output comes from repoman, and we really can't control what kind of options repoman passes on to CVS. If ssh doesn't support disabling that via an environment variable, I don't think we can really do anything about it.
Comment 2 Nathan Phillip Brink (binki) (RETIRED) gentoo-dev 2011-09-02 13:06:51 UTC
Renamed to repo-commit.
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2019-03-17 15:46:08 UTC
I don't think we need to be concerned by CVS anymore.