[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Make define-builtin-markup{, -list}-command #:category #:properties
From: |
Carl Sorensen |
Subject: |
Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048) |
Date: |
Fri, 4 Dec 2009 08:55:12 -0700 |
On 12/4/09 12:02 AM, "David Kastrup" <address@hidden> wrote:
> Carl Sorensen <address@hidden> writes:
>
>> On 12/3/09 2:47 PM, "David Kastrup" <address@hidden> wrote:
>>
>>> There are patches that are "obviously right" and a direct
>>> improvement. If I had been in his place, I'd likely have committed a
>>> fix as well. I'd likely have used "make && make doc" before doing
>>> so, but I doubt I would have waited for a review to come in.
>>
>> make && make doc worked, because the snippet was still in the database
>> and not updated.
>
> I was only talking about the commit that did not compile. The general
> functionality patch went through review AFAICS and was basically just
> "cosmetically" changed.
>
The patch that did not compile got messed up in the patching. I'm not sure
exactly what I did wrong in patching and submitting, but the patch was one
revision earlier than my final code. So I had actually tested the code, but
somehow made an error with git.
Carl
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), (continued)
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), David Kastrup, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), Graham Percival, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), David Kastrup, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), Graham Percival, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), David Kastrup, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), Neil Puttock, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), David Kastrup, 2009/12/04
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), David Kastrup, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), Carl Sorensen, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), David Kastrup, 2009/12/04
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048),
Carl Sorensen <=
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), Carl Sorensen, 2009/12/03
- Re: Make define-builtin-markup{, -list}-command #:category #:properties keywords (issue160048), David Kastrup, 2009/12/03