It doesn't sort PBXGroup section some times

[edhub]

When I use it for our project, I noticed that, it doesn't sort PBXGroup until I change something in the project file (for example, add a empty group). Otherwise, it works great so far. It happens when we first apply it for our project. I'm not sure whether it'll happen in the future.

[truebit]

could you please provide the diff project.pbxproj before and after adding the empty group?

—
Sent from my mobile phone

On Thu, Feb 12, 2015 at 11:16 AM, edhub notifications@github.com wrote:

When I use it for our project, I noticed that, it doesn't sort PBXGroup until I change something in the project file (for example, add a empty group). Otherwise, it works great so far. It happens when we first apply it for our project. I'm not sure whether it'll happen in the future.

Reply to this email directly or view it on GitHub:
#17

[edhub]

it's a business project so I couldn't do that. But once i got some time (not sure when), I can look into it.

[edhub]

but I did manage to find a exercise project file with PBXGroup section not sorted correctly. It doesn't support file uploading, so I just copy & paste the PBXGroup part. If you need the whole project file, please let me know how can I send it to you.

check this line: FE01095F654E29E4669FD59051BF4A79

/* Begin PBXGroup section /
1C8BAAD40D3B5F927C6DC0A762A820BF / Supporting Files / = {
isa = PBXGroup;
children = (
7AB7B18155E2D756C57AD67E3DE5D5A7 / Info.plist /,
EBF0396F918D5ED54FC59C803080400E / main.m /,
);
name = "Supporting Files";
sourceTree = "";
};
304EE41ED71EA2205AC642471F07B575 / vcproject / = {
isa = PBXGroup;
children = (
1C8BAAD40D3B5F927C6DC0A762A820BF / Supporting Files /,
FE01095F654E29E4669FD59051BF4A79 / login /,
B96C9F610970F8CF3E0BCCB41EFA9178 / nav xibs /,
55F65BF1DF0D0D51AD7F9D4F02567D55 / AppDelegate.h /,
DC5DAD08E08AA82D3CCD97C6C5817593 / AppDelegate.m /,
638BBD9F9483E77C61604E4447BD5B5A / Images.xcassets /,
4545E029013F47D478AD3E2350C7E995 / LaunchScreen.xib /,
04C3104D63060826C7FD07A082CC4887 / Main.storyboard /,
1C0373E446C58D7B88A4E529C2539D8E / ViewController.h /,
7A4E408247BB90D5897E87505A7DFFCD / ViewController.m /,
);
path = vcproject;
sourceTree = "";
};
38585E645596D3E1FB84EEC2D5B221CA / vcprojectTests / = {
isa = PBXGroup;
children = (
88592D313A25A82215458E7B82554F6C / Supporting Files /,
10D2C4D9F826BFED19FB0F9D5F8F9CE3 / vcprojectTests.m /,
);
path = vcprojectTests;
sourceTree = "";
};
FE01095F654E29E4669FD59051BF4A79 / login / = {
isa = PBXGroup;
children = (
E237CB6A3187D7D3F653F196A0A7273F / LoginState.h /,
4155547A3FA1C427FF74C73319C03E99 / LoginState.m /,
E2200F9387C7D7E3ACE29F6D9203E909 / LoginViewController.h /,
02CD46DD35B87A9EB159E11E70461E4B / LoginViewController.m /,
D0C3E787681A0F13852B7116EB629323 / LoginViewController.xib /,
);
path = login;
sourceTree = "";
};
88592D313A25A82215458E7B82554F6C / Supporting Files / = {
isa = PBXGroup;
children = (
B469DD62CF1BF879F989E91E75D50B69 / Info.plist /,
);
name = "Supporting Files";
sourceTree = "";
};
B96C9F610970F8CF3E0BCCB41EFA9178 / nav xibs / = {
isa = PBXGroup;
children = (
FA256945FD94BCE957FBB8E0A30B289F / Tier1ViewController.h /,
2C3BDEA6335D293232CA22C7A945419A / Tier1ViewController.m /,
18906F410DE23C9449BA7597CADEDBA9 / Tier1ViewController.xib /,
5DD643C87B7B22E2F2864AB2780E265F / Tier2ViewController.h /,
A5AC728535B19B3281D3BBA6684670D2 / Tier2ViewController.m /,
21BB40856DE6B55A58E1BBBB34C69944 / Tier2ViewController.xib /,
DE933B89FE2D233DF042643A507567F3 / Tier3ViewController.h /,
8FBA5805AB7DD519CC775079CB7066C3 / Tier3ViewController.m /,
081B1E64EB95AC7FDD3EC5EB9114CB16 / Tier3ViewController.xib /,
);
path = "nav xibs";
sourceTree = "";
};
D640E569C0F0419DD5B042F6FE2C45CE / Products / = {
isa = PBXGroup;
children = (
F1CBBCD2C102E1D4A6C3DC4DF69C68FC / vcproject.app /,
04F573292FD3F4F4296248E759FE90EC / vcprojectTests.xctest /,
);
name = Products;
sourceTree = "";
};
E5EF063CFBDB072E436C9F57DCD4EA35 = {
isa = PBXGroup;
children = (
D640E569C0F0419DD5B042F6FE2C45CE / Products /,
304EE41ED71EA2205AC642471F07B575 / vcproject /,
38585E645596D3E1FB84EEC2D5B221CA / vcprojectTests /,
CC39A572ED7657CE5B27C217660E91A0 / MainTabs.storyboard /,
);
sourceTree = "";
};
/ End PBXGroup section */

[truebit]

Hi, Could you send the problematic original project file via replying github notification email? Thanks

[edhub]

here it is.

On Sat, Feb 14, 2015 at 11:26 AM, Sean Wang notifications@github.com
wrote:

Hi, Could you send the problematic original project file via replying
github notification email? Thanks

—
Reply to this email directly or view it on GitHub
#17 (comment).

[edhub]

i've replied the email with the file as attachment, but I didn't see it in the web page.

[truebit]

It should not display on web page. But I also did not see it in email. Did you " reply all"?

—
Sent from my mobile phone

On Sun, Feb 15, 2015 at 10:32 AM, edhub notifications@github.com wrote:

i've replied the email with the file as attachment, but I didn't see it in the web page.

Reply to this email directly or view it on GitHub:
#17 (comment)

[edhub]

this time i reply all. Hope it's ok now.

On Sun, Feb 15, 2015 at 10:34 AM, Sean Wang notifications@github.com
wrote:

It should not display on web page. But I also did not see it in email. Did
you " reply all"?

—
Sent from my mobile phone

On Sun, Feb 15, 2015 at 10:32 AM, edhub notifications@github.com wrote:

i've replied the email with the file as attachment, but I didn't see it

in the web page.

Reply to this email directly or view it on GitHub:
#17 (comment)

—
Reply to this email directly or view it on GitHub
#17 (comment).

[truebit]

Still not got it... Could you put the project file content on somewhere public? Thanks a lot.

[edhub]

I've uploaded the project to git hub :)
https://github.com/edhub/vcproject

[truebit]

@edhub Thanks a lot:)
But I found the project file has already been uniquified and sorted?
Could you push the problematic one?

[edhub]

please checkout haha branch. and it's not sorted well at this line:

B96C9F610970F8CF3E0BCCB41EFA9178 /* nav xibs */ = {

/* Begin PBXGroup section / 1C8BAAD40D3B5F927C6DC0A762A820BF / Supporting
Files / = { isa = PBXGroup; children = ( 7AB7B18155E2D756C57AD67E3DE5D5A7
/ Info.plist /, EBF0396F918D5ED54FC59C803080400E / main.m /, ); name =
"Supporting Files"; sourceTree = ""; };
304EE41ED71EA2205AC642471F07B575 / vcproject / = { isa = PBXGroup;
children = ( 1C8BAAD40D3B5F927C6DC0A762A820BF / Supporting Files /,
B96C9F610970F8CF3E0BCCB41EFA9178 / nav xibs /,
55F65BF1DF0D0D51AD7F9D4F02567D55 / AppDelegate.h /,
DC5DAD08E08AA82D3CCD97C6C5817593 / AppDelegate.m /,
638BBD9F9483E77C61604E4447BD5B5A / Images.xcassets /,
4545E029013F47D478AD3E2350C7E995 / LaunchScreen.xib /,
04C3104D63060826C7FD07A082CC4887 / Main.storyboard /,
1C0373E446C58D7B88A4E529C2539D8E / ViewController.h /,
7A4E408247BB90D5897E87505A7DFFCD / ViewController.m /, ); path =
vcproject; sourceTree = ""; }; 38585E645596D3E1FB84EEC2D5B221CA /
vcprojectTests / = { isa = PBXGroup; children = (
88592D313A25A82215458E7B82554F6C / Supporting Files /,
10D2C4D9F826BFED19FB0F9D5F8F9CE3 / vcprojectTests.m /, ); path =
vcprojectTests; sourceTree = ""; }; B96C9F610970F8CF3E0BCCB41EFA9178
/ nav xibs / = { isa = PBXGroup; children = (
FA256945FD94BCE957FBB8E0A30B289F / Tier1ViewController.h /,
2C3BDEA6335D293232CA22C7A945419A / Tier1ViewController.m /,
18906F410DE23C9449BA7597CADEDBA9 / Tier1ViewController.xib /,
5DD643C87B7B22E2F2864AB2780E265F / Tier2ViewController.h /,
A5AC728535B19B3281D3BBA6684670D2 / Tier2ViewController.m /,
21BB40856DE6B55A58E1BBBB34C69944 / Tier2ViewController.xib /,
DE933B89FE2D233DF042643A507567F3 / Tier3ViewController.h /,
8FBA5805AB7DD519CC775079CB7066C3 / Tier3ViewController.m /,
081B1E64EB95AC7FDD3EC5EB9114CB16 / Tier3ViewController.xib /, ); path =
"nav xibs"; sourceTree = ""; }; 88592D313A25A82215458E7B82554F6C /
Supporting Files / = { isa = PBXGroup; children = (
B469DD62CF1BF879F989E91E75D50B69 / Info.plist /, ); name = "Supporting
Files"; sourceTree = ""; }; D640E569C0F0419DD5B042F6FE2C45CE /
Products / = { isa = PBXGroup; children = (
F1CBBCD2C102E1D4A6C3DC4DF69C68FC / vcproject.app /,
04F573292FD3F4F4296248E759FE90EC / vcprojectTests.xctest /, ); name =
Products; sourceTree = ""; }; E5EF063CFBDB072E436C9F57DCD4EA35 = {
isa = PBXGroup; children = ( D640E569C0F0419DD5B042F6FE2C45CE / Products
/, 304EE41ED71EA2205AC642471F07B575 / vcproject /,
38585E645596D3E1FB84EEC2D5B221CA / vcprojectTests /,
CC39A572ED7657CE5B27C217660E91A0 / MainTabs.storyboard /, ); sourceTree =
""; };/ End PBXGroup section */

On Wed, Feb 25, 2015 at 11:47 PM, Sean Wang notifications@github.com
wrote:

@edhub https://github.com/edhub Thanks a lot:)
But I found the project file has already been uniquified and sorted?
Could you push the problematic one?

—
Reply to this email directly or view it on GitHub
#17 (comment).

[truebit]

I still not understand, there's only a haha branch in the project file, and I used that one.
concerning "it's not sorted well", could you explain more?

Do you mean that it is not sorted by lower letters? That's the default action of str.sort in Python. I did not change it. And I also added a rule that placing folders ( without . in name) before any files.
If that's not you mean, I will reopen this issue.

BTW, to save both our time, you should just say what's the issue. just saying "not sort well" and pasting the long content is too confused...

[edhub]

sorry, i replied it in email, and the result is a mess. Here, take a look at current orders in PBXGroup:
1C8BAAD40D3B5F927C6DC0A762A820BF /* Supporting Files /
304EE41ED71EA2205AC642471F07B575 / vcproject /
38585E645596D3E1FB84EEC2D5B221CA / vcprojectTests
-> B96C9F610970F8CF3E0BCCB41EFA9178 /* nav xibs /
88592D313A25A82215458E7B82554F6C / Supporting /
D640E569C0F0419DD5B042F6FE2C45CE / Products */
E5EF063CFBDB072E436C9F57DCD4EA35 = {

shouldn't it be:
1C8BAAD40D3B5F927C6DC0A762A820BF /* Supporting Files /
304EE41ED71EA2205AC642471F07B575 / vcproject /
38585E645596D3E1FB84EEC2D5B221CA / vcprojectTests
88592D313A25A82215458E7B82554F6C /* Supporting /
->B96C9F610970F8CF3E0BCCB41EFA9178 / nav xibs /
D640E569C0F0419DD5B042F6FE2C45CE / Products */
E5EF063CFBDB072E436C9F57DCD4EA35 = {