unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
* Re: [Guile-commits] GNU Guile branch, master, updated. release_1-9-11-164-gd26a26f
       [not found] <E1OXPjX-0000lG-6v@vcs-noshell.in.savannah.gnu.org>
@ 2010-07-11 22:55 ` Ludovic Courtès
  2010-07-12  0:09   ` No Itisnt
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2010-07-11 22:55 UTC (permalink / raw)
  To: No Itisnt; +Cc: guile-devel

Hi!

"No Itisnt" <theseaisinhere+git@gmail.com> writes:

> commit d26a26f6c0fbdb971995d1b3dcf3345831eb12d7
> Author: No Itisnt <theseaisinhere+git@gmail.com>
> Date:   Fri Jul 9 21:22:27 2010 -0500
>
>     Remove trailing whitespace
>     
>     * module/language/tree-il.scm: Remove trailing whitespace

Unless you have an explicit OK, please avoid commits “just” to remove
whitespace.

I think people on this list had +/- agreed that trailing whitespaces are
Bad(TM), and that we would avoid them (e.g., using an Emacs hook), but
that we could nonetheless live with them and would avoid whitespace
commits.

Thanks,
Ludo’.



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Guile-commits] GNU Guile branch, master, updated.  release_1-9-11-164-gd26a26f
  2010-07-11 22:55 ` [Guile-commits] GNU Guile branch, master, updated. release_1-9-11-164-gd26a26f Ludovic Courtès
@ 2010-07-12  0:09   ` No Itisnt
  2010-07-12  0:19     ` No Itisnt
  2010-07-12 12:01     ` Ludovic Courtès
  0 siblings, 2 replies; 4+ messages in thread
From: No Itisnt @ 2010-07-12  0:09 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guile-devel, No Itisnt

Hi, yes that was part of a patch Andy approved but I never committed
the second part due to issues.

On Sun, Jul 11, 2010 at 5:55 PM, Ludovic Courtès <ludo@gnu.org> wrote:
> Hi!
>
> "No Itisnt" <theseaisinhere+git@gmail.com> writes:
>
>> commit d26a26f6c0fbdb971995d1b3dcf3345831eb12d7
>> Author: No Itisnt <theseaisinhere+git@gmail.com>
>> Date:   Fri Jul 9 21:22:27 2010 -0500
>>
>>     Remove trailing whitespace
>>
>>     * module/language/tree-il.scm: Remove trailing whitespace
>
> Unless you have an explicit OK, please avoid commits “just” to remove
> whitespace.
>
> I think people on this list had +/- agreed that trailing whitespaces are
> Bad(TM), and that we would avoid them (e.g., using an Emacs hook), but
> that we could nonetheless live with them and would avoid whitespace
> commits.
>
> Thanks,
> Ludo’.
>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Guile-commits] GNU Guile branch, master, updated.  release_1-9-11-164-gd26a26f
  2010-07-12  0:09   ` No Itisnt
@ 2010-07-12  0:19     ` No Itisnt
  2010-07-12 12:01     ` Ludovic Courtès
  1 sibling, 0 replies; 4+ messages in thread
From: No Itisnt @ 2010-07-12  0:19 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guile-devel, No Itisnt

See "[PATCH] Allow printing of malformed tree-il"

On Sun, Jul 11, 2010 at 7:09 PM, No Itisnt <theseaisinhere@gmail.com> wrote:
> Hi, yes that was part of a patch Andy approved but I never committed
> the second part due to issues.
>
> On Sun, Jul 11, 2010 at 5:55 PM, Ludovic Courtès <ludo@gnu.org> wrote:
>> Hi!
>>
>> "No Itisnt" <theseaisinhere+git@gmail.com> writes:
>>
>>> commit d26a26f6c0fbdb971995d1b3dcf3345831eb12d7
>>> Author: No Itisnt <theseaisinhere+git@gmail.com>
>>> Date:   Fri Jul 9 21:22:27 2010 -0500
>>>
>>>     Remove trailing whitespace
>>>
>>>     * module/language/tree-il.scm: Remove trailing whitespace
>>
>> Unless you have an explicit OK, please avoid commits “just” to remove
>> whitespace.
>>
>> I think people on this list had +/- agreed that trailing whitespaces are
>> Bad(TM), and that we would avoid them (e.g., using an Emacs hook), but
>> that we could nonetheless live with them and would avoid whitespace
>> commits.
>>
>> Thanks,
>> Ludo’.
>>
>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Guile-commits] GNU Guile branch, master, updated.  release_1-9-11-164-gd26a26f
  2010-07-12  0:09   ` No Itisnt
  2010-07-12  0:19     ` No Itisnt
@ 2010-07-12 12:01     ` Ludovic Courtès
  1 sibling, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2010-07-12 12:01 UTC (permalink / raw)
  To: guile-devel

No Itisnt <theseaisinhere@gmail.com> writes:

> Hi, yes that was part of a patch Andy approved but I never committed
> the second part due to issues.

OK, fine then!  :-)

Ludo’.




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-07-12 12:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <E1OXPjX-0000lG-6v@vcs-noshell.in.savannah.gnu.org>
2010-07-11 22:55 ` [Guile-commits] GNU Guile branch, master, updated. release_1-9-11-164-gd26a26f Ludovic Courtès
2010-07-12  0:09   ` No Itisnt
2010-07-12  0:19     ` No Itisnt
2010-07-12 12:01     ` Ludovic Courtès

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).