3:51 a.m.
Signed-off-by: Kedar Sovani <kedars(a)marvell.com>
---
libprelude.spec | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/libprelude.spec b/libprelude.spec
index c769ab7..36995c3 100644
--- a/libprelude.spec
+++ b/libprelude.spec
@@ -5,7 +5,7 @@
Name: libprelude
Version: 0.9.21.2
-Release: 1%{?dist}
+Release: 1%{?dist}.fa1
Summary: The prelude library
Group: System Environment/Libraries
License: GPLv2+
@@ -15,7 +15,7 @@ BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u}
-n)
BuildRequires: gnutls-devel, python-devel, ruby, ruby-devel, lua-devel
BuildRequires: swig chrpath
-%ifnarch ppc ppc64
+%ifnarch ppc ppc64 %{arm}
BuildRequires: valgrind
%endif
@@ -111,7 +111,7 @@ chmod -w %{buildroot}%{perl_vendorarch}/auto/Prelude/Prelude.so
touch -r ./configure.in %{buildroot}%{_sysconfdir}/prelude/default/*
# PPC strips ld.so so valgrind can't run
-%ifnarch ppc ppc64
+%ifnarch ppc ppc64 %{arm}
%check
make check
%endif
--
1.5.3.4
3:58 a.m.
On Mon, 2008-12-29 at 10:58 +0100, Lennert Buytenhek wrote:
On Mon, Dec 29, 2008 at 03:21:42PM +0530, Kedar Sovani wrote:
> diff --git a/libprelude.spec b/libprelude.spec
> index c769ab7..36995c3 100644
> --- a/libprelude.spec
> +++ b/libprelude.spec
> @@ -5,7 +5,7 @@
>
> Name: libprelude
> Version: 0.9.21.2
> -Release: 1%{?dist}
> +Release: 1%{?dist}.fa1
This is fine for local use but shouldn't be submitted upstream.
> @@ -15,7 +15,7 @@
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
>
> BuildRequires: gnutls-devel, python-devel, ruby, ruby-devel, lua-devel
> BuildRequires: swig chrpath
> -%ifnarch ppc ppc64
> +%ifnarch ppc ppc64 %{arm}
> BuildRequires: valgrind
> %endif
This I understand..
> @@ -111,7 +111,7 @@ chmod -w %{buildroot}%{perl_vendorarch}/auto/Prelude/Prelude.so
> touch -r ./configure.in %{buildroot}%{_sysconfdir}/prelude/default/*
>
> # PPC strips ld.so so valgrind can't run
> -%ifnarch ppc ppc64
> +%ifnarch ppc ppc64 %{arm}
> %check
> make check
> %endif
But what is that for? At least the comment explains why PPC can't run
the testsuite, but there's no such comment for ARM. Is it the same
reason? Or a different reason?
This is because the valgrind rpm is not yet built on ARM. I'll update
the comment in there.
Kedar.
3:58 a.m.
On Mon, Dec 29, 2008 at 03:21:42PM +0530, Kedar Sovani wrote:
diff --git a/libprelude.spec b/libprelude.spec
index c769ab7..36995c3 100644
--- a/libprelude.spec
+++ b/libprelude.spec
@@ -5,7 +5,7 @@
Name: libprelude
Version: 0.9.21.2
-Release: 1%{?dist}
+Release: 1%{?dist}.fa1
This is fine for local use but shouldn't be submitted upstream.
@@ -15,7 +15,7 @@
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildRequires: gnutls-devel, python-devel, ruby, ruby-devel, lua-devel
BuildRequires: swig chrpath
-%ifnarch ppc ppc64
+%ifnarch ppc ppc64 %{arm}
BuildRequires: valgrind
%endif
This I understand..
@@ -111,7 +111,7 @@ chmod -w
%{buildroot}%{perl_vendorarch}/auto/Prelude/Prelude.so
touch -r ./configure.in %{buildroot}%{_sysconfdir}/prelude/default/*
# PPC strips ld.so so valgrind can't run
-%ifnarch ppc ppc64
+%ifnarch ppc ppc64 %{arm}
%check
make check
%endif
But what is that for? At least the comment explains why PPC can't run
the testsuite, but there's no such comment for ARM. Is it the same
reason? Or a different reason?
5590
days inactive
5590
days old
2 comments
2 participants
participants (2)
-
Kedar Sovani -
Lennert Buytenhek