-
Notifications
You must be signed in to change notification settings - Fork 621
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
[meson] use the recommended way to store generated files #2532
Conversation
50a4f40
to
3b51a42
Compare
f50634f
to
46e7956
Compare
46e7956
to
a28e3d7
Compare
a28e3d7
to
6514f51
Compare
6514f51
to
c3d548c
Compare
c3d548c
to
783fdea
Compare
@@ -4,9 +4,9 @@ | |||
|
|||
os.chdir (os.environ.get ('srcdir', os.path.dirname (__file__))) | |||
|
|||
HBHEADERS = os.environ.get ('HBHEADERS', '').split () or \ | |||
HBHEADERS = [os.path.basename (x) for x in os.environ.get ('HBHEADERS', '').split ()] or \ |
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.
Why these are needed? File objects in meson (created using by files()
in this patch) when turned to string are like src/hb-x.hh
instead just the base name they used to, alternative approach would be top drop the above chdir
yet I like to keep the script still runnable from command line that's why I am doing this for now.
foreach f : hb_sources | ||
HBSOURCES += '@0@'.format(f) | ||
endforeach | ||
env.set('HBSOURCES', ' '.join(HBSOURCES)) |
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.
The for each above is to generate string out of files objects array, somehow a hack, I wish I could just pick the basename like the way it was before the patch at least so I didn't need to change check scripts.
As described in mesonbuild/meson#7156 (comment)
Fixes #2530