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 IBM classloader regression #1301

Merged
merged 1 commit into from May 5, 2016
Merged

Fix IBM classloader regression #1301

merged 1 commit into from May 5, 2016

Conversation

@Sp0i3eR
Copy link
Contributor

@Sp0i3eR Sp0i3eR commented Apr 28, 2016

Migration during #979 fix introduced regression on #175
Migration to Location from String in ClassPathResolver on cbc9fef introduced regression in websphere hack from #175 in ClassPathScanner. Location there resolves to "classpath:db/migration/flyway.location" which renders resource not visible for IBM classloader. Added getPath() like in normal classloader flow. Tested on WebSphere 8.5.5.9 jdk8.

Migration to Location from String in ClassPathResolver on cbc9fef introduced regression in websphere classloader, location there resolves to "classpath:db/migration/flyway.location" which causes IBM classloader to miss resources, added getPath() like in normal classloader flow, tested on WebSphere 8.5.5.9 jdk8.
@axelfontaine axelfontaine merged commit d37410c into flyway:master May 5, 2016
@axelfontaine axelfontaine added this to the Flyway 4.0.1 milestone May 5, 2016
axelfontaine added a commit to flyway/flywaydb.org that referenced this pull request May 5, 2016
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented May 5, 2016

Thanks! Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants