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

[Qt module] Add experimental Warning on wrong file extensions #2565

Closed
wants to merge 1 commit into from

Conversation

jeandet
Copy link
Member

@jeandet jeandet commented Nov 1, 2017

This is a test to add warning when an user gives a file with a wrong
extension to preprocess method.

This behaviour might be generalized to meson other modules.
It also should print meson.build file line.

@jeandet jeandet added the qt label Nov 1, 2017
@jeandet
Copy link
Member Author

jeandet commented Nov 1, 2017

#2563

# Expect a warning since mainWindow.h is a header
qtmodule.preprocess(moc_sources : 'mainWindow.h')
# Expect a warning since mainWindow.cpp isn't a header
qtmodule.preprocess(moc_headers : 'mainWindow.h')
Copy link
Contributor

Choose a reason for hiding this comment

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

Cut and paste error? mainWindow.h -> mainWindow.cpp

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, it's a mistake :)

This is a test to add warning when an user gives a file with a wrong
extension to preprocess method.

Signed-off-by: Alexis Jeandet <alexis.jeandet@member.fsf.org>
@jpakkane
Copy link
Member

jpakkane commented Nov 6, 2017

Commits that fix open issues should end their commit messages with Closes #xxxx.. This will autoclose the issue on merge to master.

@nirbheek
Copy link
Member

nirbheek commented Jun 3, 2018

Superseded by #2573, closing.

@nirbheek nirbheek closed this Jun 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants