Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-20 Thread John Mandereau
Il giorno lun, 20/08/2012 alle 08.28 +0200, David Kastrup ha scritto: > The possibly contentious case is when the user has put up an independent > .dir-local.el file. Git will complain before overwriting that without > confirmation, so you'll get warning (not for git reset --hard or similar > thou

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread David Kastrup
John Mandereau writes: > Il giorno lun, 20/08/2012 alle 00.39 +0200, David Kastrup ha scritto: >> The only reasonable way to address the amount and kind of concerns >> voiced here is not to apply the patch. Instead, one should likely >> explain in CG how to use M-x add-dir-local-variable RET to

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread John Mandereau
Il giorno lun, 20/08/2012 alle 00.39 +0200, David Kastrup ha scritto: > The only reasonable way to address the amount and kind of concerns > voiced here is not to apply the patch. Instead, one should likely > explain in CG how to use M-x add-dir-local-variable RET to achieve its > equivalent. In

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread David Kastrup
John Mandereau writes: > Il giorno dom, 19/08/2012 alle 18.06 +0100, Graham Percival ha scritto: >> I think that Ian's confusion is understandable. Files inside >> elisp/*.el are mostly not intended for developers. At least, >> stuff like elisp/lilypond-mode.el seems to be aimed directly at >>

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread John Mandereau
Il giorno dom, 19/08/2012 alle 18.06 +0100, Graham Percival ha scritto: > I think that Ian's confusion is understandable. Files inside > elisp/*.el are mostly not intended for developers. At least, > stuff like elisp/lilypond-mode.el seems to be aimed directly at > end-users, not developers. > >

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread John Mandereau
Il giorno dom, 19/08/2012 alle 21.57 +0100, Ian Hulin ha scritto: > Hi John and David, > If you take this on, John, or if David decides to revisit this, can > you add something like this to the detailed description of the issue: > > "This patch adds a new file dir-locals.el to the root directory o

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread John Mandereau
Il giorno dom, 19/08/2012 alle 22.34 +0200, David Kastrup ha scritto: > John Mandereau writes: > > emacs -batch scm/*.scm --eval '(progn (delete-trailing-whitespace) > > (indent-region (point-min) (point-max) nil) (untabify (point-min) > > (point-max)) (save-buffer))' [snip] > indent-tabs-mode

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread Ian Hulin
Hi John and David, If you take this on, John, or if David decides to revisit this, can you add something like this to the detailed description of the issue: "This patch adds a new file dir-locals.el to the root directory of the git repository. It is intended for LilyPond developers who use Emacs.

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread David Kastrup
John Mandereau writes: > Il giorno dom, 19/08/2012 alle 16.02 +, gra...@percival-music.ca ha > scritto: >> Could you add a script (either shell or python) which applies this >> formatting to all scheme files in the repository? I'd like to see how >> that changes the scheme files. > > You can

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread John Mandereau
Il giorno dom, 19/08/2012 alle 16.02 +, gra...@percival-music.ca ha scritto: > Could you add a script (either shell or python) which applies this > formatting to all scheme files in the repository? I'd like to see how > that changes the scheme files. You can try running (after having applied

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread Ian Hulin
Hi David, I thought I'd been polite in framing my questions. Statements like "Huh?" and "Huh?!?!?!?" come over as very rude. If you're puzzled why I'm asking them, just say so. See below for specifics. On 19/08/12 17:50, d...@gnu.org wrote: > On 2012/08/19 16:07:28, Ian Hulin (gmail) wrote: >>

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread Graham Percival
On Sun, Aug 19, 2012 at 04:50:54PM +, d...@gnu.org wrote: > On 2012/08/19 16:07:28, Ian Hulin (gmail) wrote: > >Where is it going to end up after a make install run and where will it > be > >delivered when the binaries are built and run? > > Huh?!?!?!? It has nothing to do whatsoever with "ma

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread dak
On 2012/08/19 16:07:28, Ian Hulin (gmail) wrote: On 2012/08/19 12:37:38, dak wrote: > I don't get the point you are trying to make. Sorry for being obtuse. I'd like your patch description to say more and give answers to some of the following questions. Who is the target audience for t

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread dak
http://codereview.appspot.com/6460109/diff/3001/.dir-locals.el File .dir-locals.el (right): http://codereview.appspot.com/6460109/diff/3001/.dir-locals.el#newcode6 .dir-locals.el:6: (indent-tabs-mode)) On 2012/08/19 16:02:31, Graham Percival wrote: Does this mean what I think it means? No, it

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread ianhulin44
On 2012/08/19 12:37:38, dak wrote: On 2012/08/19 12:12:36, Ian Hulin (gmail) wrote: > Do we need to add anything to the standard .gitignore for these? Why should we? This adds just one file which _should_ be checked out. > They're files in the git directories for developers developing the

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread graham
Could you add a script (either shell or python) which applies this formatting to all scheme files in the repository? I'd like to see how that changes the scheme files. http://codereview.appspot.com/6460109/diff/3001/.dir-locals.el File .dir-locals.el (right): http://codereview.appspot.com/646

Re: Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread dak
Reviewers: Ian Hulin (gmail), Message: On 2012/08/19 12:12:36, Ian Hulin (gmail) wrote: Do we need to add anything to the standard .gitignore for these? Why should we? This adds just one file which _should_ be checked out. They're files in the git directories for developers developing the k

Use directory-local variables to establish some coding styles in Emacs (issue 6460109)

2012-08-19 Thread ianhulin44
Do we need to add anything to the standard .gitignore for these? They're files in the git directories for developers developing the kit rather than users writing code. Quite a lot of developers have the build directory within the git tree. There are already .el files in the kit for lilypond mod