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, Apr 21, 11:48 AM
Unknown Object (File)
Wed, Apr 17, 9:34 PM
Unknown Object (File)
Feb 11 2024, 6:31 AM
Unknown Object (File)
Dec 20 2023, 8:36 AM
Unknown Object (File)
Nov 7 2023, 4:59 PM
Unknown Object (File)
Oct 28 2023, 10:34 PM
Unknown Object (File)
Oct 23 2023, 5:02 PM
Unknown Object (File)
Oct 20 2023, 8:03 AM

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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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