alunny / node-xcode

tools and utilities for working with xcode/ios projects
Apache License 2.0
226 stars 105 forks source link

Fix parser to merge properties of multiple sections with the same name #35

Closed mbektchiev closed 9 years ago

mbektchiev commented 10 years ago

Xcode handles this correctly and some tools might produce such projects.

mbektchiev commented 9 years ago

@imhotep Can you review and possibly merge this PR, please?

imhotep commented 9 years ago

Hi really sorry for the delay. Tests are not passing.

mbektchiev commented 9 years ago

Yes, indeed. I've fixed them and now they pass. The problem is that the check compares the strings verbatim but when there are multiple sections the files cannot be equal when merged. I've excluded the split sections project file from the verbatim checking scripts.

imhotep commented 9 years ago

Thanks @mbektchiev