Re: [PATCH] compat-util: add _DEFAULT_SOURCE define

2014-09-16 Thread Sergey Senozhatsky
Hello,

On (09/15/14 12:02), Junio C Hamano wrote:
 Date: Mon, 15 Sep 2014 12:02:33 -0700
 From: Junio C Hamano gits...@pobox.com
 To: Sergey Senozhatsky sergey.senozhat...@gmail.com
 Cc: git@vger.kernel.org
 Subject: Re: [PATCH] compat-util: add _DEFAULT_SOURCE define
 User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux)
 
 Sergey Senozhatsky sergey.senozhat...@gmail.com writes:
 
  glibc has deprecated the use of _BSD_SOURCE define
 
warning _BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE
 
  To make it easier to maintain a cross platform source code, that
  warning can be suppressed by _DEFAULT_SOURCE.
 
  Define both _BSD_SOURCE, _DEFAULT_SOURCE and cleanup the build.
 
 I can see you defined DEFAULT_SOURCE, but where did you do cleanup
 the build?  Or did you mean define both (in order) to clean up?

yes. sorry, I meant in order to clean up. otherwise, the build is
a bit noisy:

[..]
CC base85.o
CC bisect.o
* new link flags
* new prefix flags
CC branch.o
CC blob.o
In file included from /usr/include/unistd.h:25:0,
 from git-compat-util.h:98,
 from cache.h:4,
 from blob.c:1:
/usr/include/features.h:148:3: warning: #warning _BSD_SOURCE and _SVID_SOURCE 
are deprecated, use _DEFAULT_SOURCE [-Wcpp]
 # warning _BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE
   ^
In file included from /usr/include/unistd.h:25:0,
 from git-compat-util.h:98,
 from branch.c:1:
/usr/include/features.h:148:3: warning: #warning _BSD_SOURCE and _SVID_SOURCE 
are deprecated, use _DEFAULT_SOURCE [-Wcpp]
 # warning _BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE
   ^
In file included from /usr/include/unistd.h:25:0,
 from git-compat-util.h:98,
 from cache.h:4,
 from base85.c:1:
/usr/include/features.h:148:3: warning: #warning _BSD_SOURCE and _SVID_SOURCE 
are deprecated, use _DEFAULT_SOURCE [-Wcpp]
 # warning _BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE
   ^
In file included from /usr/include/unistd.h:25:0,
 from git-compat-util.h:98,
 from cache.h:4,
 from bisect.c:1:
/usr/include/features.h:148:3: warning: #warning _BSD_SOURCE and _SVID_SOURCE 
are deprecated, use _DEFAULT_SOURCE [-Wcpp]
 # warning _BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE
   ^

[..]

 Just making sure we are not only seeing a patch half eaten by mail
 transport somewhere.

sure.

-ss

 
  Signed-off-by: Sergey Senozhatsky sergey.senozhat...@gmail.com
  ---
   git-compat-util.h | 1 +
   1 file changed, 1 insertion(+)
 
  diff --git a/git-compat-util.h b/git-compat-util.h
  index 4e7e3f8..08a9ee2 100644
  --- a/git-compat-util.h
  +++ b/git-compat-util.h
  @@ -82,6 +82,7 @@
   #define _ALL_SOURCE 1
   #define _GNU_SOURCE 1
   #define _BSD_SOURCE 1
  +#define _DEFAULT_SOURCE 1
   #define _NETBSD_SOURCE 1
   #define _SGI_SOURCE 1
 
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] compat-util: add _DEFAULT_SOURCE define

2014-09-15 Thread Junio C Hamano
Sergey Senozhatsky sergey.senozhat...@gmail.com writes:

 glibc has deprecated the use of _BSD_SOURCE define

   warning _BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE

 To make it easier to maintain a cross platform source code, that
 warning can be suppressed by _DEFAULT_SOURCE.

 Define both _BSD_SOURCE, _DEFAULT_SOURCE and cleanup the build.

I can see you defined DEFAULT_SOURCE, but where did you do cleanup
the build?  Or did you mean define both (in order) to clean up?

Just making sure we are not only seeing a patch half eaten by mail
transport somewhere.


 Signed-off-by: Sergey Senozhatsky sergey.senozhat...@gmail.com
 ---
  git-compat-util.h | 1 +
  1 file changed, 1 insertion(+)

 diff --git a/git-compat-util.h b/git-compat-util.h
 index 4e7e3f8..08a9ee2 100644
 --- a/git-compat-util.h
 +++ b/git-compat-util.h
 @@ -82,6 +82,7 @@
  #define _ALL_SOURCE 1
  #define _GNU_SOURCE 1
  #define _BSD_SOURCE 1
 +#define _DEFAULT_SOURCE 1
  #define _NETBSD_SOURCE 1
  #define _SGI_SOURCE 1
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] compat-util: add _DEFAULT_SOURCE define

2014-09-13 Thread Sergey Senozhatsky
glibc has deprecated the use of _BSD_SOURCE define

  warning _BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE

To make it easier to maintain a cross platform source code, that
warning can be suppressed by _DEFAULT_SOURCE.

Define both _BSD_SOURCE, _DEFAULT_SOURCE and cleanup the build.

Signed-off-by: Sergey Senozhatsky sergey.senozhat...@gmail.com
---
 git-compat-util.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/git-compat-util.h b/git-compat-util.h
index 4e7e3f8..08a9ee2 100644
--- a/git-compat-util.h
+++ b/git-compat-util.h
@@ -82,6 +82,7 @@
 #define _ALL_SOURCE 1
 #define _GNU_SOURCE 1
 #define _BSD_SOURCE 1
+#define _DEFAULT_SOURCE 1
 #define _NETBSD_SOURCE 1
 #define _SGI_SOURCE 1
 
-- 
2.1.0.350.g8b25fe0

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html