{"thread":{"id":"27368","subject":"RFD: Misleading log --graph online","startedAt":"2011-05-16T08:58:41Z","lastAt":"2011-05-16T08:58:41Z","messageCount":1,"participants":["Michael J Gruber"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"167974","messageId":"4DD0E741.1050303@drmicha.warpmail.net","threadId":"27368","inReplyTo":null,"subject":"RFD: Misleading log --graph online","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2011-05-16T08:58:41Z","receivedAt":"2011-05-16T08:58:41Z","isPatch":false,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Consider these snippets of output:\ngit log --graph --oneline --decorate --all\n\n* ca19d2e (upstream/trunk, upstream/master, svn/trunk) update upload data\n* cd2cb19 update download url\n...\n* ab11781 new filelocator module\n* 8125929 fix: dodata->doplot\n| * e234268 (svn-tags/pyx_0_11) tagging release 0.11\n| * 580cbdf (svn-tags/pyx_0_11@3128) documentation update...\n...\n| * ffa2e8f *** empty log message ***\n| * 90dd12c *** empty log message ***\n| * 2c29cab (svn-tags/pyx_0_10) tagging 0.10\n|/\n* 42fdff5 (tag: svn/pyx_0_10) some updates to the webpage\n* fac723b some adjustments to keep support for a broad range...\n...\n\nObviously, there is a side branch svn-tags/pyx_0_11 which forked off\n42fdff5, right?\n\nNo, in fact 90dd12c is the root commit of that branch! 2c29cab is the\ntip of svn-tags/pyx_0_10.\n\nIn other words, our graph output can be very misleading in a case like\nthat. I would suggest to mark a root commit with \"o\" like we do for\nboundary commits. (tig uses \"o\" or \"*\" for ordinary commits and \"I\" for\na root.)\n\nBut I'm not sure that we don't create misleading output like that in\nother situations also, like for A@{1}..A where we produce \"virtual\nroots\" as well.\n\nThe main problem appears to be that \"*\" does not convey any connectivity\ninformation. Wondering whether we could use graphic symbols from some\ncharset...\n\nMichael\n"}]}