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 feed arg to custom_target() #8923
Conversation
Codecov Report
@@ Coverage Diff @@
## master #8923 +/- ##
=======================================
Coverage 66.32% 66.33%
=======================================
Files 372 372
Lines 84009 84043 +34
Branches 17515 17525 +10
=======================================
+ Hits 55723 55748 +25
- Misses 23376 23381 +5
- Partials 4910 4914 +4
Continue to review full report at Codecov.
|
248072a
to
480624d
Compare
@@ -383,6 +383,10 @@ following. | |||
`{'NAME1': 'value1', 'NAME2': 'value2'}` or `['NAME1=value1', 'NAME2=value2']`, | |||
or an [`environment()` object](#environment-object) which allows more | |||
sophisticated environment juggling. | |||
- `feed` *(since 0.59.0)*: there are some compilers that can't be told to read |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps want to write 'programs' rather than 'compilers'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. The capture
description mentions "compilers", so I just went with that for consistency. Can change if "programs" is better.
|
||
# Note that this will not add a dependency to the compiler executable. | ||
# Code will not be rebuilt if it changes. | ||
comp = '@0@/@1@'.format(meson.current_source_dir(), 'my_compiler.py') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Um... why are we doing this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is copy-pasta from 109 custom target capture
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that seems to be that way since it was added in 70d94a5. I have no idea why :)
This requires a release note snippet. |
480624d
to
a835e7b
Compare
Done. |
Thanks for the review! |
Since [1], Meson allows feeding the input file as stdin and capturing stout to the output file. We don't need the sh hack anymore. [1]: mesonbuild/meson#8923
Since [1], Meson allows feeding the input file as stdin and capturing stout to the output file. We don't need the sh hack anymore. [1]: mesonbuild/meson#8923
Since [1], Meson allows feeding the input file as stdin and capturing stout to the output file. We don't need the sh hack anymore. [1]: mesonbuild/meson#8923
Since [1], Meson allows feeding the input file as stdin and capturing stout to the output file. We don't need the sh hack anymore. [1]: mesonbuild/meson#8923
Since [1], Meson allows feeding the input file as stdin and capturing stout to the output file. We don't need the sh hack anymore. [1]: mesonbuild/meson#8923
An example of a tool not accepting an input file param is scdoc.