On Sun, Nov 30, 2008 at 06:18:36PM +0100, Farkas Levente wrote:
Richard W.M. Jones wrote:
> On Sun, Nov 30, 2008 at 12:27:15AM +0100, Farkas Levente wrote:
>> hi,
>> did you review my past filesystem patch?
>
> OK, I'll have a look at it tomorrow^Wtoday .. Can you repost what you
> think is the latest version so we are on the same page?
here it's again.
> Thanks for taking a look at those other reviews.
the other patch in the libxml2 what we already discuss in the review:
--- mingw32-libxml2.spec.lfarkas 2008-11-10 18:03:16.000000000 +0100
+++ mingw32-libxml2.spec 2008-11-10 18:35:10.000000000 +0100
@@ -40,6 +40,10 @@
%build
+%if 0%{?fedora} < 9
+aclocal
+autoreconf
+%endif
Am I right in thinking that this code only applies for Fedora <= 8 (or
does that include EPEL?) Anyway, yes the patch is fine, because it
won't affect us, but it will break things in mock/F-8 because you also
need to add a BuildRequires: autoconf, automake, libtool (or something
like that).
Index: mingw32-filesystem.spec
===================================================================
RCS file: /cvs/pkgs/rpms/mingw32-filesystem/devel/mingw32-filesystem.spec,v
retrieving revision 1.6
diff -u -u -r1.6 mingw32-filesystem.spec
--- mingw32-filesystem.spec 24 Nov 2008 16:08:07 -0000 1.6
+++ mingw32-filesystem.spec 27 Nov 2008 20:54:54 -0000
I have some time to look at this tomorrow. If it works I'll just
apply it.
Rich.