Feeling Lucky
?
?
?
[#8617] - travis again
- Closed
- 8 Dec 2015
- Medium
- Build: staging
- # 8617
- Diff
- dgt41:patch-1
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
| Status | New | ⇒ | Pending |
| Status | Pending | ⇒ | New |
| Labels |
Added:
?
?
?
|
||
| Status | New | ⇒ | Pending |
dgt41
- comment
- 8 Dec 2015
Check krakjoe/apcu#154
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-12-08 18:05:54 |
| Closed_By | ⇒ | Kubik-Rubik |
mbabker
- comment
- 8 Dec 2015
If I'm following that correctly, then the env vars should be INSTALL_APCU for PHP 5.5/5.6 and INSTALL_APCU_BC for PHP 7 and the command for that install changed to pecl install apcu_bc-beta.
All this pull request did was rename env vars and missed changing the if conditional for the APCu beta install which makes the PHP 7 build bypass installing it giving the appearance of a passing build.
dgt41
- comment
- 8 Dec 2015
Yep my bad, fixing it now
Please add more information to your issue. Without test instructions and/or any description we will close this issue within 4 weeks. Thanks.
This is an automated message from the J!Tracker Application.