[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/4] Use AF_ALG in checksum utilities
From: |
Matteo Croce |
Subject: |
Re: [PATCH 0/4] Use AF_ALG in checksum utilities |
Date: |
Tue, 24 Apr 2018 12:50:02 +0200 |
On Mon, Apr 23, 2018 at 3:54 PM, Bruno Haible <address@hidden> wrote:
> A few hints regarding the gnulib coding style:
>
> * As you already noticed, we need to avoid build failures and runtime
> failures
> on platforms where this is not supported.
>
I see "#ifdef __linux__" already in use in gnulibs, I assume this is
the preferred way.
> * In the module description, section 'Include', you should not list all
> include
> files but only those that the user is supposed to include. In this case,
> I think, the af_alg business is invisible to the caller of the 4 modules.
>
How to conditionally add files to lib_SOURCES depending on the system?
I guess that such code snippet is pasted somewhere in the Makefile, so
something like should work:
lib_SOURCES += sha1.c
UNAME_S := $(shell uname -s)
ifeq ($(UNAME_S),Linux)
lib_SOURCES += af_alg.c
endif
or do you have a better solution?
--
Matteo Croce
per aspera ad upstream