italomandara / CXPatcher

A patcher to upgrade Crossover dependencies and improve compatibility
GNU General Public License v3.0
834 stars 22 forks source link

Feedback: Explain GStreamer and options #138

Open vesper8 opened 9 months ago

vesper8 commented 9 months ago

This is just some general feedback that I think could help make things a bit clearer for first time users.

A little button next to GStreamer that explains why you need this, why you should install it (why I need foundations libraries)

And same thing next to "use separate bottles path". I can of get it from reading the issues about it but it's still unclear. Like.. will my existing bottles benefit from CXPatcher now that I've chosen this option?

Thanks!

italomandara commented 9 months ago

This is just some general feedback that I think could help make things a bit clearer for first time users.

A little button next to GStreamer that explains why you need this, why you should install it (why I need foundations libraries)

And same thing next to "use separate bottles path". I can of get it from reading the issues about it but it's still unclear. Like.. will my existing bottles benefit from CXPatcher now that I've chosen this option?

Thanks!

Thanks for the feedback I'll check if I can improve that in the next release

MysticalOS commented 9 months ago

Is GStreamer even still needed with crossover 23.5? my understanding is they now include it themselves.

italomandara commented 9 months ago

Is GStreamer even still needed with crossover 23.5? my understanding is they now include it themselves.

re7 doesn't work, so it's missing something, the gstreamer patch for cxpatcher is better

Gcenx commented 9 months ago

A little button next to GStreamer that explains why you need this, why you should install it (why I need foundations libraries)

As GStreamer is a hard requirement it can’t be optional, as for an explanation “GStreamer is used for multimedia decoding”

And same thing next to "use separate bottles path". I can of get it from reading the issues about it but it's still unclear. Like.. will my existing bottles benefit from CXPatcher now that I've chosen this option?

Could probably shorten this to

use separate bottles

After applying CXPatcher it implements things that stock CrossOver doesn’t and this can cause bugs in the users bottles. Having completely separate bottles avoids these issues.

The original intention was to allow a user to have stock CrossOver-23.0.1 plus CrossOver-22.1.1 that had CXPatcher applied.

vesper8 commented 8 months ago

@italomandara along the same lines, because I don't want to open a new issue for a very similar thing, it would be nice if you had some little info buttons for the new features too such as disabling fast math and enabling msync.. I have no idea what those are and why they shouldn't be enabled at all times..

italomandara commented 8 months ago

@italomandara along the same lines, because I don't want to open a new issue for a very similar thing, it would be nice if you had some little info buttons for the new features too such as disabling fast math and enabling msync.. I have no idea what those are and why they shouldn't be enabled at all times..

After the gstreamer integration release I'll have a more time to do this kid of things.