Re: [systemd-devel] [PATCH] buildsys: remove always true autoconf checks

2015-05-21 Thread Lennart Poettering
On Tue, 19.05.15 20:17, Cristian Rodríguez (crrodrig...@opensuse.org) wrote:

 All this checks are always true in any modernish linux system.
 ---
  configure.ac | 11 ---
  1 file changed, 11 deletions(-)
 
 diff --git a/configure.ac b/configure.ac
 index 3efee22..cd6375b 100644
 --- a/configure.ac
 +++ b/configure.ac
 @@ -1293,17 +1293,6 @@ AC_CHECK_HEADERS_ONCE([valgrind/memcheck.h 
 valgrind/valgrind.h])
  have_myhostname=no
  AC_ARG_ENABLE(myhostname, AS_HELP_STRING([--disable-myhostname], [disable 
 nss-myhostname support]))
  if test x$enable_myhostname != xno; then
 -AC_HEADER_STDC
 -AC_CHECK_HEADERS([arpa/inet.h fcntl.h inttypes.h netdb.h
 netinet/in.h stdlib.h string.h sys/socket.h sys/time.h unistd.h
 nss.h sys/ioctl.h sys/auxv.h])

sys/auxv.h at least is a relatively recent addition to glibc, hence
simply removing this all appears too broad...

Not sure however why this is in the enable_myhostname part however, it
should just be moved up. Could you prep a patch that does that?

 -
 -AC_C_CONST
 -AC_TYPE_SIZE_T
 -AC_HEADER_TIME
 -
 -AC_FUNC_MALLOC
 -AC_FUNC_SELECT_ARGTYPES
 -AC_CHECK_FUNCS([gethostbyaddr gethostbyname gettimeofday inet_ntoa 
 memset select socket strcspn strdup strerror strncasecmp strcasecmp strspn])
 -
  have_myhostname=yes
  fi
  AM_CONDITIONAL(HAVE_MYHOSTNAME, [test $have_myhostname = yes])
 -- 
 2.4.1
 
 ___
 systemd-devel mailing list
 systemd-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/systemd-devel


Lennart

-- 
Lennart Poettering, Red Hat
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel


Re: [systemd-devel] [PATCH] buildsys: remove always true autoconf checks

2015-05-21 Thread Cristian Rodríguez
On Thu, May 21, 2015 at 11:39 AM, Lennart Poettering
lenn...@poettering.net wrote:
 On Tue, 19.05.15 20:17, Cristian Rodríguez (crrodrig...@opensuse.org) wrote:

 All this checks are always true in any modernish linux system.
 ---
  configure.ac | 11 ---
  1 file changed, 11 deletions(-)

 diff --git a/configure.ac b/configure.ac
 index 3efee22..cd6375b 100644
 --- a/configure.ac
 +++ b/configure.ac
 @@ -1293,17 +1293,6 @@ AC_CHECK_HEADERS_ONCE([valgrind/memcheck.h 
 valgrind/valgrind.h])
  have_myhostname=no
  AC_ARG_ENABLE(myhostname, AS_HELP_STRING([--disable-myhostname], [disable 
 nss-myhostname support]))
  if test x$enable_myhostname != xno; then
 -AC_HEADER_STDC
 -AC_CHECK_HEADERS([arpa/inet.h fcntl.h inttypes.h netdb.h
 netinet/in.h stdlib.h string.h sys/socket.h sys/time.h unistd.h
 nss.h sys/ioctl.h sys/auxv.h])

 sys/auxv.h at least is a relatively recent addition to glibc, hence
 simply removing this all appears too broad...

Right.. that's an oversight..


 Not sure however why this is in the enable_myhostname part however, it
 should just be moved up. Could you prep a patch that does that?

Sure.. stay tuned ;)
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel


[systemd-devel] [PATCH] buildsys: remove always true autoconf checks

2015-05-19 Thread Cristian Rodríguez
All this checks are always true in any modernish linux system.
---
 configure.ac | 11 ---
 1 file changed, 11 deletions(-)

diff --git a/configure.ac b/configure.ac
index 3efee22..cd6375b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1293,17 +1293,6 @@ AC_CHECK_HEADERS_ONCE([valgrind/memcheck.h 
valgrind/valgrind.h])
 have_myhostname=no
 AC_ARG_ENABLE(myhostname, AS_HELP_STRING([--disable-myhostname], [disable 
nss-myhostname support]))
 if test x$enable_myhostname != xno; then
-AC_HEADER_STDC
-AC_CHECK_HEADERS([arpa/inet.h fcntl.h inttypes.h netdb.h netinet/in.h 
stdlib.h string.h sys/socket.h sys/time.h unistd.h nss.h sys/ioctl.h 
sys/auxv.h])
-
-AC_C_CONST
-AC_TYPE_SIZE_T
-AC_HEADER_TIME
-
-AC_FUNC_MALLOC
-AC_FUNC_SELECT_ARGTYPES
-AC_CHECK_FUNCS([gethostbyaddr gethostbyname gettimeofday inet_ntoa 
memset select socket strcspn strdup strerror strncasecmp strcasecmp strspn])
-
 have_myhostname=yes
 fi
 AM_CONDITIONAL(HAVE_MYHOSTNAME, [test $have_myhostname = yes])
-- 
2.4.1

___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel


Re: [systemd-devel] [PATCH] buildsys: remove always true autoconf checks

2015-05-19 Thread systemd github import bot
Patchset imported to github.
Pull request:
https://github.com/systemd-devs/systemd/compare/master...systemd-mailing-devs:1432077432-9267-1-git-send-email-crrodriguez%40opensuse.org

--
Generated by https://github.com/haraldh/mail2git
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel