isle-portable icon indicating copy to clipboard operation
isle-portable copied to clipboard

Config: explain the different options

Open gwuen opened this issue 6 months ago • 3 comments

First of all, great project! I'm really amazed how simple it was to compile this under Linux and have LEGO Island up and running natively in under ten minutes. Thank you all for that :)

The only hurdle I encountered was the various configuration options, which unfortunately are not explained any further. By looking at the error messages I was quickly able to figure out the correct diskpath and cdpath, but I'm still unsure what mediapath is meant to point to (using the ISO from the LEGO Island Wiki). I have really looked around everywhere (not only in this GH project) but couldn't find any explanation, even though a mediaPath doesn't seem to be exclusive to this portable version.

So for one, I would really appreciate it if someone could explain to me what the media path is supposed to be. For another, I think it would be good to have short explanations or examples in the config GUI and/or the README for new users.

gwuen avatar Jun 22 '25 16:06 gwuen

mediapath has no function whatsoever. It's a leftover from the original decompilation. We should probably remove it from config and maybe from the INI file altogether.

foxtacles avatar Jun 22 '25 16:06 foxtacles

Sounds like a good idea, thanks for the quick response!

What do you think of adding tooltips to the Disk Path and CD Path options? I could also (later) open a PR for that if you're not already on it. Other options like the Direct 3D Devices (that's the different backends for the d3drm translation layer, right?) could also use some kind of explanation, but I guess it's not that important as the default works ootb.

gwuen avatar Jun 22 '25 16:06 gwuen

Tooltips would be great - I don't think anybody is working on it right now. There's also other config related issues: https://github.com/isledecomp/isle-portable/issues/339

(that's the different backends for the d3drm translation layer, right?)

Yes

foxtacles avatar Jun 22 '25 16:06 foxtacles

Now that #427 has been merged, I'd say this issue is complete and can be marked as resolved.

VoxelTek avatar Jun 26 '25 03:06 VoxelTek