videojs / videojs-contrib-hls

HLS library for video.js
http://videojs.github.io/videojs-contrib-hls/
Other
2.84k stars 791 forks source link

fix travis config to _actually_ use containerzed infra #1437

Closed squarebracket closed 5 years ago

squarebracket commented 6 years ago

Description

When I was testing how to get the tests running in the dockerized travis infra, I was running in the travis docker image as root. When moving to the actual travis config, it told me I needed to be root, so without thinking I added sudo to the front. I was sure they'd allow package installs, which they do, but only using the apt addon; having sudo in the script would automatically kick you over to the VM-based infra.

Until they broke that yesterday.

Anyway, now we'll actually be using the containerized infra.

Requirements Checklist

squarebracket commented 5 years ago

Travis is switching to all non-containerized infra, so this PR is no longer relevant.