[Bug 977208] Review Request: Phalcon - A web framework implemented as a C extension
bugzilla at redhat.com
bugzilla at redhat.com
Sun Jul 21 08:19:47 UTC 2013
https://bugzilla.redhat.com/show_bug.cgi?id=977208
--- Comment #24 from Renich Bon Ciric <renich at woralelandia.com> ---
(In reply to Roman Mohr from comment #23)
> Created attachment 776248 [details]
> better description and updated spec file for running unittests
>
> (In reply to Renich Bon Ciric from comment #21)
> > (In reply to Roman Mohr from comment #20)
> > > Created attachment 773914 [details]
> > > Patch to run most of the non-db-dependent unit tests
> > >
> > > Apply in combination with attachment 773907 [details]
> >
> > Thank you, Roman, for your hard work and collaboration.
> >
>
> You are welcome :)
>
> > Here's what I got:
> >
> > Patch #0 (php-phalcon-phpunit.patch):
> > + /usr/bin/cat /builddir/build/SOURCES/php-phalcon-phpunit.patch
> > + /usr/bin/patch -p0 --fuzz=0
> > can't find file to patch at input line 3
> > Perhaps you used the wrong -p or --strip option?
> > The text leading up to this was:
> > --------------------------
> > |--- unit-tests/phpunit.xml.orig 2013-07-15 22:43:31.311938857 +0200
> > |+++ unit-tests/phpunit.xml.orig 2013-07-15 22:44:08.794605062 +0200
> > --------------------------
> Strange, where does that header com from? The header of the uploaded patch
> looks like this:
>
> --- unit-tests/phpunit.xml.orig 2013-07-15 22:43:31.311938857 +0200
> +++ unit-tests/phpunit.xml 2013-07-15 22:44:08.794605062 +0200
>
> It should be enough that the correct file name is in the '+++' line.
>
> (In reply to Renich Bon Ciric from comment #22)
> > Ok, removed .orig from the header of the file and I got this now:
> >
> > Patch #0 (php-phalcon-phpunit.patch):
> > + /usr/bin/cat /builddir/build/SOURCES/php-phalcon-phpunit.patch
> > + /usr/bin/patch -p0 --fuzz=0
> > patching file unit-tests/phpunit.xml
> > Hunk #1 FAILED at 14.
> > Hunk #2 FAILED at 31.
> > Hunk #3 FAILED at 63.
>
> The official github tag is not commit b72745 used by us. I am quite sure
> that you are using an old archive. Please place the correct source tar.gz in
> your SOURCES folder.
Well, actually, we're using a tag now: v1.2.0. Also, I am using the spectool
you taught me.
I re-did the patch and I had to comment out several more Db tests; as well as
some Collections and stuff.
Anyway, the patch is remade in the SRPM and it now builds correctly.
> I have uploaded a new spec patch with
> - improved description
> - removed CFLAGS
> - improved (architecture independent) builddir detection
> - removed architecture limitation
>
> There is just one point left in my eyes: I see no reason why phalcon should
> not run on ppc and arm. If you know of any reason then readd the exclusive
> arch directive and add a comment why it won't run.
I've emailed the author regarding these issues. I, also, asked his help in
reviewing the review; so he can clarify any pending points (like the CFLAGS)
and other matters.
Thank you once again. I appreciate your great effort and patience.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=7j1zVwHPh4&a=cc_unsubscribe
More information about the package-review
mailing list