Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean
@ 2026-08-20 18:20 Simon Marchi
  2026-08-20 18:20 ` [PATCH 2/2] gdb: remove stale entries from gdb/.gitignore Simon Marchi
  2026-08-20 21:07 ` [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean Kevin Buettner
  0 siblings, 2 replies; 5+ messages in thread
From: Simon Marchi @ 2026-08-20 18:20 UTC (permalink / raw)
  To: gdb-patches; +Cc: Simon Marchi

The local-maintainer-clean target removes a hardcoded list of
parser-generator output files, and then removes $(YYFILES) two lines
below.  $(YYFILES) contains the same list of files, so the first rm is
redundant.  Remove it.

Change-Id: I14d27930953e33d54010d964cb4bb7db4e8323a0
---
 gdb/Makefile.in | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/gdb/Makefile.in b/gdb/Makefile.in
index 57f384170abf..0457ad7926da 100644
--- a/gdb/Makefile.in
+++ b/gdb/Makefile.in
@@ -2441,10 +2441,6 @@ realclean: maintainer-clean
 local-maintainer-clean:
 	@echo "This command is intended for maintainers to use;"
 	@echo "it deletes files that may require special tools to rebuild."
-	rm -f c-exp.c \
-		cp-name-parser.c \
-		ada-lex.c ada-exp.c \
-		d-exp.c f-exp.c go-exp.c m2-exp.c p-exp.c
 	rm -f TAGS
 	rm -f $(YYFILES)
 	rm -f nm.h config.status

base-commit: d2102b0d6a95ba3b37204976fb3f6ebd935822fb
-- 
2.55.0


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 2/2] gdb: remove stale entries from gdb/.gitignore
  2026-08-20 18:20 [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean Simon Marchi
@ 2026-08-20 18:20 ` Simon Marchi
  2026-08-20 21:08   ` Kevin Buettner
  2026-08-20 21:07 ` [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean Kevin Buettner
  1 sibling, 1 reply; 5+ messages in thread
From: Simon Marchi @ 2026-08-20 18:20 UTC (permalink / raw)
  To: gdb-patches; +Cc: Simon Marchi

Remove some entries in gdb/.gitignore that refer to files that are no
longer generated.

Change-Id: Ic6d7863c5fee61867d0ab4f7a8609e4996da4a2b
---
 gdb/.gitignore | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/gdb/.gitignore b/gdb/.gitignore
index b9b2c6f2d192..a68ed4e33178 100644
--- a/gdb/.gitignore
+++ b/gdb/.gitignore
@@ -1,5 +1,4 @@
 /Makefile
-/build-gnulib
 /gdb-gdb.gdb
 /nm.h
 
@@ -13,14 +12,9 @@
 /d-exp.c
 /f-exp.c
 /gdb
-/gdbtui
 /gcore
 /go-exp.c
 /init.c
 /jit-reader.h
-/jv-exp.c
 /m2-exp.c
-/objc-exp.c
-/observer.inc
 /p-exp.c
-/rust-exp.c
-- 
2.55.0


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean
  2026-08-20 18:20 [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean Simon Marchi
  2026-08-20 18:20 ` [PATCH 2/2] gdb: remove stale entries from gdb/.gitignore Simon Marchi
@ 2026-08-20 21:07 ` Kevin Buettner
  2026-08-20 21:50   ` Simon Marchi
  1 sibling, 1 reply; 5+ messages in thread
From: Kevin Buettner @ 2026-08-20 21:07 UTC (permalink / raw)
  To: gdb-patches; +Cc: Simon Marchi

On Thu, 20 Aug 2026 14:20:23 -0400
Simon Marchi <simon.marchi@efficios.com> wrote:

> The local-maintainer-clean target removes a hardcoded list of
> parser-generator output files, and then removes $(YYFILES) two lines
> below.  $(YYFILES) contains the same list of files, so the first rm is
> redundant.  Remove it.

LGTM.

Approved-By: Kevin Buettner <kevinb@redhat.com>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 2/2] gdb: remove stale entries from gdb/.gitignore
  2026-08-20 18:20 ` [PATCH 2/2] gdb: remove stale entries from gdb/.gitignore Simon Marchi
@ 2026-08-20 21:08   ` Kevin Buettner
  0 siblings, 0 replies; 5+ messages in thread
From: Kevin Buettner @ 2026-08-20 21:08 UTC (permalink / raw)
  To: gdb-patches; +Cc: Simon Marchi

On Thu, 20 Aug 2026 14:20:24 -0400
Simon Marchi <simon.marchi@efficios.com> wrote:

> Remove some entries in gdb/.gitignore that refer to files that are no
> longer generated.

LGTM.

Approved-By: Kevin Buettner <kevinb@redhat.com>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean
  2026-08-20 21:07 ` [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean Kevin Buettner
@ 2026-08-20 21:50   ` Simon Marchi
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Marchi @ 2026-08-20 21:50 UTC (permalink / raw)
  To: Kevin Buettner, gdb-patches; +Cc: Simon Marchi



On 2026-08-20 17:07, Kevin Buettner wrote:
> On Thu, 20 Aug 2026 14:20:23 -0400
> Simon Marchi <simon.marchi@efficios.com> wrote:
> 
>> The local-maintainer-clean target removes a hardcoded list of
>> parser-generator output files, and then removes $(YYFILES) two lines
>> below.  $(YYFILES) contains the same list of files, so the first rm is
>> redundant.  Remove it.
> 
> LGTM.
> 
> Approved-By: Kevin Buettner <kevinb@redhat.com>
> 

Thanks, pushed both.

Simon

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2026-08-20 21:50 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-08-20 18:20 [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean Simon Marchi
2026-08-20 18:20 ` [PATCH 2/2] gdb: remove stale entries from gdb/.gitignore Simon Marchi
2026-08-20 21:08   ` Kevin Buettner
2026-08-20 21:07 ` [PATCH 1/2] gdb: remove redundant rm in local-maintainer-clean Kevin Buettner
2026-08-20 21:50   ` Simon Marchi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox