[Buildroot] [PATCH] trinity: Disable for Microblaze architecture

[Buildroot] [PATCH] trinity: Disable for Microblaze architecture

[Vicente Olivert Riera]

This package doesn't support the Microblaze architecture. The full list
of supported architectures is in the trinity website, and Microblaze is
not among them.

Fixes:
	http://autobuild.buildroot.net/results/ea7/ea788f1f0ed5ee4e9b1bf0c8a0d1304e7467dc77/

Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
---
 package/trinity/Config.in |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/package/trinity/Config.in b/package/trinity/Config.in
index de9c467..9882573 100644
--- a/package/trinity/Config.in
+++ b/package/trinity/Config.in
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_TRINITY
 	bool "trinity"
+	depends on !BR2_microblazeel && !BR2_microblazeeb
 	help
 	  A Linux System call fuzz tester
 
-- 
1.7.1

[Buildroot] [PATCH] trinity: Disable for Microblaze architecture

[Thomas Petazzoni]

Dear Vicente Olivert Riera,

On Mon, 27 Oct 2014 16:11:10 +0000, Vicente Olivert Riera wrote:
> This package doesn't support the Microblaze architecture. The full list
> of supported architectures is in the trinity website, and Microblaze is
> not among them.

Then what we want is not something like the patch you propose, but
instead a patch using positive logic to enable Trinity only on the
architecture that are known to work.

The website says: "Trinity supports Alpha, Aarch64, ARM, i386, IA-64,
MIPS, PowerPC-64, S390, S390x, SPARC-64, x86-64.". So it means that not
only Microblaze is going to cause issue, but also Blackfin, ARC,
Xtensa, SPARC 32 bits, and probably some others I forgot.

Can you respin the patch with something like:

	depends BR2_... || BR2_... || BR2_...

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] trinity: Disable for Microblaze architecture

[Vicente Olivert Riera]

Dear Thomas Petazzoni,

On 10/27/2014 04:43 PM, Thomas Petazzoni wrote:
> Dear Vicente Olivert Riera,
>
> On Mon, 27 Oct 2014 16:11:10 +0000, Vicente Olivert Riera wrote:
>> This package doesn't support the Microblaze architecture. The full list
>> of supported architectures is in the trinity website, and Microblaze is
>> not among them.
>
> Then what we want is not something like the patch you propose, but
> instead a patch using positive logic to enable Trinity only on the
> architecture that are known to work.
>
> The website says: "Trinity supports Alpha, Aarch64, ARM, i386, IA-64,
> MIPS, PowerPC-64, S390, S390x, SPARC-64, x86-64.". So it means that not
> only Microblaze is going to cause issue, but also Blackfin, ARC,
> Xtensa, SPARC 32 bits, and probably some others I forgot.
>
> Can you respin the patch with something like:
>
> 	depends BR2_... || BR2_... || BR2_...
>
> Thanks!
>
> Thomas

Done: http://patchwork.ozlabs.org/patch/403703/

I have marked the former patch as superseded.

Best regards,
-- 
Vicente Olivert Riera
Graduate Software Engineer, MIPS Processor IP
Imagination Technologies Limited
t: +44 (0)113 2429814
www.imgtec.com

[Buildroot] [PATCH] trinity: Disable for Microblaze architecture

[Thomas Petazzoni]

Dear Vicente Olivert Riera,

On Mon, 27 Oct 2014 17:19:36 +0000, Vicente Olivert Riera wrote:

> Done: http://patchwork.ozlabs.org/patch/403703/

This is exactly the same patch as before. What happened ? :-)

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] trinity: Disable for Microblaze architecture

[Vicente Olivert Riera]

Dear Thomas Petazzoni,

On 10/27/2014 06:30 PM, Thomas Petazzoni wrote:
> Dear Vicente Olivert Riera,
>
> On Mon, 27 Oct 2014 17:19:36 +0000, Vicente Olivert Riera wrote:
>
>> Done: http://patchwork.ozlabs.org/patch/403703/
>
> This is exactly the same patch as before. What happened ? :-)

git add... :-(

I have sent the right patch now.

Thanks!
-- 
Vicente Olivert Riera
Graduate Software Engineer, MIPS Processor IP
Imagination Technologies Limited
t: +44 (0)113 2429814
www.imgtec.com