@ron Is there a stack trace on the details page? I can't figure out where this is coming from.

The only class that is dealing with Code Styles is the

FXPreloader

class, which is configuring star imports for the

tornadofx

package.

This is all the code in that class:

@User uploaded a file: Untitled

I did not check for a stack trace 😢

@ron Next time then 🙂 I'm trying to force it here, but can't seem to trip it myself. You're running 2017.1?

@gtnarg I gave it a shot. I believe that's most or even all of it 🙂

Walking the dog, thinking about what I should work on next 🙂

@edvin do you want to keep the cachedir default to the current directory?

@ron Do you have a suggestion for something saner?

USERLIB/${appName}

Maybe

USERLIB/${appName}/cache

is more future proof?

sure

Hm.. actually, when you run the app through a native installer, current directory is probably best.

we can do both

When you're not running natively however, that would be a better default, but I don't think we want to create both USERHOME/AppData/Local/App and USERLIB/App

when making a native installer we can adjust the cache setting

That's right.

if the developer has not entered a cache in the configuration

I'm looking at the multitude of spinner builders again. I really hate this part of the API, but JavaFX doesn't make it easy to create a confirm API for this.

(I would do that, as I am thinking of the stable and unstable releases)

Oh I thought of a way to support that without doing code in fxlauncher for it

just make profiles in Maven

especially once this plugin is gone you could just put it in a “stable” profile and a “unstable” profile

Good idea!

Would be good to showcase this as well, to give people some ideas.

I think I found a good compromise for the spinners: Provide default values, and also let the initialValue be extracted from the property if there was a property supplied.

and I think you can do something like this with gradle too

Yes, and we should update the Gradle plugin to use the CreateManifest class internally as well.

this is how I call it from the plugin