Cemu icon indicating copy to clipboard operation
Cemu copied to clipboard

Refactor CMake files

Open Zopolis4 opened this issue 3 years ago • 5 comments

Just a work in progress, putting it here so nobody doubles up by accident.

Zopolis4 avatar Aug 24 '22 11:08 Zopolis4

There is no way this should be building-- look at what I have done to gui/CMakeLists.txt. The CI is broken.

Zopolis4 avatar Aug 25 '22 23:08 Zopolis4

There is no way this should be building-- look at what I have done to gui/CMakeLists.txt. The CI is broken.

Looked a bit into it and it's complicated, so no quick fix. Seems to only affect commits done after initial pull request. As long as you can confirm it compiles locally I'd merge the pull request. Just let me know when you are done

Exzap avatar Aug 26 '22 02:08 Exzap

Unfortunately I cannot build locally due to CMake not finding the vcpkg installed dependencies. I'll try a CI PR in a bit.

Zopolis4 avatar Aug 26 '22 04:08 Zopolis4

#12 and #10 have now been merged. These 2 files need to be installed on Unix Systems in the following locations: info.cemu.Cemu.desktop into PREFIX/share/applications info.cemu.Cemu.metainfo.xml into PREFIX/share/metainfo Could you add that into the install step as discussed #20?

JakobDev avatar Aug 26 '22 07:08 JakobDev

Re-posting here, as it got lost between all the other comments.

From https://github.com/cemu-project/Cemu/issues/1#issuecomment-1227704188

@Zopolis4 I just noticed your CMake PR. I've been working on improving the current state of CMake too, as I'm looking into packaging Cemu for Debian and Ubuntu (as in creating a distro-provided package, not what's currently available here).

Are you working on packaging too?

Tachi107 avatar Aug 26 '22 09:08 Tachi107

The fmt stuff is a bit ugly, and will not work when not using vcpkg unless you decide to install version 7.0.2 manually, but that's about as easy as manually installing 7.0.x, which you needed to do anyways. It only needs to exist until we update to 9.x, anyways.

Zopolis4 avatar Aug 30 '22 01:08 Zopolis4

The fmt stuff is a bit ugly, and will not work when not using vcpkg unless you decide to install version 7.0.2 manually, but that's about as easy as manually installing 7.0.x, which you needed to do anyways. It only needs to exist until we update to 9.x, anyways.

Why 7.0.2? Every release on the 7.x branch should work just fine (it works with 7.1.3, for example)

Tachi107 avatar Aug 30 '22 09:08 Tachi107

Anyway, the diff is huge, could you split the changes in different commits, where each one contains a single logical change, so that this is easier to review?

Tachi107 avatar Aug 30 '22 10:08 Tachi107

Most of the diff is just from listing the files, the amount of intellectual changes is pretty low.

Zopolis4 avatar Aug 30 '22 11:08 Zopolis4

Closing in favour of a series of smaller PRs.

Zopolis4 avatar Sep 07 '22 00:09 Zopolis4