# [PATCH] Conditionally define _GNU_SOURCE in mailinfo.c

2 messages from 2006-04-30 to 2006-04-30. Participants: Art Haas, Junio C Hamano.
Thread: https://gitlist.dev/t/4012

## Art Haas, 2006-04-30 18:19

Subject: [PATCH] Conditionally define _GNU_SOURCE in mailinfo.c
Message-ID: <20060430181931.GE6626@artsapartment.org>
URL: https://gitlist.dev/e/20060430181931.GE6626%40artsapartment.org

```
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
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, 2006-04-30 21:51

Subject: Re: [PATCH] Conditionally define _GNU_SOURCE in mailinfo.c
Message-ID: <7vodyiybnr.fsf@assigned-by-dhcp.cox.net>
URL: https://gitlist.dev/e/7vodyiybnr.fsf%40assigned-by-dhcp.cox.net
In-Reply-To: <20060430181931.GE6626@artsapartment.org>

```
"Art Haas" <ahaas@airmail.net> writes:

> 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...

```