Skip to content

Commit

Permalink
devel/libnotify: fix build with lld 17
Browse files Browse the repository at this point in the history
Building devel/libnotify with lld 17 results in the following link
error:

  cc  -o libnotify/libnotify.so.4.0.0 libnotify/libnotify.so.4.0.0.p/meson-generated_.._notify-enum-types.c.o libnotify/libnotify.so.4.0.0.p/meson-generated_.._notify-marshal.c.o libnotify/libnotify.so.4.0.0.p/notify.c.o libnotify/libnotify.so.4.0.0.p/notification.c.o -Wl,--as-needed -Wl,--no-undefined -Wl,-O1 -shared -fPIC -Wl,--start-group -Wl,-soname,libnotify.so.4 -fstack-protector-strong -O2 -pipe -fstack-protector-strong -fno-strict-aliasing -Wl,-rpath,/usr/local/lib -Wl,-rpath-link,/usr/local/lib -Wl,--version-script,/wrkdirs/share/dim/ports/devel/libnotify/work/libnotify-0.8.2/libnotify/libnotify.map /usr/local/lib/libgdk_pixbuf-2.0.so /usr/local/lib/libgobject-2.0.so /usr/local/lib/libglib-2.0.so /usr/local/lib/libintl.so /usr/local/lib/libgio-2.0.so -Wl,--end-group
  ld: error: version script assignment of 'global' to symbol '__progname' failed: symbol not defined
  ld: error: version script assignment of 'global' to symbol 'environ' failed: symbol not defined
  cc: error: linker command failed with exit code 1 (use -v to see invocation)

This is because lld 17 defaults to errors when undefined symbols are
referenced in linker version scripts.

Apparently the __progname and environ symbols (which usually come from
libc, not libnotify.so) were explicitly added in:

commit 0b0b10c
Author: Baptiste Daroussin <bapt@FreeBSD.org>
Date:   2019-12-16T19:40:37+01:00

    Fix build with old binutils.

    Somehow linker scripts should not hide __progname and environ from symbols.

    Reported by:    kib

but it is unclear what the original error was, or how old "old" is, and
there is no reference to a PR.

In any case, I think the patch-libnotify_libnotify.map file can simply
be deleted now.

PR:		274108
Approved by:    maintainer timeout (10 days)
MFH:		2023Q4
  • Loading branch information
DimitryAndric committed Oct 6, 2023
1 parent a2f61c3 commit 623262a
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions devel/libnotify/files/patch-libnotify_libnotify.map

This file was deleted.

0 comments on commit 623262a

Please sign in to comment.