chicken-hackers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Add .gitignore file


From: Mario Domenech Goulart
Subject: Re: [PATCH] Add .gitignore file
Date: Tue, 02 May 2023 21:41:08 +0200

Hi,

On Thu, 27 Apr 2023 12:44:58 +0200 felix.winkelmann@bevuta.com wrote:

>> Here's a patch that adds a .gitignore file at the root of the 
>> chicken-core repo, so that `git status` doesn't show generated files 
>> that are not supposed to be committed to the repo.
>> 
>> It doesn't seem possible to ignore *.import.scm wholesale since a few of 
>> those files are committed:
>> 
>> $ git ls-files | grep -F .import.scm
>> chicken.base.import.scm
>> chicken.condition.import.scm
>> chicken.csi.import.scm
>> chicken.foreign.import.scm
>> chicken.syntax.import.scm
>> chicken.time.import.scm
>> 
>> Likewise, *.c cannot be ignored wholesale since some hand-written C 
>> source files are present in the same directory as the C files output by csc.
>
> Thanks, Lassi!
>
> Here is a signed-off version. I first considered this straightforward,
> but after some discussion on IRC it seems that this may change the
> workflow in subtle or unexpected ways, so I leave this open for the
> moment. 

Many thanks, Lassi.  I've pushed your patch.

I noticed that files produced by tests are not ignored (also .{exe,dll}
on Windows).  Not sure if that's intentional or not.  Anyway, we can
always ignore them later, if desired.

Too bad your patch missed the 10-year anniversary of
https://lists.gnu.org/archive/html/chicken-hackers/2012-10/msg00048.html :-)

All the best.
Mario
-- 
http://parenteses.org/mario



reply via email to

[Prev in Thread] Current Thread [Next in Thread]