Re: [emacs-leuven-theme] diff-index foreground (#2bis)

[Dmitry Gutov]

On 01.10.2013 22:56, Fabrice Niessen wrote:
> > - time-stamp-* local variables at the bottom of the file.
>
> ... *thanks to* the local variables:
>
>    ╭────
>    │ ;; Local Variables:
>    │ ;; time-stamp-format: "%:y%02m%02d.%02H%02M"
>    │ ;; time-stamp-start: "Version: "
>    │ ;; time-stamp-end: "$"
>    │ ;; End:
>    ╰────
>
> This instructs the time-stamp package to update the "Version:" time stamp every
> time you save the buffer ("last change"), if you have the following in your
> `.emacs' config file:
>
>    (add-hook 'before-save-hook 'time-stamp)

I see, thanks for the explanation.

> > Melpa does not use the version header (just sets the version to the time of the
> > last commit), and the time-stamp vars just seem useless to me.
> >
> > Shall I remove both of these additions before checking in the new version?
>
> This version number seems useful to me, to always know when different versions
> have been saved, even outside of any VC system.

IMHO, the "last save time" alone is pretty useless, so one shouldn't 
modify important files outside of VS systems.

> May I ask to leave them, if possible?

Sure, I guess, since it serves some purpose. I don't think we have any 
guidelines about that.