-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use the Travis' Trusty build environment #6515
Conversation
To see if that solves the XML validation issues (since the issues are possibly happening due to old libxml versions).
125a518
to
af1b66a
Compare
.travis.yml
Outdated
@@ -18,6 +18,7 @@ before_script: | |||
- composer install --prefer-source | |||
- if [ "$DEPENDENCIES" != "low" ]; then composer update; fi; | |||
- if [ "$DEPENDENCIES" == "low" ]; then composer update --prefer-lowest; fi; | |||
- if [[ $DB == "mysql" || $DB == "mariadb" ]]; then SCHEMA=`xmllint --xpath 'string(//var[@name="db_name"]/@value)' tests/travis/$DB.travis.xml` && mysql -e "CREATE SCHEMA $SCHEMA; GRANT ALL PRIVILEGES ON $SCHEMA.* to travis@'%'"; fi; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Ocramius xmllint
is not installed, I was trying to avoid hardcoded stuff but I'm not sure that we should install libxml2-utils
just because of that...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Surely not just for this line. We can actually hardcode it in .travis.yml
though...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alright will do it 😉
af1b66a
to
21ad7a1
Compare
@Ocramius done |
🚢 |
To see if that solves the XML validation issues (since the issues are
possibly happening due to old libxml versions).