[darcs-devel] [patch1614] UI display of named patches / explicit dependencies

[Ben Franksen]

New submission from Ben Franksen <ben.franksen at online.de>:

Some of these patches may be controversial. Tell me what you think.

6 patches for repository bf at hub.darcs.net:darcs-current:

patch 154d95ccb23ea82ed4d98501629954823d559922
Author: Ben Franksen <ben.franksen at online.de>
Date:   Thu Oct  5 08:03:21 CEST 2017
  * moved showNamedPrefix from D.P.Show to D.P.Named

patch 7b33ae71b0411ef081b9d7922b3f01cca09b2d9f
Author: Ben Franksen <ben.franksen at online.de>
Date:   Thu Oct  5 16:51:00 CEST 2017
  * improved UI display of named patches
  
  Previously we used the same format to display the PatchInfo as for storage
  (i.e. showPatchInfo), and similarly for the PatchInfos of explicit
  dependencies. This could be seen by hitting 'v' when using 'darcs log' in
  interactive mode. We now use showPatchInfoUI when we show the
PatchInfo of a
  patch for display. Explicit dependencies are displayed less verbosely but
  still so that they can be identified easily:
  
  Explicit dependencies:
    patch <hash>
      * <patch name>
    patch <hash>
      * <patch name>
    ...

patch f5952fad159ad69ac8c734acad0da084d071136a
Author: Ben Franksen <benjamin.franksen at helmholtz-berlin.de>
Date:   Thu Oct  5 17:12:55 CEST 2017
  * yes, you should call showPatchInfo here!

patch 3a0da988b52bba2140ae9b03da95e60c934080f4
Author: Ben Franksen <benjamin.franksen at helmholtz-berlin.de>
Date:   Thu Oct  5 17:13:32 CEST 2017
  * use showPatchInfoUI, not showPatchInfo, for display

patch dcc337b03e38afbad2cf37e3120b642ea140157a
Author: Ben Franksen <benjamin.franksen at helmholtz-berlin.de>
Date:   Fri Oct  6 15:59:06 CEST 2017
  * make showPatchInfo more similar to showPatch
  
  It now takes a ShowPatchFor as first argument and showPatchInfoUI has been
  renamed to displayPatchInfo.

patch 33a95e78ebc810e0e959e1c7f9b1f3f867a5e2fd
Author: Ben Franksen <ben.franksen at online.de>
Date:   Sat Oct  7 11:02:48 CEST 2017
  * fix: log --machine-readable should use showPatch ForStorage
  
  At least this is what the tests (ask_deps.sh, look_for_moves.sh) seem to
  expect. These tests were broken by
  
    patch 7b33ae71b0411ef081b9d7922b3f01cca09b2d9f
      * improved UI display of named patches

----------
files: moved-shownamedprefix-from-d_p_show-to-d_p_named.dpatch
messages: 19719
nosy: bf
status: needs-screening
title: UI display of named patches / explicit dependencies

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/patch1614>
__________________________________
-------------- next part --------------
A non-text attachment was scrubbed...
Name: moved-shownamedprefix-from-d_p_show-to-d_p_named.dpatch
Type: application/x-darcs-patch
Size: 34909 bytes
Desc: not available
URL: <http://lists.osuosl.org/pipermail/darcs-devel/attachments/20171010/38e09a29/attachment-0001.bin>