[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
- Re: [PATCH] Add .gitignore file,
Mario Domenech Goulart <=