On Tue, 2010-11-16 at 11:18 -0500, Eric "Sparks" Christensen wrote:
> On Tue, Nov 16, 2010 at 11:04, Patrick MONNERAT <pm@xxxxxxxxxxxxx> wrote:
> > On Tue, 2010-11-16 at 17:48 +0200, Andy Shevchenko wrote:
> >> You need to do patch on top of source tree container
> >>
> >> mycoolpkg-5.3/
> >> /Makefile
> >> /source.c
> >> ...
> >> mycoolpkg-5.3.new/
> >> /Makefile
> >> /source.c
> >> ...
> >>
> >> run diff -ruN -p mycoolpkg-5.3 mycoolpkg-5.3.new
> >>
> >
> > ... and use %patch0 -p 1 ...
>
> Okay, I ran the above diff and created a nice patch:
>
> diff -ruN -p gpredict-1.2/Makefile gpredict-1.2.new/Makefile
> --- gpredict-1.2/Makefile 2010-11-15 20:07:20.676418835 -0500
> +++ gpredict-1.2.new/Makefile 2010-11-16 11:04:49.677590541 -0500
> @@ -168,7 +168,7 @@ OTOOL64 =
> PACKAGE = gpredict
> PACKAGE_BUGREPORT =
> PACKAGE_CFLAGS = -pthread -I/usr/include/gtk-2.0
> -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo
> -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/pango-1.0
> -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include
> -I/usr/include/pixman-1 -I/usr/include/freetype2
> -I/usr/include/libpng12 -I/usr/include/goocanvas-1.0
> -PACKAGE_LIBS = -pthread -lgoocanvas -lgtk-x11-2.0 -lgdk-x11-2.0
> -latk-1.0 -lgio-2.0 -lpangoft2-1.0 -lpangocairo-1.0 -lgdk_pixbuf-2.0
> -lm -lcairo -lpng12 -lpango-1.0 -lfreetype -lfontconfig -lgobject-2.0
> -lgmodule-2.0 -lgthread-2.0 -lrt -lglib-2.0 -lcurl
> +PACKAGE_LIBS = -lm -pthread -lgoocanvas -lgtk-x11-2.0 -lgdk-x11-2.0
> -latk-1.0 -lgio-2.0 -lpangoft2-1.0 -lpangocairo-1.0 -lgdk_pixbuf-2.0
> -lm -lcairo -lpng12 -lpango-1.0 -lfreetype -lfontconfig -lgobject-2.0
> -lgmodule-2.0 -lgthread-2.0 -lrt -lglib-2.0 -lcurl
> PACKAGE_NAME =
> PACKAGE_STRING =
> PACKAGE_TARNAME =
>
> I changed the SPEC to say "%patch0 -p 1". I get the same error (below):
>
> [user@server rpmbuild]$ rpmbuild -ba SPECS/gpredict.spec
> Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.wAGj7E
> + umask 022
> + cd /home/christensene/rpmbuild/BUILD
> + LANG=C
> + export LANG
> + unset DISPLAY
> + cd /home/christensene/rpmbuild/BUILD
> + rm -rf gpredict-1.2
> + /usr/bin/gzip -dc /home/user/rpmbuild/SOURCES/gpredict-1.2.tar.gz
> + /bin/tar -xf -
> + STATUS=0
> + '[' 0 -ne 0 ']'
> + cd gpredict-1.2
> + /bin/chmod -Rf a+rX,u+w,g-w,o-w .
> + echo 'Patch #0 (gpredict-1.2-DSO.patch):'
> Patch #0 (gpredict-1.2-DSO.patch):
> + /bin/cat /home/user/rpmbuild/SOURCES/gpredict-1.2-DSO.patch
> + /usr/bin/patch -s -p1 --fuzz=0
> The text leading up to this was:
> --------------------------
> |diff -ruN -p gpredict-1.2/Makefile gpredict-1.2.new/Makefile
> |--- gpredict-1.2/Makefile 2010-11-15 20:07:20.676418835 -0500
> |+++ gpredict-1.2.new/Makefile 2010-11-16 11:04:49.677590541 -0500
> --------------------------
That might be because the Makefile actually isn't there -- it's usually
created from Makefile.in after calling ./configure. IMHO you have
several options
* use sed to make the change, after running configure
* patch Makefile.in instead of Makefile
* patch Makefile.am and reconfigure (autoreconf --force,
or ./autogen.sh, depending how autotools are handled in
gpredict)
And anyway, why are you trying to patch it this way? Unless I missed
something you're adding -lm to PACKAGE_LIBS which is however already
present (prior to -lcairo) there.
Martin
Attachment:
signature.asc
Description: This is a digitally signed message part
-- devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/devel
