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
F84581577: D14532.id39790.diff
Sat, May 25, 7:57 PM
Unknown Object (File)
Mon, May 13, 11:49 AM
Unknown Object (File)
Thu, May 2, 4:22 AM
Unknown Object (File)
Apr 16 2024, 4:42 PM
Unknown Object (File)
Apr 14 2024, 2:10 AM
Unknown Object (File)
Apr 12 2024, 4:44 AM
Unknown Object (File)
Dec 20 2023, 5:28 AM
Unknown Object (File)
Dec 4 2023, 3:49 PM
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.