Page MenuHomeFreeBSD

net-im/libsignal-node: Fix poudriere build
ClosedPublic

Authored by jrm on Oct 6 2024, 3:16 PM.
Tags
None
Referenced Files
F106260344: D46966.id144339.diff
Sat, Dec 28, 3:02 AM
Unknown Object (File)
Tue, Dec 17, 1:44 AM
Unknown Object (File)
Nov 27 2024, 7:24 PM
Unknown Object (File)
Nov 26 2024, 8:20 AM
Unknown Object (File)
Nov 21 2024, 7:06 AM
Unknown Object (File)
Nov 20 2024, 12:22 AM
Unknown Object (File)
Nov 19 2024, 1:47 AM
Unknown Object (File)
Nov 3 2024, 7:00 PM
Subscribers
None

Details

Summary

After the update to 0.58.0, the port failed to build in poudriere during
the fetch phase, giving the error:

make: Could not create temporary file
/wrkdirs/usr/ports/net-im/libsignal-node/work/makekSsI9M: No such file
or directory

Instead of removing ${WRKDIR} in the pre-fetch phase, clearing its
contents resolves the issue and allows the build to succeed.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 59794
Build 56680: arc lint + arc unit

Event Timeline

jrm requested review of this revision.Oct 6 2024, 3:16 PM
jrm created this revision.
net-im/libsignal-node/Makefile
72

If we want to be cautious, we could instead use [ -d "${WRKDIR}" ] && ${RM} -r ${WRKDIR:?}/*; \ or ${RM} -r ${WRKDIR} && ${MKDIR} ${WRKDIR}; \.

This revision is now accepted and ready to land.Oct 7 2024, 6:52 AM

In that case, I'll completely remove the "${RM}" to match your previous fix. Given you did this in the past, I assume there is no harm in leaving the temporary ${WRKDIR} contents around.

I forgot to merge this change in https://github.com/MikaelUrankar/signal-desktop

You have my implicit approval to fix my stupid mistakes

Thank you.

Thanks for clarifying the issue.

This revision was automatically updated to reflect the committed changes.