# |
Jul 5th 2017, 19:51 |
savant |
probably some asshat out there if there are RCs has already PRd it |
# |
Jul 5th 2017, 19:49 |
cleptric |
To lazy to compile it by myself |
# |
Jul 5th 2017, 19:49 |
cleptric |
@mfc or better @savant Does hombrew added php72-intl That was my only show stopper a few days ago |
# |
Jul 5th 2017, 19:48 |
jeremyharris |
:P |
# |
Jul 5th 2017, 19:48 |
jeremyharris |
NOT IT |
# |
Jul 5th 2017, 19:48 |
admad |
wonderful phpunit issue again |
# |
Jul 5th 2017, 19:47 |
WyriHaximus |
There you go: https://travis-ci.org/cakephp/cakephp/jobs/250489011#L654-L656 |
# |
Jul 5th 2017, 19:32 |
WyriHaximus |
Was about to do the same :) |
# |
Jul 5th 2017, 19:29 |
admad |
I took the liberty of adding a commit on top :slightly_smiling_face: |
# |
Jul 5th 2017, 19:29 |
WyriHaximus |
Looking at it |
# |
Jul 5th 2017, 19:24 |
admad |
*travis config.. |
# |
Jul 5th 2017, 19:24 |
admad |
WyriHaximus: the nightly builds aren't running. The travis needs to be tweaked further https://travis-ci.org/cakephp/cakephp/jobs/250473818#L596 |
# |
Jul 5th 2017, 19:22 |
mcf |
@admad no problem. I'm trying to fix it as well, but I'm running into some interface weirdness... |
# |
Jul 5th 2017, 19:14 |
admad |
@mcf Thanks for the submitting the issue |
# |
Jul 5th 2017, 19:04 |
mcf |
@savant https://github.com/cakephp/cakephp/issues/10862 |
# |
Jul 5th 2017, 18:56 |
savant |
@mcf let me know when you’ve filed that issue |
# |
Jul 5th 2017, 18:56 |
savant |
cool |
# |
Jul 5th 2017, 18:52 |
WyriHaximus |
s/on/enabled |
# |
Jul 5th 2017, 18:52 |
WyriHaximus |
Actually both already have fast finish on :) |
# |
Jul 5th 2017, 18:52 |
WyriHaximus |
on chronos I assume? Sure in the same PR? |
# |
Jul 5th 2017, 18:51 |
jarard01 |
grr of course |
# |
Jul 5th 2017, 18:51 |
jeremyharris |
->where([‘PropertyCategories.id IN’ => [1,2,3,4]] |
# |
Jul 5th 2017, 18:51 |
jarard01 |
is this not valid: ->where(['PropertyCategories.id' => [1,2,3,4]] |
# |
Jul 5th 2017, 18:49 |
savant |
so it doesnt wait on the allow_failure to report success |
# |
Jul 5th 2017, 18:49 |
savant |
@wyrihaximus can you add fast finishes? I think thats an option in travis |
# |
Jul 5th 2017, 18:49 |
savant |
:rubberduck: |
# |
Jul 5th 2017, 18:48 |
WyriHaximus |
done |
# |
Jul 5th 2017, 18:47 |
savant |
please do |
# |
Jul 5th 2017, 18:46 |
mcf |
I can log an issue since the fix is not trivial. |
# |
Jul 5th 2017, 18:46 |
mcf |
PHP 7.2 has changed the behavior of count() BTW. That is causing problems with the QueryCompiler. |
# |
Jul 5th 2017, 18:45 |
mcf |
@savant - NP, it seems to hit windows servers more dramatically. |
# |
Jul 5th 2017, 18:45 |
savant |
even those running on mod_php… |
# |
Jul 5th 2017, 18:45 |
savant |
related to that, no one on the core team could replicate the issue |
# |
Jul 5th 2017, 18:44 |
savant |
sincerely appreciated, we def dropped the ball on filing the issue upstream |
# |
Jul 5th 2017, 18:44 |
savant |
@mcf also I’d like to thank you for looking into the bug and following up |
# |
Jul 5th 2017, 18:44 |
WyriHaximus |
sure |
# |
Jul 5th 2017, 18:44 |
savant |
and chronos |
# |
Jul 5th 2017, 18:44 |
savant |
yes |
# |
Jul 5th 2017, 18:44 |
WyriHaximus |
main cakephp/cakephp repo? |
# |
Jul 5th 2017, 18:44 |
WyriHaximus |
Sure give me a moment |
# |
Jul 5th 2017, 18:44 |
savant |
`allow_failures` is the travis.ci key |