https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #14 from Remi Collet fed...@famillecollet.com ---
Typo:
-Requires: php-pear(pear.swiftmailer.org/Swift) %{swift_max_ver}
+Requires: php-pear(pear.swiftmailer.org/Swift) %{swift_max_ver}
--
You are receiving this mail
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #15 from Remi Collet fed...@famillecollet.com ---
Tips for phpunit option (I discover this recently, often simpler)
Instead of
-d include_path=./src:%{_datadir}/php:%{pear_phpdir} \
Use
--include-path=./src \
--
You
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #16 from Remi Collet fed...@famillecollet.com ---
Perhaps you should also Obsoletes php-channel-symfony2 from php-symfony2-common
(as this package have probably no reason to be kept)
--
You are receiving this mail because:
You
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #18 from Remi Collet fed...@famillecollet.com ---
Created attachment 825643
-- https://bugzilla.redhat.com/attachment.cgi?id=825643action=edit
review.txt
Generated by fedora-review 0.5.0 (920221d) last change: 2013-08-30
Command
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
Remi Collet fed...@famillecollet.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #19 from Remi Collet fed...@famillecollet.com ---
Update from installed packages (pear ones) to new packages = ok.
PHPUnit still works (despite pear metadata are now inconsistent)
Packages which requires another php-symfony2-*
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #20 from Remi Collet fed...@famillecollet.com ---
Same comment as for symfony/icu.
What do you think of using php-symfony prefix instead of php-symfony2 ?
--
You are receiving this mail because:
You are on the CC list for the
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #21 from Remi Collet fed...@famillecollet.com ---
I forget, in the description of the main package PHP full-stack web framework
.
I think full can be ambiguous (as the meta-package don't install the full
famework...). Any better
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
Shawn Iwinski shawn.iwin...@gmail.com changed:
What|Removed |Added
Depends On||1031400
---
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #12 from Remi Collet fed...@famillecollet.com ---
Another minor issue using the same .src.rpm for symfony2 and icu.
On symfony2 update, you will have to manage NVER of icu (you cannot use the
same).
So, for example:
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #11 from Remi Collet fed...@famillecollet.com ---
Result is much better :) as test suite is useful
FYI:
https://admin.fedoraproject.org/updates/php-password-compat-1.0.0-5.git58151cf.el6
Just a small problem: sources should not
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #10 from Shawn Iwinski shawn.iwin...@gmail.com ---
(In reply to Remi Collet from comment #8)
I think(In reply to Shawn Iwinski from comment #0)
* Intl requires Icu and Icu requires Intl so I'm not sure I can separate out
the
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #9 from Remi Collet fed...@famillecollet.com ---
Quick notes
ln -s %{name}-common-%{version} %{buildroot}%{_docdir}/%{name}-%{version}
This will not work on Fedora = 20
See
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #8 from Remi Collet fed...@famillecollet.com ---
I think(In reply to Shawn Iwinski from comment #0)
* Intl requires Icu and Icu requires Intl so I'm not sure I can separate out
the Icu pkg
I think is will be preferable to have a
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
Lameire Alexis alexisis-pristont...@hotmail.com changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
Remi Collet rcol...@redhat.com changed:
What|Removed |Added
CC||rcol...@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #5 from Remi Collet fed...@famillecollet.com ---
@shawn: some interesting information in
https://github.com/symfony/symfony/blob/master/.travis.yml
Perhaps a workaround (temp. excluding src/Symfony/Bundle) is
for dir in
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #6 from Shawn Iwinski shawn.iwin...@gmail.com ---
(In reply to Remi Collet from comment #5)
One of the biggest issues with the RPM build for the tests is that we are not
using the Composer-generated autoload files which add more
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
--- Comment #7 from Remi Collet fed...@famillecollet.com ---
Thanks for pointing the exact problem: this is not designed to be packaged.
Sorry, but you know my feeling about composer
(bull-shit-to-create-broken-system), so I will try to not
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
Mosaab Alzoubi moc...@hotmail.com changed:
What|Removed |Added
CC||moc...@hotmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=1021749
Shawn Iwinski shawn.iwin...@gmail.com changed:
What|Removed |Added
CC|
21 matches
Mail list logo