Skip to content
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

Fix the build #173

Closed
wants to merge 4 commits into from
Closed

Fix the build #173

wants to merge 4 commits into from

Conversation

p8
Copy link
Contributor

@p8 p8 commented Jul 9, 2019

The current build fails because of updates in Travis.

  1. A newer JDK should be used.
    Changes were copied from Bump JRuby and the jdk on Travis delayed_job#1092
  2. Travis switched to using Xenial as the default which requires the database services be defined.
    "Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock'" travis-ci/travis-ci#6842 (comment)
  3. Xenial uses mysql 5.7 which requires a fix for a primary key issue for Rails <= 4.0.
  4. Some Rubocop cops have changed.

@p8 p8 changed the title Bump jdk and jRuby versions Fix the build Jul 9, 2019
p8 added 2 commits July 9, 2019 19:34
Travis switched to using Xenial as the default, and the DB is not
enabled by default.
travis-ci/travis-ci#6842 (comment)
Disable Style/FrozenStringLiteralComment cop for now and remove
disabling of unknown cop.
@p8
Copy link
Contributor Author

p8 commented Jul 9, 2019

Most of the failures have been fixed. There still is a problem with an occasional jruby build failing due to github api rate limiting? See: https://travis-ci.community/t/rvm-tries-to-download-blank-version-of-itself/3972/7

MySQL 5.7 no longer supports null default values for the primary key.
Override the Native default for primary keys in MySql.
https://stackoverflow.com/a/34555109
@p8
Copy link
Contributor Author

p8 commented Jul 17, 2019

Travis is building jruby correctly again.

@p8 p8 mentioned this pull request Jul 17, 2019
@albus522
Copy link
Member

I noticed this PR too late, but I did cherry pick your work on mysql 5.7 and that is now in master

@albus522 albus522 closed this Aug 20, 2019
@p8 p8 deleted the bump-jdk-version branch September 6, 2019 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants