Solène Rapenne <sol...@perso.pw> writes:
>> Updated port attached, which should fix this issue.
>>
>>>> There is a texinfo doc but I don't know what to do with it, it's not
>>>> included in the package.
>>>
>>> The install method provided by upstream properly installs the info
>>> page.
>>>
>>>> I didn't used ${INSTALL_PROGRAM} macro because after stripping the
>>>> binary, starting stumpwm would only start a vanilla sbcl (the binary
>>>> should be around 60 Mb and after stripping it's only 620 kb).
>>>
>>> Upstream doesn't use install -s by default.
>>>
>>> The trick is to get upstream's install target to put stuff in the
>>> right
>>> directory. Here, setting DESTDIRNAME=destdir is enough ('cause
>>> upstream
>>> uses a variable named 'destdir' instead of something more standard
>>> like
>>> 'DESTDIR')
>>>
>>> Updated version, with the following changes:
>>> - trim COMMENT
>>> - sbcl is a BUILD_DEPEND only
>>> - use the autoconf CONFIGURE_STYLE, which automatically adds the
>>> proper
>>> deps to BUILD_DEPENDS, drop the custom do-configure step
>>> - use the latest autoconf version
>>> - trim CONFIGURE_ARGS
>>> - set PORTHOME (required, as you already noticed)
>>> - use upstream's install target
>
> I made some little changes
>
> - Removed CONFIGURE_ARGS because --with-lisp=sbcl is the default
> upstream
> - Replaced spaces by tabs in Makefile
> - added to Makefile patch "--disable-debugger" parameter to sbcl so if
> the build fails sbcl exits with an error without starting the
> interactive debugger
Looks fine to me.
--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
