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

Update the build system #7

Open
lu-zero opened this issue Aug 27, 2022 · 8 comments · May be fixed by #69
Open

Update the build system #7

lu-zero opened this issue Aug 27, 2022 · 8 comments · May be fixed by #69
Labels
enhancement request New feature or request

Comments

@lu-zero
Copy link

lu-zero commented Aug 27, 2022

The current one fails to link plugins due the old libtool failing to link modern libcs.

@AndersonTorres
Copy link

AndersonTorres commented Aug 27, 2022

I wanna try Meson! I will fork and study this.

An indirect benefit of this is that Meson supports Xcode and MSVC.

@therealdreg
Copy link
Member

I wanna try Meson! I will fork and study this.

An indirect benefit of this is that Meson supports Xcode and MSVC.

Awesome! thx @AndersonTorres :D

@thesamesam
Copy link

Meson would be lovely :)

@AndersonTorres
Copy link

I am planning to study the current system first. It has some low level interactions like building bios files from 32-bit assembly...

@winterheart
Copy link

Hello.
Seems, migration to meson build system stalled. Any chances to adopt CMake? I can perform that migration. I can also update current autotools project to recent versions - this relatively easy task and can be done quicklier than migration.

@AndersonTorres
Copy link

AndersonTorres commented Jun 20, 2023

Oh man, I need to take this seriously!
I vote for a new autotools first.
Cmake is too bloated, I am not a big fan of it. Meson is way easier.

bog-dan-ro added a commit to bog-dan-ro/Bochs that referenced this issue Sep 4, 2023
@bog-dan-ro bog-dan-ro linked a pull request Sep 4, 2023 that will close this issue
stlintel pushed a commit to bog-dan-ro/Bochs that referenced this issue Sep 4, 2023
@hugsy
Copy link

hugsy commented Sep 7, 2023

Hello. Seems, migration to meson build system stalled. Any chances to adopt CMake? I can perform that migration. I can also update current autotools project to recent versions - this relatively easy task and can be done quicklier than migration.

@winterheart 💯
Moving to CMake would be an amazing addition to the project. Not to mention it would easily to integrate to other projects already using cmake. I don't think moving bochs to cmake would be hard, all necessary features already are built-in (like pre-building scripts, even packaging and more)

Cmake is too bloated, I am not a big fan of it. Meson is way easier.

CMake is less trivial but more advanced than Meson (in my opinion and experience - not trying to start a war 🙂)

I'd be more than happy to also contribute if you decide to go that route.

@AndersonTorres
Copy link

Not continuing the war, in my experience as a Nixpkgs contributor, cmake is harder to isolate.

In Nixpkgs we regularly patch cmake scripts that try to download things during build.
Meson also allows downloads, but

  1. In Meson you can block downloads via cmdline args;
  2. In Meson you can provide beforehand the files that would be downloaded; it just checks the hash;
  3. In cmake there is no way to do the things above: cmake forces the download, ignoring the files you provided;
  4. The only way to avoid the things in the above bullet is just by heavy patching the scripts;
  5. It is not easy to convince people that "hey, can you at least add this custom code in order to honor previously downloaded packages?".

For the programmer it maybe makes few to no difference; however for a package manager, it can be really annoying.

@stlintel stlintel added the enhancement request New feature or request label Dec 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants