Skip to content

Commit

Permalink
[linux] Do not embed kernel version in CFLAGS (#314)
Browse files Browse the repository at this point in the history 
This change makes building on Linux reproducible, closing issue #310.
  • Loading branch information
@OstCollector
OstCollector authored May 1, 2024
1 parent ceb40c3 commit 4964dca
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions configure.ac
View file
Original file line number Diff line number Diff line change
Expand Up @@ -242,8 +242,10 @@ AM_CONDITIONAL([SOLARIS], [test x$LSOF_DIALECT_DIR = xsun])
AM_CONDITIONAL([AIX], [test x$LSOF_DIALECT_DIR = xaix])

# Pass OS version
LSOF_TMP=$(echo $LSOF_VSTR | sed 's/(/\\\\(/g' | sed 's/)/\\\\)/g')
CFLAGS="$CFLAGS -DLSOF_VSTR=\\\"$LSOF_TMP\\\""
AS_IF([test x$LSOF_DIALECT_DIR != xlinux], [
LSOF_TMP=$(echo $LSOF_VSTR | sed 's/(/\\\\(/g' | sed 's/)/\\\\)/g')
CFLAGS="$CFLAGS -DLSOF_VSTR=\\\"$LSOF_TMP\\\""
])

# Pass LSOF_DIALECT/LSOF_DIALECT_DIR to Makefile.am
AC_SUBST([LSOF_DIALECT])
Expand Down

1 comment on commit 4964dca

@kpcyrd
Copy link

@kpcyrd kpcyrd commented on 4964dca May 1, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! 🖤

Please sign in to comment.