generated sources Was: git: 408957613bc0 - main - Regen
- Reply: Brooks Davis : "Re: generated sources Was: git: 408957613bc0 - main - Regen"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Thu, 23 May 2024 18:20:08 UTC
Hi, sorry for slightly diverting the discussion, but looks like this thread is the right place... On Wed, May 22, 2024 at 12:55:24PM -0700, John Baldwin wrote: J> I don't think it's worth reverting at this point. For `make sysent` I would J> suggest 'sysent: Regen' as a commit log in the future. We typically use J> 'src.conf.5: Regen' after running `make makeman`. On Wed, May 22, 2024 at 05:05:14PM +0000, Brooks Davis wrote: B> This is more or less standard practice for generated files. These B> commits are of basically no interest. Historically they had to exist B> because the source file $FreeBSD$ entries were embedded in the generated B> files and thus had to be final. That isn't true any more so I often just B> include them in the origional commit unless I'm doing a lot of minor B> changes to the source files. Is it possible to generate the files during normal kernel build? That would require lua packages installed, but we already almost require git installed to work with FreeBSD sources. Absence of lua can be checked in Makefile and a verbose error message would be printed. Or may be we can use the small lua that we have in the loader? Sorry if the very last suggestion was stupid - really don't have any knowledge in the area. After the xz/liblzma backdoor all the generated code committed manually makes me concerned, as it has 0% chance to get reviewed by community. Just if my suggestion sounds stupid and there is a good reason to have what we have - an automatic generation followed by a manual commit - then please explain this reason. -- Gleb Smirnoff