Bug 270963 - textproc/sxml: fails to build when base built WITH_LLVM_BINUTILS
Summary: textproc/sxml: fails to build when base built WITH_LLVM_BINUTILS
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks: 258872
  Show dependency treegraph
 
Reported: 2023-04-20 18:18 UTC by Ed Maste
Modified: 2024-02-22 16:00 UTC (History)
2 users (show)

See Also:
takefu: maintainer-feedback+


Attachments
sxml-1.0.7.patch (4.36 KB, patch)
2023-04-23 23:16 UTC, takefu
takefu: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Maste freebsd_committer freebsd_triage 2023-04-20 18:18:48 UTC
From exp-run PR 258872

https://pkg-status.freebsd.org/gohan05/data/mainamd64PR258872-default/2023-04-17_22h40m57s/logs/errors/sxml-1.0.7.log

=======================<phase: stage          >============================
===>  Staging for sxml-1.0.7
===>   Generating temporary packing list
/bin/rm /wrkdirs/usr/ports/textproc/sxml/work/stage/usr/local/lib/libsxml.a
rm: /wrkdirs/usr/ports/textproc/sxml/work/stage/usr/local/lib/libsxml.a: No such file or directory
*** Error code 1 (ignored)
install  -m 0644 libsxml.a /wrkdirs/usr/ports/textproc/sxml/work/stage/usr/local/lib
ranlib -t /wrkdirs/usr/ports/textproc/sxml/work/stage/usr/local/lib/libsxml.a
ranlib: error: Invalid option: '-t'
*** Error code 1
Comment 1 takefu 2023-04-23 23:16:43 UTC
Created attachment 241691 [details]
sxml-1.0.7.patch

fix
  install program
  portlint(1) and portclippy(1) compliant.

note
  I haven't tried it with current as of today.
  So please report your results. :-)
Comment 2 Ed Maste freebsd_committer freebsd_triage 2023-04-24 16:44:24 UTC
I applied the patch and it still failed to build -- I still see a `strip -x`:

# grep -r STRIP math/trlan
math/trlan/Makefile:    @${STRIP_CMD} -x -o ${WRKDIR}/libtrlan.a ${WRKSRC}/libtrlan.a
Comment 3 takefu 2023-04-24 22:06:34 UTC
(In reply to Ed Maste from comment #2)

It's not related to my patch.
Also, isn't review D39754 a countermeasure?

bug #270958 :-)
Comment 4 Ed Maste freebsd_committer freebsd_triage 2023-04-25 15:23:27 UTC
(In reply to takefu from comment #3)
Oops, yes my comment above is on the wrong PR.
Comment 5 Ed Maste freebsd_committer freebsd_triage 2024-02-21 16:36:47 UTC
I get lots of warnings when building (which upstream will hopefully address), a la

sxml.c:107:1: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
strxsep(stringp, delim)
^

But build is successful with the patch.
Comment 6 Robert Clausecker freebsd_committer freebsd_triage 2024-02-22 16:00:06 UTC
Attachment #241691 [details] needs to be updated for proper INSTALL_... uses as per ยง 5.17.1 Porter's Handbook.  You can't just use INSTALL_PROGRAM for all of these files.

Also, your do-install target does not install the library files correctly.  You need to install a libfoo.so.x.y.z and a symlink to that from libfoo.so.x and libfoo.so.  Please check and correct the do-install target.