Manolis Ragkousis <manolis...@gmail.com> skribis:
>>> + (arguments
>>> + `(#:out-of-source? #t
>>> + #:phases (alist-replace
>>> + 'install
>>> + (lambda* (#:key outputs #:allow-other-keys)
>>> + (let ((out (assoc-ref outputs "out")))
>>> + ;; We need to copy libihash.a to the output folder
>>> manually,
>>> + ;; since there is no target for that in the makefile
>>> + (copy-recursively "libihash"
>>> + (string-append out "/libihash"))
>>
>> This creates
>> /gnu/store/.../libihash/{libihash.a,Makefile,ihash.c,ihash.o,...}. Is
>> it really what’d intended?
>>
>> I would expect that the intent is to create
>> /gnu/store/.../{lib/libihash.a,include/ihash.h}, no?
>>
> I will create two folders, lib with libihash.a and include with
> ihash.h. What do you think?
Yes, that’s what I’d expect. Double-check to make sure ihash.h doesn’t
include any other file that’d need to be installed.
(Please use “directory” rather than “folder” in comments & co.)
>>> ++if test "x$cross_compiling" = "xyes"; then
>>> ++ # It may be that we don't have a working libc yet, for instance
>>> ++ # because we're bootstrapping the cross-compilation tool chain.
>>> ++ # Thus, use this undocumented Autoconf macro designed for this.
>>> ++ AC_NO_EXECUTABLES
>>> ++ echo done
>>> ++fi
>>
>> I gather that this works as advertised, but could you confirm by
>> replying to
>> <http://lists.gnu.org/archive/html/bug-hurd/2014-04/msg00139.html>?
>> That would allow us to push the patch upstream.
>>
>
> I had to put one more line after AC_NO_EXECUTABLES, otherwise it would fail
> with
>> syntax error near unexpected token `fi'.
Oh I see, this macro actually expands to nothing.
> Other than that it works. I will confirm there.
Thanks.
>>> +-# Check for Sun RPC headers and library.
>>> ++# Check only for Sun RPC headers. We do not need the library yet.
>>> + AC_CHECK_HEADER([rpc/types.h], [HAVE_SUN_RPC=yes], [HAVE_SUN_RPC=no])
>>> +-AC_SEARCH_LIBS([clnt_create], [], [:], [HAVE_SUN_RPC=no])
>>> ++# AC_SEARCH_LIBS([clnt_create], [], [:], [HAVE_SUN_RPC=no])
>>> + AC_SUBST([HAVE_SUN_RPC])
>>
>> What’s the problem that’s addressed here?
>
> It tries to link with a library in glibc (which at that point doesn't
> yet exist ),
> but it cant because of the AC_NO_EXECUTABLES flag saying that linking
> is not allowed after this flag is passed. So I am just disabling the test, to
> get the job done.
OK, I see. Make sure to mention it in the patch comment as well.
Ludo’.
