Page MenuHomeFreeBSD

Remove name mangling in generate-ucl.sh
ClosedPublic

Authored by markj on Apr 21 2021, 2:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 7:10 AM
Unknown Object (File)
Sat, Dec 21, 1:12 PM
Unknown Object (File)
Dec 3 2024, 6:41 AM
Unknown Object (File)
Oct 2 2024, 6:39 PM
Unknown Object (File)
Oct 2 2024, 1:04 PM
Unknown Object (File)
Oct 1 2024, 12:16 AM
Unknown Object (File)
Sep 27 2024, 12:44 PM
Unknown Object (File)
Sep 26 2024, 4:14 PM

Details

Summary

I don't plan to commit this yet, as we're not sure why the name mangling
was there to begin with. r364009 introduced a regression for packages
containing a "_" in the name, such as libcompiler_rt. Removing the name
mangling fixes the problem.

Test Plan

I no longer see errors about missing dependencies when installing base packages.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 38751
Build 35640: arc lint + arc unit

Event Timeline

markj requested review of this revision.Apr 21 2021, 2:51 PM
markj added reviewers: manu, pkgbase.
In D29883#670691, @me_igalic.co wrote:

i wonder if this regression is the cause of https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=254173

It doesn't seem so. There is a missing dependency specification: ld actually depends on the -dev package for a couple of libraries, as you found. In particular, clibs-dev, libcompiler_rt-dev and utilities-dev are all required in order to dynamically link a program.

I think this is indeed a bug. Other -dev packages don't explicitly depend on the "core" -dev packages either.

I don't remember why I did this but yes this solve some dependencies issues.

In D29883#672527, @manu wrote:

I don't remember why I did this but yes this solve some dependencies issues.

Ok, thanks. I think I will go ahead and commit this with a generous MFC period to see if I break anything. The original name mangling was added long enough ago that I want to believe that whatever problem it solved no longer exists.

In D29883#672527, @manu wrote:

I don't remember why I did this but yes this solve some dependencies issues.

Ok, thanks. I think I will go ahead and commit this with a generous MFC period to see if I break anything. The original name mangling was added long enough ago that I want to believe that whatever problem it solved no longer exists.

Yes I believe that too.

This revision is now accepted and ready to land.Apr 26 2021, 1:10 PM