[PATCH 0/1] Fix doc directory cleanup

[PATCH 0/1] Fix doc directory cleanup

[Elena Zannoni]

Currently, uninstalling the dtrace RPM leaves an 
empty doc directory behind. Such as:

$ ls /usr/share/doc/dtrace-2.0.3/
INCOMPATIBILITIES  NEWS  README  showUSDT
$ sudo rpm -e dtrace-2.0.3
Removed '/etc/systemd/system/basic.target.wants/dtprobed.service'.
Removed '/etc/systemd/system/basic.target.wants/dtrace-usdt.target'.
$ ls /usr/share/doc/dtrace-2.0.3/
$

This patch changes the list of files to include the directory
itself, not just its contents.

After this patch the behavior is:

$ ls /usr/share/doc/dtrace-2.0.3/
INCOMPATIBILITIES  NEWS  README  showUSDT
$ sudo rpm -e dtrace-2.0.3
Removed '/etc/systemd/system/basic.target.wants/dtprobed.service'.
Removed '/etc/systemd/system/basic.target.wants/dtrace-usdt.target'.
$ ls /usr/share/doc/dtrace-2.0.3/
ls: cannot access '/usr/share/doc/dtrace-2.0.3/': No such file or directory


Elena Zannoni (1):
  Make sure the doc dir is cleaned up on RPM uninstall

 dtrace.spec | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.49.0

[PATCH 1/1] Make sure the doc dir is cleaned up on RPM uninstall

[Elena Zannoni]

 Add to the RPM file list the entire dtrace doc directory,
 not just its contents

Signed-off-by: Elena Zannoni <elena.zannoni@oracle.com>
---
 dtrace.spec | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dtrace.spec b/dtrace.spec
index 115cea60..a325993a 100644
--- a/dtrace.spec
+++ b/dtrace.spec
@@ -211,7 +211,7 @@ systemctl start dtprobed || :
 %{_datadir}/pkgconfig/dtrace_sdt.pc
 %{_mandir}/man8/dtrace.8.gz
 %{_mandir}/man8/dtprobed.8.gz
-%doc %{_docdir}/dtrace-%{version}/*
+%doc %{_docdir}/dtrace-%{version}
 %{_unitdir}/dtprobed.service
 %{_unitdir}/dtrace-usdt.target
 %{_presetdir}/50-dtprobed.preset
-- 
2.49.0

Re: [PATCH 1/1] Make sure the doc dir is cleaned up on RPM uninstall

[Nick Alcock]

On 12 Sep 2025, Elena Zannoni told this:

>  Add to the RPM file list the entire dtrace doc directory,
>  not just its contents
>
> Signed-off-by: Elena Zannoni <elena.zannoni@oracle.com>

Reviewed-by: Nick Alcock <nick.alcock@oracle.com>

-- 
NULL && (void)