threads / patch / 27646

patchsh-i18n--envsubst.c: do not #include getopt.h

Subject: [PATCH] sh-i18n--envsubst.c: do not #include getopt.h

## tl;dr

2 messages between Jun 17, 2011 and Jun 17, 2011. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Brandon Casey· Jun 17, 2011, 18:19 UTC · lore

From: Brandon Casey <drafnel@gmail.com>

The getopt.h header file is not used. It's inclusion is left over from the original version of this source. Additionally, getopt.h does not exist on all platforms (SunOS 5.7) and will cause a compilation failure. So, let's remove it.

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
---
 sh-i18n--envsubst.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

Show changes to sh-i18n--envsubst.c +0 −1

diff --git a/sh-i18n--envsubst.c b/sh-i18n--envsubst.c
index 2eb0ee4..9d2e971 100644
--- a/sh-i18n--envsubst.c
+++ b/sh-i18n--envsubst.c
@@ -51,7 +51,6 @@
    Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.  */
 
 #include <errno.h>
-#include <getopt.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

-- 
1.7.6.rc1.3.g600a62

Junio C Hamano· Jun 17, 2011, 21:28 UTC · re: Brandon Casey · lore

Re: [PATCH] sh-i18n--envsubst.c: do not #include getopt.h

Brandon Casey <casey@nrlssc.navy.mil> writes:

Show 8 quoted lines

> From: Brandon Casey <drafnel@gmail.com>
>
> The getopt.h header file is not used.  It's inclusion is left over from the
> original version of this source.  Additionally, getopt.h does not exist on
> all platforms (SunOS 5.7) and will cause a compilation failure.  So, let's
> remove it.
>
> Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>

Thanks; will queue.