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

port:meson : mesonlib.darwin_get_object_archs only accepts str type #64

Closed
wants to merge 1 commit into from
Closed

port:meson : mesonlib.darwin_get_object_archs only accepts str type #64

wants to merge 1 commit into from

Conversation

Tatsh
Copy link
Contributor

@Tatsh Tatsh commented Aug 21, 2019

Related to #52. mesonlib.darwin_get_object_archs only accepts str and does not convert Path objects automatically.

@RJVB
Copy link
Owner

RJVB commented Aug 21, 2019 via email

@Tatsh
Copy link
Contributor Author

Tatsh commented Aug 21, 2019

From what I can tell, this is fixed in newer versions of Meson but not in a release.

@RJVB
Copy link
Owner

RJVB commented Aug 21, 2019 via email

@Tatsh
Copy link
Contributor Author

Tatsh commented Apr 26, 2020

My patch was accepted upstream. Could you include this patch until they make a new release?

mesonbuild/meson#7022

@RJVB
Copy link
Owner

RJVB commented Apr 27, 2020 via email

@RJVB RJVB closed this in d47df1c May 17, 2020
@RJVB
Copy link
Owner

RJVB commented May 17, 2020

Patch incorporated (still applies) even though there has been a new version 2 days ago?!

@Tatsh
Copy link
Contributor Author

Tatsh commented May 17, 2020

No. The new version has my change.

@RJVB
Copy link
Owner

RJVB commented May 17, 2020 via email

@Tatsh
Copy link
Contributor Author

Tatsh commented May 17, 2020

I stand corrected. The 0.54.2 release is based on a commit from 225 commits ago. It does not contain my change. They seem to do releases like so:

  1. Make major version branch - 0.54, tag 0.54.0. This is where all patches to 0.54 will go, but changes like mine would never go here. That would be version 0.55 I guess.
  2. Merge patches, tag each point release from this branch - 0.54.2

@RJVB
Copy link
Owner

RJVB commented May 17, 2020 via email

@Tatsh Tatsh deleted the meson-str-fix branch May 17, 2020 10:47
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.

2 participants