Hi Bruno,

In your fix for the sys_un module you added this to the module
description:

 MOSTLYCLEANFILES += sys/un.h sys/un.h-t
+MOSTLYCLEANDIRS += sys

I didn't know/forgot MOSTLYCLEANDIRS existed. I noticed it also missing
in sys_types and sys_time module. I've pushed this so it gets cleaned
even if no other sys_* modules are in use.

Collin

>From ece92ff7d380b2ea96d272109eea0062b909e945 Mon Sep 17 00:00:00 2001
From: Collin Funk <collin.fu...@gmail.com>
Date: Fri, 26 Jul 2024 18:37:09 -0700
Subject: [PATCH] sys_time, sys_types: Remove sys directory at 'make
 mostlyclean'.

* modules/sys_time (Makefile.am): Add sys to MOSTLYCLEANDIRS.
* modules/sys_types (Makefile.am): Likewise.
---
 ChangeLog         | 6 ++++++
 modules/sys_time  | 1 +
 modules/sys_types | 1 +
 3 files changed, 8 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index a9fa667880..b0170a81a4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2024-07-26  Collin Funk  <collin.fu...@gmail.com>
+
+	sys_time, sys_types: Remove sys directory at 'make mostlyclean'.
+	* modules/sys_time (Makefile.am): Add sys to MOSTLYCLEANDIRS.
+	* modules/sys_types (Makefile.am): Likewise.
+
 2024-07-26  Bruno Haible  <br...@clisp.org>
 
 	sys_un: Avoid compilation error in C++ <string.h> on glibc systems.
diff --git a/modules/sys_time b/modules/sys_time
index 73965fd954..08bc2a23d1 100644
--- a/modules/sys_time
+++ b/modules/sys_time
@@ -44,6 +44,7 @@ sys/time.h: sys_time.in.h $(top_builddir)/config.status $(CXXDEFS_H) $(ARG_NONNU
 	      $(srcdir)/sys_time.in.h > $@-t
 	$(AM_V_at)mv $@-t $@
 MOSTLYCLEANFILES += sys/time.h sys/time.h-t
+MOSTLYCLEANDIRS += sys
 
 Include:
 <sys/time.h>
diff --git a/modules/sys_types b/modules/sys_types
index 3518b17dbe..b878a03e1b 100644
--- a/modules/sys_types
+++ b/modules/sys_types
@@ -40,6 +40,7 @@ sys/types.h: sys_types.in.h $(top_builddir)/config.status
 	      $(srcdir)/sys_types.in.h > $@-t
 	$(AM_V_at)mv $@-t $@
 MOSTLYCLEANFILES += sys/types.h sys/types.h-t
+MOSTLYCLEANDIRS += sys
 
 Include:
 <sys/types.h>
-- 
2.45.2

Reply via email to