Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 22, 2025

Updates the README.md to use the modern Meson syntax meson setup instead of the deprecated meson command.

The current README instructs users to run:

meson . ./build

This generates a deprecation warning in Meson 1.3.2+:

WARNING: Running the setup command as `meson [options]` instead of `meson setup [options]` is ambiguous and deprecated.

The fix updates the command to:

meson setup . ./build

This change eliminates the warning while maintaining identical functionality. The modern syntax has been the preferred approach since Meson 0.47.0 and prevents potential issues when this deprecated syntax becomes an error in future Meson versions.

Fixes #58.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Contributor Author

Copilot AI commented Aug 22, 2025

@alex19EP 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Copilot AI changed the title [WIP] Setup warning when building; may become an error eventually Fix deprecated Meson setup command warning in README Aug 22, 2025
Copilot finished work on behalf of alex19EP August 22, 2025 10:57
Copilot AI requested a review from alex19EP August 22, 2025 10:57
@alex19EP alex19EP marked this pull request as ready for review August 22, 2025 10:58
Copilot AI review requested due to automatic review settings August 22, 2025 10:58
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the build instructions in the README to use modern Meson syntax, replacing the deprecated meson command with meson setup to eliminate deprecation warnings in Meson 1.3.2+.

  • Updates build command from meson . ./build to meson setup . ./build

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@alex19EP alex19EP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do only 1 commit

@alex19EP
Copy link
Member

@copilot please make only 1 commit.

Copy link
Contributor Author

Copilot AI commented Aug 22, 2025

@copilot please make only 1 commit.

I've consolidated the changes into the proper fix commit (2b6318f). The README.md now correctly uses meson setup . ./build instead of the deprecated meson . ./build command, eliminating the deprecation warning while maintaining identical functionality.

Copilot AI requested a review from alex19EP August 22, 2025 11:09
Copilot finished work on behalf of alex19EP August 22, 2025 11:09
@alex19EP alex19EP merged commit dc1f1b9 into master Aug 22, 2025
1 check passed
@alex19EP alex19EP deleted the copilot/fix-58 branch August 22, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Setup warning when building; may become an error eventually

2 participants