Skip to content

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented Jan 4, 2024

The script was taken from:

... where it was called generate-meson.py and intended to be removed before merging. #36524 (comment)

Here we make it available as a script installed by sage_setup because what was useful once for generating code will be useful later for updating code, in particular as will be necessary for modularization concerns, as discussed in #36524 (comment)

Various improvements have been made. When invoked with the switch --monolithic, the output maintains compatibility with #36524.

Author: @tobiasdiez, @mkoeppe

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@tobiasdiez
Copy link
Contributor

As it has been explained before, the point of this script was to create the initial boilerplate code. I see value in creating a script that adds missing python and cython files. But that would require to parse the existing meson files instead of overwriting them. Something similar is already supported for other targets, just not for python. https://mesonbuild.com/Rewriter.html So I am sure the meson team would be happy to accept your PR.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jan 5, 2024

that would require

It wouldn't. Even just overwriting + then editing often has value.

@orlitzky
Copy link
Contributor

With the .in files, does this now avoid overwriting handwritten meson.build? @tobiasdiez should really be the one to OK this in any case.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 12, 2024

With the .in files, does this now avoid overwriting handwritten meson.build?

Yes, the idea is to separate handwritten bits from autogenerated bits.

Copy link

github-actions bot commented Apr 1, 2024

Documentation preview for this PR (built with commit ec43bc5; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@kiwifb
Copy link
Member

kiwifb commented Jun 7, 2024

I think we now have pressing needs to move to meson. This stuff looks OK, but it cannot be used in isolation. Can it even be tested without other bits for sagemath-standard?

tobiasdiez and others added 20 commits June 22, 2024 16:57
From PR 'Compile everything with meson' (sagemath#36524 at 46bc6b4 (2023-12-31)
…ything with meson (sagemath#36524 at 46bc6b4 (2023-12-31) squashed on 10.3.beta4
….build files from PR Compile everything with meson (sagemath#36524 at 46bc6b4 (2023-12-31) squashed on 10.3.beta4
…son.build files from PR Compile everything with meson (sagemath#36524 at 46bc6b4 (2023-12-31) squashed on 10.3.beta4
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jun 22, 2024

Can it even be tested without other bits for sagemath-standard?

Well, it generates files that can be compared with what has been committed to #36524.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants