diff mbox series

[2/2] isar-exclude-docs: delete also links not just files

Message ID 20220923145332.8410-2-henning.schild@siemens.com
State Accepted, archived
Headers show
Series [1/2] isar-exclude-docs: remove only files in postinst | expand

Commit Message

Henning Schild Sept. 23, 2022, 2:53 p.m. UTC 
From: "Roberto A. Foglietta" <roberto.foglietta@gmail.com>

Do not match files but instead everything but directories. And also drop
the "-r" for the rm it is not needed.

Signed-off-by: Roberto A. Foglietta <roberto.foglietta@gmail.com>
Signed-off-by: Henning Schild <henning.schild@siemens.com>
---
 meta/recipes-support/isar-exclude-docs/files/postinst         | 4 ++--
 ...{isar-exclude-docs_0.2.1.bb => isar-exclude-docs_0.2.2.bb} | 0
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-support/isar-exclude-docs/{isar-exclude-docs_0.2.1.bb => isar-exclude-docs_0.2.2.bb} (100%)

Comments

Henning Schild Sept. 23, 2022, 4:30 p.m. UTC | #1 
I tried that manually and the result looks good. I also tested the
installation of "openjdk-11-jre-headless", did not work before as
reported by Roberto, works after.

It is really about doing what that dpkg.conf does, not about making
sure that package can be installed .. but it can now.

The ubuntu scripts make sure all is like dpkg.conf by re-installing all
packages that deployed anything to the excluded paths in their
containers and livecds. But that is really not nice so i would not want
to copy that pattern.

https://git.launchpad.net/livecd-rootfs/tree/live-build/auto/build#n197

Henning

Am Fri, 23 Sep 2022 16:53:32 +0200
schrieb Henning Schild <henning.schild@siemens.com>:

> From: "Roberto A. Foglietta" <roberto.foglietta@gmail.com>
> 
> Do not match files but instead everything but directories. And also
> drop the "-r" for the rm it is not needed.
> 
> Signed-off-by: Roberto A. Foglietta <roberto.foglietta@gmail.com>
> Signed-off-by: Henning Schild <henning.schild@siemens.com>
> ---
>  meta/recipes-support/isar-exclude-docs/files/postinst         | 4
> ++-- ...{isar-exclude-docs_0.2.1.bb => isar-exclude-docs_0.2.2.bb} | 0
>  2 files changed, 2 insertions(+), 2 deletions(-)
>  rename
> meta/recipes-support/isar-exclude-docs/{isar-exclude-docs_0.2.1.bb =>
> isar-exclude-docs_0.2.2.bb} (100%)
> 
> diff --git a/meta/recipes-support/isar-exclude-docs/files/postinst
> b/meta/recipes-support/isar-exclude-docs/files/postinst index
> 581cdc1339e8..78e5d58354ea 100644 ---
> a/meta/recipes-support/isar-exclude-docs/files/postinst +++
> b/meta/recipes-support/isar-exclude-docs/files/postinst @@ -3,5 +3,5
> @@ # time and do not need to be "the first" package
>  # what we delete needs to be in sync with the dpkg configuration we
> ship 
> -find /usr/share/man/ -type f -exec rm -rf {} \;
> -find /usr/share/doc/ -type f ! -name "copyright" ! -name
> "changelog.*" -exec rm -rf {} \; +find /usr/share/man/ ! -type d
> -exec rm -f {} \; +find /usr/share/doc/ ! -type d ! -name "copyright"
> ! -name "changelog.*" -exec rm -f {} \; diff --git
> a/meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.1.bb
> b/meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.2.bb
> similarity index 100% rename from
> meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.1.bb
> rename to
> meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.2.bb
Roberto A. Foglietta Sept. 23, 2022, 6:07 p.m. UTC | #2 
Il giorno ven 23 set 2022 alle ore 18:30 Henning Schild <
henning.schild@siemens.com> ha scritto:

> I tried that manually and the result looks good. I also tested the
> installation of "openjdk-11-jre-headless", did not work before as
> reported by Roberto, works after.
>
> It is really about doing what that dpkg.conf does, not about making
> sure that package can be installed .. but it can now.
>
> The ubuntu scripts make sure all is like dpkg.conf by re-installing all
> packages that deployed anything to the excluded paths in their
> containers and livecds. But that is really not nice so i would not want
> to copy that pattern.
>
> https://git.launchpad.net/livecd-rootfs/tree/live-build/auto/build#n197


Thanks a lot for your effort in going deep into the question and verifying
the outcome. I appreciate that. R-
diff mbox series

Patch

diff --git a/meta/recipes-support/isar-exclude-docs/files/postinst b/meta/recipes-support/isar-exclude-docs/files/postinst
index 581cdc1339e8..78e5d58354ea 100644
--- a/meta/recipes-support/isar-exclude-docs/files/postinst
+++ b/meta/recipes-support/isar-exclude-docs/files/postinst
@@ -3,5 +3,5 @@ 
 # time and do not need to be "the first" package
 # what we delete needs to be in sync with the dpkg configuration we ship
 
-find /usr/share/man/ -type f -exec rm -rf {} \;
-find /usr/share/doc/ -type f ! -name "copyright" ! -name "changelog.*" -exec rm -rf {} \;
+find /usr/share/man/ ! -type d -exec rm -f {} \;
+find /usr/share/doc/ ! -type d ! -name "copyright" ! -name "changelog.*" -exec rm -f {} \;
diff --git a/meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.1.bb b/meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.2.bb
similarity index 100%
rename from meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.1.bb
rename to meta/recipes-support/isar-exclude-docs/isar-exclude-docs_0.2.2.bb