threads / patch / 4012

patchConditionally define _GNU_SOURCE in mailinfo.c

Subject: [PATCH] Conditionally define _GNU_SOURCE in mailinfo.c

## tl;dr

2 messages between Apr 30, 2006 and Apr 30, 2006. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Art Haas· Apr 30, 2006, 18:19 UTC · lore
Hi.

Place the '#define _GNU_SOURCE' within an #ifndef/#endif block testing if '_GNU_SOURCE' is already defined.

Signed-off-by: Art Haas <ahaas@airmail.net>
---
 mailinfo.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
d3932a8fca2faac8ed14bfd0e65c1f0135dc1d4b
Show changes to mailinfo.c +2 −0
diff --git a/mailinfo.c b/mailinfo.c
index b276519..4b99ccf 100644
--- a/mailinfo.c
+++ b/mailinfo.c
@@ -2,7 +2,9 @@
  * Another stupid program, this one parsing the headers of an
  * email to figure out authorship and subject
  */
+#ifndef _GNU_SOURCE
 #define _GNU_SOURCE
+#endif
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-- 
1.3.1.g66ae

-- 
Man once surrendering his reason, has no remaining guard against absurdities
the most monstrous, and like a ship without rudder, is the sport of every wind.

-Thomas Jefferson to James Smith, 1822
Junio C Hamano· Apr 30, 2006, 21:51 UTC · re: Art Haas · lore

Re: [PATCH] Conditionally define _GNU_SOURCE in mailinfo.c

"Art Haas" <ahaas@airmail.net> writes:
Show 15 quoted lines
> Place the '#define _GNU_SOURCE' within an #ifndef/#endif block testing
> if '_GNU_SOURCE' is already defined.
>
> d3932a8fca2faac8ed14bfd0e65c1f0135dc1d4b
> diff --git a/mailinfo.c b/mailinfo.c
> index b276519..4b99ccf 100644
> --- a/mailinfo.c
> +++ b/mailinfo.c
> @@ -2,7 +2,9 @@
>   * Another stupid program, this one parsing the headers of an
>   * email to figure out authorship and subject
>   */
> +#ifndef _GNU_SOURCE
>  #define _GNU_SOURCE
> +#endif

Hmph.... I understand that this makes any difference only if you use -D_GNU_SOURCE in CFLAGS, which implies there are _other_ things you need to have "#define _GNU_SOURCE" to compile for your setup. I wonder what it is...

← back to recent threads