more of an upstream bug, but still a bug. ('clisp-link add-module-set' has been changed to 'clisp-link add', but making appropriate change in Makefile still gives error 'directory already exists'.) Reproducible: Always Steps to Reproduce:
(In reply to comment #0) > ('clisp-link add-module-set' has been > changed to 'clisp-link add', but making appropriate change in Makefile still > gives error 'directory already exists'.) If anyone wondered this short bug submission, the error (without fixing attempts) looks probably like this: make[2]: Leaving directory `/var/tmp/portage/app-text/xindy-2.3/work/xindy-2.3/rte/ordrules' /usr/lib64/clisp-2.48//clisp-link add-module-set ordrules /usr/lib64/clisp-2.48//full xindy-build-dir Usage: /usr/lib64/clisp-2.48//clisp-link [ link | create | add | run ] ... make[1]: *** [xindy-build-dir] Error 1 When trying to fix it with the reported s/add-module-set[s]?/add/ it will probably result in something like this: make[2]: Leaving directory `/var/tmp/portage/app-text/xindy-2.3/work/xindy-2.3/rte/ordrules' /usr/lib64/clisp-2.48//clisp-link add ordrules /usr/lib64/clisp-2.48//full xindy-build-dir /usr/lib64/clisp-2.48//clisp-link: /usr/lib64/clisp-2.48//full already exists make[1]: *** [xindy-build-dir] Error 1 make[1]: Leaving directory `/var/tmp/portage/app-text/xindy-2.3/work/xindy-2.3/rte' make: *** [all-recursive] Error 1 A-ha, the directory should not exist? Or which one does xindy really even want, the old add-module-set (singular, gone from clisp-2.48 script) or add-module-sets (plural, renamed to add in clisp-2.48)? Actually, looking at the clisp-link script -- I cannot understand why it isn't failing with earlier clisp versions as well. Making the destination directory with make_dest has not been changed, and it appears to be supposed to always fail if the target exists.
Created attachment 203350 [details, diff] A patch in order to fix clisp 2.48 clisp-link add problem of Makefile in xindy rte directory. A patch in order to fix clisp 2.48 clisp-link add-module-set problem of Makefile in xindy rte directory. The "add-module-set" option is changed with "add" option and appropriate changes made acording to "add" option.
@ comment #2: Thanks for the patch. Works. Should have rtfm of clisp-link :). @ comment #1: Thanks for the details. Was in the middle of a 'emerge -e world' party and was in a hurry :).
I've talk to xindy's developers [0], so for 2.4 version we wont have this dependency problem. [0] https://sourceforge.net/tracker/?func=detail&atid=110646&aid=2854493&group_id=10646
patch works, thanks!
We probably need an ebuild that applies the patch when the installed clisp is higher than version 2.48 and otherwise doesn't. Otherwise applying the patch will break xindy for older clisps.
(In reply to comment #6) > We probably need an ebuild that applies the patch when the installed clisp is > higher than version 2.48 and otherwise doesn't. Otherwise applying the patch > will break xindy for older clisps. > Couldn't we let xindy 2.3 depend on old clisp, and create "new" version 2.3-r1 that depends on the new clisp applying the corresponding patch?
patch is ok, works well , Thank you all :)
@tex herd: any update? Can I review, merge, send-upstream the attached patch?
applied, thanks for the patch
