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
F158589371: D14532.id39790.diff
Wed, Jun 3, 5:28 PM
Unknown Object (File)
Mon, Jun 1, 1:43 PM
Unknown Object (File)
Apr 28 2026, 8:57 PM
Unknown Object (File)
Apr 27 2026, 12:10 PM
Unknown Object (File)
Apr 20 2026, 6:39 AM
Unknown Object (File)
Apr 14 2026, 7:04 PM
Unknown Object (File)
Apr 14 2026, 11:20 AM
Unknown Object (File)
Apr 14 2026, 9:47 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

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.