Page MenuHomeFreeBSD

Fix the problem described in PR #226221. (I think)
ClosedPublic

Authored by mat on Feb 27 2018, 2:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 12, 9:26 AM
Unknown Object (File)
Sun, Nov 10, 10:19 PM
Unknown Object (File)
Thu, Nov 7, 7:17 PM
Unknown Object (File)
Sep 28 2024, 12:59 PM
Unknown Object (File)
Sep 27 2024, 11:01 AM
Unknown Object (File)
Sep 25 2024, 4:14 PM
Unknown Object (File)
Sep 24 2024, 3:15 AM
Unknown Object (File)
Sep 20 2024, 11:33 AM
Subscribers
None

Details

Summary

The problem is that GH_SUBDIR are handled in a somewhat random manner. (The
truth is that in the end of things, they end up being sorted by the group name
being used in the GH_TUPLE.)

So if you have a submodule in bar/foo, and a sub-submodule in bar/foo/baz, it
may happen that foo/bar/baz is handled before foo/bar and then things are
messed up.

This makes it so the GH_SUBDIR target handling is sorted first by the
number of / in the path. (So, bar/foo is always handled before
bar/foo/baz.)

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 15300
Build 15358: arc lint + arc unit

Event Timeline

  • currently should not be used.

Looks good to me, though I am not a ports committer (yet), so it would be better to have someone else to review it :-)

This revision is now accepted and ready to land.Feb 27 2018, 7:16 PM
In D14532#304868, @ygy wrote:

Looks good to me, though I am not a ports committer (yet), so it would be better to have someone else to review it :-)

But does it fix the problem you are experiencing?

In D14532#305009, @mat wrote:
In D14532#304868, @ygy wrote:

Looks good to me, though I am not a ports committer (yet), so it would be better to have someone else to review it :-)

But does it fix the problem you are experiencing?

I did the test against my working port, and this patch worked beautifully. Thanks!

This revision was automatically updated to reflect the committed changes.