On Tue, Aug 20, 2013 at 2:11 PM, Stephan Beal <[email protected]> wrote:

> i've just written down a TODO to add this info to the timeline if it's not
> terribly problematic. The main problem is that the timeline doesn't have
> that level of data. Ah, but it could - we could still see the PGP key in
> the crosslink phase, i think, and that's where the timeline is updated,
> anyway.
>
> Written down on my TODO list.
>

i love low-hanging fruit...

http://fossil-scm.org/index.html/timeline?r=timeline-pgp-marker

i'm not entirely satisfied with this solution, so i've put it into a branch
and am hoping someone has a prettier/better suggestion. What i've done is
during manifest parsing, set a flag if we see a PGP key, and add "(*PGP
SIGNED*)" to the event.comment (not ecomment) if that flag is set. e.g.

[507ee45f25] <http://localhost:8080/info/507ee45f25> Fix an off-by-one bug
in the network protocol handler so that it can accept a zero-length file.
(*PGP SIGNED*) (user:
drh<http://localhost:8080/timeline?u=drh&c=2007-08-25+12%3A31%3A55&nd>,
tags: trunk<http://localhost:8080/timeline?r=trunk&nd&c=2007-08-25+12%3A31%3A55>
)04:02
[9b30224db7] <http://localhost:8080/info/9b30224db7> Closed-Leaf: Merging
formatting changes to timeline and concepts documentation (*PGP SIGNED*)
(user: aku<http://localhost:8080/timeline?u=aku&c=2007-08-25+04%3A02%3A27&nd>,
tags: trunk<http://localhost:8080/timeline?r=trunk&nd&c=2007-08-25+04%3A02%3A27>
)


That didn't paste so well, but you get the idea.

Caveat: this marker only applies to new commits or after a rebuild (then
it's retroactive).

Can you please try that out, Gour?

-- 
----- stephan beal
http://wanderinghorse.net/home/stephan/
http://gplus.to/sgbeal
_______________________________________________
fossil-users mailing list
[email protected]
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to