public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
* [PATCH] linux-custom: fix for kernels without modules
@ 2019-02-26 16:46 Cedric Hombourger
  2019-02-26 16:53 ` Jan Kiszka
  2019-03-14 19:19 ` Maxim Yu. Osipov
  0 siblings, 2 replies; 3+ messages in thread
From: Cedric Hombourger @ 2019-02-26 16:46 UTC (permalink / raw)
  To: isar-users; +Cc: Cedric Hombourger

build-kernel.sh would try to create a .fresh_install file in the
/lib/modules/${PV} folder to replicate Debian's postinst magic
but would fail when the custom kernel is configured without any
modules. Create /lib/modules/${PV} before .fresh_install to make
sure its target directory exists.

Signed-off-by: Cedric Hombourger <Cedric_Hombourger@mentor.com>
---
 meta/recipes-kernel/linux/files/build-kernel.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-kernel/linux/files/build-kernel.sh b/meta/recipes-kernel/linux/files/build-kernel.sh
index 6e4d347..842b585 100644
--- a/meta/recipes-kernel/linux/files/build-kernel.sh
+++ b/meta/recipes-kernel/linux/files/build-kernel.sh
@@ -76,6 +76,7 @@ dpkg-gencontrol -crepack/debian/control \
 	-DArchitecture=$target_arch
 
 # Add Debian-like link installation to postinst
+mkdir -p ${REPACK_LINUX_IMAGE_DIR}/lib/modules/${PV}
 touch "${REPACK_LINUX_IMAGE_DIR}/lib/modules/${PV}/.fresh-install"
 sed -i "${REPACK_LINUX_IMAGE_DIR}/DEBIAN/postinst" \
     -e "/^set -e$/a\\
-- 
2.11.0


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

* Re: [PATCH] linux-custom: fix for kernels without modules
  2019-02-26 16:46 [PATCH] linux-custom: fix for kernels without modules Cedric Hombourger
@ 2019-02-26 16:53 ` Jan Kiszka
  2019-03-14 19:19 ` Maxim Yu. Osipov
  1 sibling, 0 replies; 3+ messages in thread
From: Jan Kiszka @ 2019-02-26 16:53 UTC (permalink / raw)
  To: Cedric Hombourger, isar-users

On 26.02.19 17:46, Cedric Hombourger wrote:
> build-kernel.sh would try to create a .fresh_install file in the
> /lib/modules/${PV} folder to replicate Debian's postinst magic
> but would fail when the custom kernel is configured without any
> modules. Create /lib/modules/${PV} before .fresh_install to make
> sure its target directory exists.
> 
> Signed-off-by: Cedric Hombourger <Cedric_Hombourger@mentor.com>
> ---
>   meta/recipes-kernel/linux/files/build-kernel.sh | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-kernel/linux/files/build-kernel.sh b/meta/recipes-kernel/linux/files/build-kernel.sh
> index 6e4d347..842b585 100644
> --- a/meta/recipes-kernel/linux/files/build-kernel.sh
> +++ b/meta/recipes-kernel/linux/files/build-kernel.sh
> @@ -76,6 +76,7 @@ dpkg-gencontrol -crepack/debian/control \
>   	-DArchitecture=$target_arch
>   
>   # Add Debian-like link installation to postinst
> +mkdir -p ${REPACK_LINUX_IMAGE_DIR}/lib/modules/${PV}
>   touch "${REPACK_LINUX_IMAGE_DIR}/lib/modules/${PV}/.fresh-install"
>   sed -i "${REPACK_LINUX_IMAGE_DIR}/DEBIAN/postinst" \
>       -e "/^set -e$/a\\
> 

Makes sense, looks good.

Jan

-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux

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

* Re: [PATCH] linux-custom: fix for kernels without modules
  2019-02-26 16:46 [PATCH] linux-custom: fix for kernels without modules Cedric Hombourger
  2019-02-26 16:53 ` Jan Kiszka
@ 2019-03-14 19:19 ` Maxim Yu. Osipov
  1 sibling, 0 replies; 3+ messages in thread
From: Maxim Yu. Osipov @ 2019-03-14 19:19 UTC (permalink / raw)
  To: Cedric Hombourger, isar-users

On 2/26/19 5:46 PM, Cedric Hombourger wrote:
> build-kernel.sh would try to create a .fresh_install file in the
> /lib/modules/${PV} folder to replicate Debian's postinst magic
> but would fail when the custom kernel is configured without any
> modules. Create /lib/modules/${PV} before .fresh_install to make
> sure its target directory exists.

Applied to the 'next',

Thanks,
Maxim.

> Signed-off-by: Cedric Hombourger <Cedric_Hombourger@mentor.com>
> ---
>   meta/recipes-kernel/linux/files/build-kernel.sh | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-kernel/linux/files/build-kernel.sh b/meta/recipes-kernel/linux/files/build-kernel.sh
> index 6e4d347..842b585 100644
> --- a/meta/recipes-kernel/linux/files/build-kernel.sh
> +++ b/meta/recipes-kernel/linux/files/build-kernel.sh
> @@ -76,6 +76,7 @@ dpkg-gencontrol -crepack/debian/control \
>   	-DArchitecture=$target_arch
>   
>   # Add Debian-like link installation to postinst
> +mkdir -p ${REPACK_LINUX_IMAGE_DIR}/lib/modules/${PV}
>   touch "${REPACK_LINUX_IMAGE_DIR}/lib/modules/${PV}/.fresh-install"
>   sed -i "${REPACK_LINUX_IMAGE_DIR}/DEBIAN/postinst" \
>       -e "/^set -e$/a\\
> 


-- 
Maxim Osipov
ilbers GmbH
Maria-Merian-Str. 8
85521 Ottobrunn
Germany
+49 (151) 6517 6917
mosipov@ilbers.de
http://ilbers.de/
Commercial register Munich, HRB 214197
General Manager: Baurzhan Ismagulov

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

end of thread, other threads:[~2019-03-14 19:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-26 16:46 [PATCH] linux-custom: fix for kernels without modules Cedric Hombourger
2019-02-26 16:53 ` Jan Kiszka
2019-03-14 19:19 ` Maxim Yu. Osipov

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