kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #41329
Re: [PATCH 1/1] Fix build order for generated headers and sources
Seth,
On 7/2/2019 12:36 PM, Seth Hillbrand wrote:
> On 2019-07-02 12:21, Simon Richter wrote:
>> Hi,
>>
>> On Tue, Jul 02, 2019 at 11:42:30AM -0400, Seth Hillbrand wrote:
>>
>>> Instead of banging on this dependency, I could push option #2 from
>>> my previous e-mail. This was creating a separate target to rebuild
>>> the generated files and checking them into the source tree. Then we
>>> don't try to rebuild them as part of our standard build. This is
>>> analogous to how we handle the bitmaps.
>>
>> If we could avoid that, that would be nice, because it'd be another
>> special
>> thing for people to know. We mainly do this for the bitmaps to avoid a
>> dependency on inkscape and pngcrush, and there seems to be a
>> nondeterministic element in there as well.
>>
>> The reason we have our own lexer is to avoid the external tool dependency
>> and still allow people to just change the word list. Otherwise, I'd just
>> convert all of this to GNU flex.
>>
>> The last patch seems to have built correctly for me, so there is a good
>> chance that this works now. :)
>>
>> Simon
>
> Oh, I missed that there was a patch attached to the last e-mail. Is
> there any chance you could output that with git format-patch? I can't
> apply it as attached, I get 'Patch format detection failed'.
>
> -Seth
I think Simon uses format-patch to inline the patch in the email message
so you can just save the entire message to disk and then do a `git am`
on the message.
Cheers,
Wayne
References