Patchwork cpanminus: mark as broken

login
register
mail settings
Submitter Thomas Petazzoni
Date Nov. 16, 2012, 2:01 p.m.
Message ID <1353074474-1684-1-git-send-email-thomas.petazzoni@free-electrons.com>
Download mbox | patch
Permalink /patch/199618/
State Accepted
Commit 30d99041ce5f32c30c470c3b114d612ba26e49ef
Headers show

Comments

Thomas Petazzoni - Nov. 16, 2012, 2:01 p.m.
cpanminus is currently responsible for about half of the autobuilder
failures, due to the missing dependency on host-qemu. However, even
with the host-qemu proposed by Arnout, cpanminus will still not work
properly: it will try to execute on the build machine executables
built for the target. While qemu is here to emulate the instruction
set, there is still the unsolved problem of kernel headers version
mismatch between the target and the build environments.

So the whole approach that consists in using host-qemu for building
simply cannot work properly, and until it is solved, the package
should be marked as broken.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/cpanminus/Config.in |    4 ++++
 1 file changed, 4 insertions(+)
Arnout Vandecappelle - Nov. 16, 2012, 2:07 p.m.
On 16/11/12 15:01, Thomas Petazzoni wrote:
> cpanminus is currently responsible for about half of the autobuilder
> failures, due to the missing dependency on host-qemu. However, even
> with the host-qemu proposed by Arnout, cpanminus will still not work
> properly: it will try to execute on the build machine executables
> built for the target. While qemu is here to emulate the instruction
> set, there is still the unsolved problem of kernel headers version
> mismatch between the target and the build environments.
>
> So the whole approach that consists in using host-qemu for building
> simply cannot work properly, and until it is solved, the package
> should be marked as broken.
>
> Signed-off-by: Thomas Petazzoni<thomas.petazzoni@free-electrons.com>

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

> ---
>   package/cpanminus/Config.in |    4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/package/cpanminus/Config.in b/package/cpanminus/Config.in
> index b8b39dc..b10d211 100644
> --- a/package/cpanminus/Config.in
> +++ b/package/cpanminus/Config.in
> @@ -1,5 +1,9 @@
>   config BR2_PACKAGE_CPANMINUS
>   	bool "cpanminus"
> +	# Requires host-qemu, which cannot work properly due to kernel
> +	# headers mismatch between the build environment and the
> +	# target.
> +	depends on BR2_BROKEN
>   	help
>   	  cpanminus is a script to get, unpack, build and install Perl modules
>   	  from CPAN.
Peter Korsgaard - Nov. 16, 2012, 9:25 p.m.
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> cpanminus is currently responsible for about half of the autobuilder
 Thomas> failures, due to the missing dependency on host-qemu. However, even
 Thomas> with the host-qemu proposed by Arnout, cpanminus will still not work
 Thomas> properly: it will try to execute on the build machine executables
 Thomas> built for the target. While qemu is here to emulate the instruction
 Thomas> set, there is still the unsolved problem of kernel headers version
 Thomas> mismatch between the target and the build environments.

 Thomas> So the whole approach that consists in using host-qemu for building
 Thomas> simply cannot work properly, and until it is solved, the package
 Thomas> should be marked as broken.

Committed, thanks.

Patch

diff --git a/package/cpanminus/Config.in b/package/cpanminus/Config.in
index b8b39dc..b10d211 100644
--- a/package/cpanminus/Config.in
+++ b/package/cpanminus/Config.in
@@ -1,5 +1,9 @@ 
 config BR2_PACKAGE_CPANMINUS
 	bool "cpanminus"
+	# Requires host-qemu, which cannot work properly due to kernel
+	# headers mismatch between the build environment and the
+	# target.
+	depends on BR2_BROKEN
 	help
 	  cpanminus is a script to get, unpack, build and install Perl modules
 	  from CPAN.