-
Notifications
You must be signed in to change notification settings - Fork 45
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
Merging fails with: merging failed: u'PBXAggregateTarget' #14
Comments
You can send your project file to [email protected]. It would be best, if you created a dump of the merge. See
|
@simonwagner I'm seeing the exact same issue. Can you quickly reiterate how to use the dump option? Because I'm getting this error message when trying as you explained in the other issue:
Thanks! |
You need to place the file path where you want the dump to be written to after --dump. (That’s why it says „expected one argument“). So for example: „—dump /tmp/mergepbxdump.zip“ will write the file to „/tmp/mergepbxdump.zip“.
|
Perfect, thanks. I just emailed you the dump |
Thanks, I will take a look at it in the evening after work.
|
Any updates on this? Would love to start using mergepbx in more projects :) |
Working on it, unfortunately it's not only the missing entry for PBXAggregateTarget (that is an easy fix), but I also need to implement an algorithm for merging the compiler flags (which is a list and I have until now only implemented an algorithm for sets - list are unfortunately more difficult for a 3-way merge) |
I'm running into this too :( It'd be great to get a fix... until then I'll try using the 'union' merge as you suggested. |
@simonwagner Is there any update on this? |
It would be nice if the tool would not fail at least when the PBXAggregateTarget sections are exactly the same |
Let me know how I can provide more detailed and helpful info
The text was updated successfully, but these errors were encountered: