Option for excluding specified groups (--exclusion, -e) doesn't function.

[felis-silvestris]

Following the usage as specified in the example: $ synx -p -e=OCMock/Core -e=OCKMockTests Source/OCMock.xcodeproj produces the following error output:

$ synx -p -e=Shared CBTest.xcodeproj
/Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/lib/synx/project.rb:88:in block in group_exclusions=': No group =Shared exists (IndexError) from /Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/lib/synx/project.rb:81:ineach'
from /Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/lib/synx/project.rb:81:in group_exclusions=' from /Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/lib/synx/project.rb:37:inset_options'
from /Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/lib/synx/project.rb:16:in sync' from /Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/bin/synx:15:inexecute'
from /Library/Ruby/Gems/2.0.0/gems/clamp-0.6.3/lib/clamp/command.rb:67:in run' from /Library/Ruby/Gems/2.0.0/gems/clamp-0.6.3/lib/clamp/command.rb:125:inrun'
from /Library/Ruby/Gems/2.0.0/gems/clamp-0.6.3/lib/clamp.rb:6:in Clamp' from /Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/bin/synx:6:in<top (required)>'
from /usr/bin/synx:23:in load' from /usr/bin/synx:23:in

'

If '=' is removed (like the --help usage suggests):
$ synx -p -e Shared CBTest.xcodeproj
the option is completely ignored and specified group is synced anyway.

[marklarr]

Ah! I need to fix the docs. Try prefixing your exclusions with /, ie,

synx -p -e=/Shared CBTest.xcodeproj

[marklarr]

It might be nice to make the option more lenient, too... maybe we should just prepend the / if they user doesn't provide it?

[felis-silvestris]

Nope, "/" doesn't help, get the same error:
/Library/Ruby/Gems/2.0.0/gems/synx-0.0.4/lib/synx/project.rb:88:in `block in group_exclusions=': No group =/Shared exists (IndexError)

[felis-silvestris]

Ok, this works:
$ synx -p -e /Shared CBTest.xcodeproj

[marklarr]

Thanks! I fixed the readme.

I think then is some general sketchiness with the exclusion feature, so let me know if you see any other issues.

[felis-silvestris]

Thanks for updating the readme! By the way, in the updated "Advanced" section example where is no "-p" option specified anymore, though the comment below it still references it:
"$ synx -e "/OCMock/Core Mocks" -e /OCMockTests Source/OCMock.xcodeproj/
if they wanted to not sync the /OCMock/Core Mocks and /OCMockTests groups, and also remove (-p) any image/source files found by synx that weren't ever referenced by any groups in Xcode."

[marklarr]

Ah! Thanks!

[lowell]

Okay, so, apparently the leading / is required. If I might make a suggestion, please make the requirement explicit or make the tool more forgiving, as suggested above. It (seemingly randomly) deleted headers and implementation files from folders both inside and outside the exclusion folder, and broke one of the submodule repos located in Vendor/, which locked up both Tower and Xcode. Strangely, git reset --hard restored only some of the deleted files; I ended up having to delete the entire folder and pulled down another clone of my project.

Anyway, thanks for sharing the project and have a great weekend! 🍻

[marklarr]

Sorry about that! This should fix it #18