-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add preserve_path_from argument when running moc in headers
mode.
#10995
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10995 +/- ##
==========================================
- Coverage 68.33% 68.30% -0.03%
==========================================
Files 412 412
Lines 87789 87855 +66
Branches 20708 20724 +16
==========================================
+ Hits 59990 60009 +19
- Misses 23303 23348 +45
- Partials 4496 4498 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Would be useful if you edit an existing |
Hey @tristan957 , Update: I really have no clue how to set this up! I cannot enter directories multiple times but directories is an integral part of the test, so I can only check one Qt version. I'll try Qt6 then. |
- add a test program
02b66c9
to
e3df907
Compare
OK, I added a test setup. I am not at all happy to restrict the test to Qt 6 but I wouldn't know how else to do it. Also, having this test not as a self-contained entity but as part of this all-encompassing Qt module test is frustrating to say the least. There is no guessing of how much these tests might interact with each other and having some arbitrary directories |
@xclaesse maybe you can provide some testing guidance? |
Any further thoughts on this? I'd like to see this functionality added at some point in the future ... :) |
@eli-schwartz could you help provide input? Ideally this could go in before 1.0.0 :) |
Some time has passed and again, I am in need of this change. Is there something I can do to get this along? |
cc @eli-schwartz | @xclaesse. Could we get this in for 1.1.0? |
👍 I need this feature too. Would be useful to add this to I would rather add the |
superseeds mesonbuild#10995. This allow to generate ui and moc under subdirectories, as this is allowed with generic generators.
@punytroll I think maybe this would be much more palatable to you if we just changed the test itself to be parameterized by Qt version using an option plus test.json, rather than doing it in a foreach loop? Would that be sufficient or do you really desire that it be run as multiple different test case directories each testing one or two things? (I'm not entirely sure I see the necessity of this, but maybe I'm missing something.) |
This fixes #10955 partially.
When using the sources kwarg, the moc tool generates .moc files. Those are meant to be #included in a .cpp file, which makes it necessary to add the correct -Ibuild/dir flag to the correct .cpp file compiling command. I'm not sure whether this is wanted or even possible ...