Opened 3 years ago

Closed 3 years ago

#50151 closed update (fixed)

dyncall: upgrade to 0.8, fix livecheck, fix build/packaging

Reported by: mojca (Mojca Miklavec) Owned by: dadler@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: ryandesign (Ryan Schmidt), dadler@…
Port: dyncall


I was checking dyncall as a dependency of MoarVM and noticed that it was outdated (livecheck doen't work).

When I tried to upgrade it to 0.8, I realized that it was trying to circumvent the $DESTDIR:

make[1]: Entering directory `/path/to/dyncall/work/dyncall-0.8/dyncall'
mkdir -p /opt/local/lib 
mkdir -p /opt/local/include
cp libdyncall_s.a /opt/local/lib
cp: /opt/local/lib/libdyncall_s.a: Operation not permitted
make[1]: *** [install] Error 1
make[1]: Leaving directory `/path/to/dyncall/work/dyncall-0.8/dyncall'
make: *** [install-libdyncall] Error 2

This seems to be an upstream bug, perhaps introduced in 0.8.

But I noticed that CMake was also supported, so I switched to CMake successfully. When comparing the resulting package with the one in, I realized that CMake forgets to install the man page.

I'm attaching a diff file that fixes most issues except for the man page. It would be nice to get maintainer's confirmation though.

Attachments (1)

dyncall-0.8.diff (1.2 KB) - added by mojca (Mojca Miklavec) 3 years ago.
Upgrade dyncall to 0.8, fix livecheck, switch to cmake

Download all attachments as: .zip

Change History (3)

Changed 3 years ago by mojca (Mojca Miklavec)

Attachment: dyncall-0.8.diff added

Upgrade dyncall to 0.8, fix livecheck, switch to cmake

comment:1 Changed 3 years ago by mf2k (Frank Schima)

Keywords: haspatch added

comment:2 Changed 3 years ago by mojca (Mojca Miklavec)

Resolution: fixed
Status: newclosed

Maintainer didn't reply yet, so I committed an update in r144347.

(I didn't find any sources of a manpage, so I assume the man page is no longer there. Probably replaced by PDF documentation.)

Note: See TracTickets for help on using tickets.