Opened 8 years ago
Closed 8 years ago
#37116 closed defect (bug) (worksforme)
Fix travis builds for previous branches
Reported by: | jorbin | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | Build/Test Tools | Keywords: | |
Focuses: | Cc: |
Description
The 4.4 and 4.5 branches are currently failing to build on travis. this is 4.4 and this is 4.5. The error has something to do with npm install.
Attachments (2)
Change History (12)
#3
@
8 years ago
@jorbin Weird, maybe try flushing the caches on both branches and restarting both builds:
#4
@
8 years ago
- Summary changed from Fix travis builds for previus branches to Fix travis builds for previous branches
#5
@
8 years ago
See also r37730 for /trunk
Output node and npm versions on travis
In order to make local debugging of travis easier, output the npm and node versions when travis runs
#6
@
8 years ago
Patch 37116-all-the-traviss.diff moves the Npm and NodeJS version checks to before_install
, we want to see what versions of NPM and NodeJS (And MySQL and PHP for /trunk) are running before we execute them.
Patch contains the patch for the 4.4, 4.5, and /trunk branches
Edit: This doesn't fix the issue per this ticket, just the start of debugging
#8
@
8 years ago
Both the 4.4 and 4.5 branch builds on Travis CI error with:
(node) warning: Recursive process.nextTick detected. This will break in the next version of node. Please use setImmediate for recursive deferral.
This appears to be a conflict between the NodeJS modules dependancy tree, which as always is crazy insane.
Approximately a dozen NPM modules require NodeJS >= 0.12.x
For example:
- The
grunt-postcss
update from0.7.0
to0.8.0
was reverted in r36980 because NodeJS 0.10.x support was dropped grunt-postcss
has an NPM dependancy of"postcss": "^5.0.0"
- NodeJS
0.10.x
support inpostcss
was dropped in version5.0.6
- This is semantic version bug in
grunt-postcss
and it should not have been released in this manner, though that horse has bolted now and very little can be done in that NPM module to rectify or reinstate NodeJS0.10.x
compatibility, I created an issue to document this for the maintainers so hopefully this can be avoided in the future.
Hence, our Travis CI builds require a minimum of NodeJS 0.12.x
otherwise there be explosions!
Here's some test PR's and all 4 pass
- NodeJS 0.12.x branch on WP 4.4 branch: https://travis-ci.org/aaronjorbin/develop.wordpress/builds/139091078
- NodeJS 0.12.x branch on WP 4.5 branch: https://travis-ci.org/aaronjorbin/develop.wordpress/builds/139091042
- NodeJS 4.x branch on WP 4.4 branch: https://travis-ci.org/aaronjorbin/develop.wordpress/builds/139089182
- NodeJS 4.x branch on WP 4.5 branch: https://travis-ci.org/aaronjorbin/develop.wordpress/builds/139088613
@jorbin The resolution to resolve this ticket is to bump NodeJS to 0.12.x or 4.x, which is what ticket #35105 is all about.
My suggestion included in 37116-nodejs-4x-lts.diff is to bump to 4.x LTS for all the branches (4.4, 4.5, and /trunk). This would save us from having to redo it again in a couple of months for the 4.4/4.5 branches when LTS support ends for NodeJS 0.12.x.
In using NodeJS 4.x LTS support here for all the branches I can't forsee any issues on the Travis CI side of things.
On the https://core.svn.wordpress.org/
side of things having NodeJS 4.x LTS installed would be beneficial in that it is only one single instance of NodeJS required on the "build" server rather than having to maintain multiple versions using NVM.
Slack discussion on bumping NodeJS on the build server: wordpress.slack.com/archives/meta/p1457914147000023
#9
@
8 years ago
A couple more notes:
- 37116-nodejs-4x-lts.diff includes the 4.4, 4.5, and /trunk branches, I've also refreshed the patch for #35105 though only included /trunk in the patch 35105.3.diff.
- Each patch contains NPM & NodeJS version check before and after installing the new NodeJS version, makes things a little easier to debug knowing before and after NPM & NodeJS versions
In 37731: