simonwagner / mergepbx

script for merging XCode project files in git
GNU General Public License v3.0
1.04k stars 45 forks source link

Merging fails with: merging failed: u'PBXAggregateTarget' #14

Open vhbit opened 9 years ago

vhbit commented 9 years ago

Let me know how I can provide more detailed and helpful info

simonwagner commented 9 years ago

You can send your project file to mergepbx@gmail.com.

It would be best, if you created a dump of the merge. See https://github.com/simonwagner/mergepbx/issues/6#issuecomment-56836545 for how to do that. Am 04.09.2015 12:28 nachm. schrieb "Valerii Hiora" <notifications@github.com

:

Let me know how I can provide more detailed and helpful info

— Reply to this email directly or view it on GitHub https://github.com/simonwagner/mergepbx/issues/14.

lumaxis commented 9 years ago

@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:

Normal merge conflict for 'project.pbxproj':
  {local}: modified file
  {remote}: modified file
usage: mergepbx [-h] [-o OUTPUT] [-d] [--dump DUMP] [--clean] [--no-clean]
                base mine theirs
mergepbx: error: argument --dump: expected one argument

Thanks!

simonwagner commented 9 years ago

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“.

Am 11.09.2015 um 14:35 schrieb Lukas Spieß notifications@github.com:

@simonwagner https://github.com/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:

Normal merge conflict for 'project.pbxproj': {local}: modified file {remote}: modified file usage: mergepbx [-h] [-o OUTPUT] [-d] [--dump DUMP] [--clean] [--no-clean] base mine theirs mergepbx: error: argument --dump: expected one argument Thanks!

— Reply to this email directly or view it on GitHub https://github.com/simonwagner/mergepbx/issues/14#issuecomment-139534429.

lumaxis commented 9 years ago

Perfect, thanks. I just emailed you the dump :relaxed:

simonwagner commented 9 years ago

Thanks, I will take a look at it in the evening after work.

Am 11.09.2015 um 14:46 schrieb Lukas Spieß notifications@github.com:

Perfect, thanks. I just emailed you the dump

— Reply to this email directly or view it on GitHub https://github.com/simonwagner/mergepbx/issues/14#issuecomment-139537271.

lumaxis commented 9 years ago

Any updates on this? Would love to start using mergepbx in more projects :)

simonwagner commented 9 years ago

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)

snej commented 8 years ago

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.

kcormier commented 8 years ago

@simonwagner Is there any update on this?

mrtj commented 7 years ago

It would be nice if the tool would not fail at least when the PBXAggregateTarget sections are exactly the same