search

simonwagner / mergepbx

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

Building produces unusable script #1

Closed ahti closed 11 years ago

ahti commented 11 years ago

When I build a file via build.py the output file is not usable:

  1. It has an empty line at the beginning, making the hashbang not recognized. This is easily fixed by hand, BUT:
  2. Running the fixed file results in the following:
> ./mergepbx 
Traceback (most recent call last):
  File "<stdin>", line 5, in <module>
ImportError: No module named mergepbx

The same problems apply to the prebuilt variant mentioned in the readme.

I'm running python 2.7.3 on OSX 10.8.

simonwagner commented 11 years ago

Could you specify the options you used for build.py? I tested the script and it works for me.

The correct usage is ./build.py mergepbx MANIFEST

ahti commented 11 years ago

i used the exact same options. i just submitted a pull request which fixes the issue for me (although i don't know why it fixes the "no module named mergepbx" issue)

simonwagner commented 11 years ago

Hm, I think removing the newline messed with the offsets in the zip file that is appended after the header. I will merge your pull request as soon as I am home, thank you.