Re: [PATCH 1/3] sb/setbuilder: Do not install if --no-install option is used

2022-09-15 Thread Chris Johns
Hi,

There is a bug in this change. I will sort it out today.

Sorry about this.

Chris

On 14/9/22 5:02 pm, chr...@rtems.org wrote:
> From: Chris Johns 
> 
> - This is a bug introduced in the recent bset tar file changes
> ---
>  source-builder/sb/setbuilder.py | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/source-builder/sb/setbuilder.py b/source-builder/sb/setbuilder.py
> index 67b66f5..f24ac87 100644
> --- a/source-builder/sb/setbuilder.py
> +++ b/source-builder/sb/setbuilder.py
> @@ -560,7 +560,8 @@ class buildset:
>  buildroot = 
> path.join(b.config.expand('%{buildroot}'), prefix)
>  if self.staging():
>  prefix = b.config.expand('%{stagingroot}')
> -self.install(self.install_mode(), b.name(), 
> buildroot, prefix)
> +if not self.opts.no_install():
> +self.install(self.install_mode(), b.name(), 
> buildroot, prefix)
>  #
>  # Sizes ...
>  #
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


[PATCH 1/3] sb/setbuilder: Do not install if --no-install option is used

2022-09-14 Thread chrisj
From: Chris Johns 

- This is a bug introduced in the recent bset tar file changes
---
 source-builder/sb/setbuilder.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/source-builder/sb/setbuilder.py b/source-builder/sb/setbuilder.py
index 67b66f5..f24ac87 100644
--- a/source-builder/sb/setbuilder.py
+++ b/source-builder/sb/setbuilder.py
@@ -560,7 +560,8 @@ class buildset:
 buildroot = path.join(b.config.expand('%{buildroot}'), 
prefix)
 if self.staging():
 prefix = b.config.expand('%{stagingroot}')
-self.install(self.install_mode(), b.name(), buildroot, 
prefix)
+if not self.opts.no_install():
+self.install(self.install_mode(), b.name(), 
buildroot, prefix)
 #
 # Sizes ...
 #
-- 
2.37.1

___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel