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

Plugin doesn't bundle runtime dependencies into app/lib/ #15

Closed
sleonidy opened this issue Mar 14, 2016 · 10 comments
Closed

Plugin doesn't bundle runtime dependencies into app/lib/ #15

sleonidy opened this issue Mar 14, 2016 · 10 comments
Assignees
Labels

Comments

@sleonidy
Copy link

I have created sample project for demonstrating the issue
javafxplugin.zip

@purringpigeon
Copy link

If you have this set:

    bundleArguments = [
        // dont bundle JRE (not recommended, but increases build-size/-speed)
        runtime: null
    ]

It will not bundle the runtime - I have that removed from my build and it works fine.

@FibreFoX
Copy link
Owner

@sleonidy Will have to take a look :)

@sleonidy
Copy link
Author

I didn't set this option(Cause I want jre to be bundled), what I mean is:
If you add dependency to gradle to runtime scope , it won't bundle it.
Take a look at a project I've attached.
Thanks

@FibreFoX
Copy link
Owner

@sleonidy I'm already seeing your issue, will have to take a look how to fix this and why it's there

@purringpigeon the issue is about runtime-dependency, not the runtime itself 😄

@FibreFoX
Copy link
Owner

@sleonidy Might have found the confusion here:
you are setting your dependency into the scope "runtime", but this plugin works with scope "compile", so thats the reason why no lib is copied.

Will have to take a look into the maven-plugin if it is possible to add something for runtime-dependencies too.

Quick workaround:

dependencies {
    compile 'org.slf4j:slf4j-api:1.7.16'
    testCompile group: 'junit', name: 'junit', version: '4.11'
}

@FibreFoX
Copy link
Owner

@sleonidy Hi, I've checked the javafx-maven-plugin for this behaviour, seems that this is a real bug ;) thanks for finding this. I will create a quick fix and will report back when having something for you.

@FibreFoX FibreFoX reopened this Mar 14, 2016
@FibreFoX
Copy link
Owner

@sleonidy could you try with version 8.4.1-SNAPSHOT using Sonatype-Snapshot-Reposiroty?

buildscript {
    repositories {
        mavenLocal()
        mavenCentral()
        maven { url "https://oss.sonatype.org/content/repositories/snapshots" }
    }
    dependencies {
        classpath group: 'de.dynamicfiles.projects.gradle.plugins', name: 'javafx-gradle-plugin', version: '8.4.1-SNAPSHOT'
    }
}

@sleonidy
Copy link
Author

It is working .
Thanks for quick fix.

@FibreFoX
Copy link
Owner

@sleonidy just to make sure, you used the new version instead of switching "runtime" to "compile", right?

The new 8.4.1 should be available through maven-central in the next few days.

@sleonidy
Copy link
Author

Yes. I've checked that "runtime" dependencies are copied to app/lib folder .

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

No branches or pull requests

3 participants